kyma-project / kyma

Kyma is an opinionated set of Kubernetes-based modular building blocks, including all necessary capabilities to develop and run enterprise-grade cloud-native applications.
https://kyma-project.io
Apache License 2.0
1.52k stars 405 forks source link

Remove Service Catalog area from Kyma opensource #13422

Closed PK85 closed 2 years ago

PK85 commented 2 years ago

Description

With Kyma 2.2 release we plan to remove Service Catalog area from this project.

Reasons

AC

klaudiagrz commented 2 years ago

Please remember to include the note about SC removal in the release notes 🙂

wozniakjan commented 2 years ago

TODO: check how much less resources (CPU and memory) Kyma will use after the service catalog removal.

@PK85, here are some statistics from an idle freshly provisioned cluster

$ k top pod service-catalog-catalog-controller-manager-7d96d47bbb-p9lxm
NAME                                                          CPU(cores)   MEMORY(bytes)
service-catalog-catalog-controller-manager-7d96d47bbb-p9lxm   3m           81Mi
$ k top pod service-catalog-catalog-webhook-5d7487cb95-zfvmk
NAME                                               CPU(cores)   MEMORY(bytes)
service-catalog-catalog-webhook-5d7487cb95-zfvmk   1m           11Mi
$ k top pod service-catalog-addons-service-binding-usage-controller-5cnw7ch
NAME                                                              CPU(cores)   MEMORY(bytes)
service-catalog-addons-service-binding-usage-controller-5cnw7ch   3m           87Mi

we also set some requests and limits on these

$ k get deployment service-catalog-addons-service-binding-usage-controller service-catalog-catalog-controller-manager service-catalog-catalog-webhook -o json | jq -c '.items[].spec.template.spec.containers[] | {"name":.name,"resources":.resources}'
{"name":"service-binding-usage-controller","resources":{"limits":{"cpu":"100m","memory":"96Mi"},"requests":{"cpu":"50m","memory":"48Mi"}}}
{"name":"controller-manager","resources":{"limits":{"cpu":"100m","memory":"128Mi"},"requests":{"cpu":"50m","memory":"48Mi"}}}
{"name":"svr","resources":{"limits":{"cpu":"100m","memory":"48Mi"},"requests":{"cpu":"50m","memory":"30Mi"}}}
vlerenc commented 2 years ago

Can you please give an estimate when you can mitigate this? Kyma clusters burn through 32+ cores, because API servers request up to 12 cores to process these requests. That's burning money on our/SAP side. That it is rooted in a Kubernetes issue (https://github.com/kubernetes/kubernetes/issues/109099) is not helping and we would ask you to please increase the priority and provide a fix.

wozniakjan commented 2 years ago

please increase the priority and provide a fix.

there is not going to be any fix, service catalog has been deprecated in kyma for a long time, this ticket is about forcefully removing it and for managed kyma instances there is a managed migration as well. Non-managed kymas will need to migrate on their own using https://github.com/SAP/sap-btp-service-operator-migration

Can you please give an estimate when you can mitigate this?

anyone is free to "mitigate" already, the team around Service Management platform has implemented a migration procedure and you can use https://github.com/SAP/sap-btp-service-operator-migration from them to migrate and then delete/disable service catalog at any point since October. For managed kymas the managed migration is already in progress, but exact dates depend on customers, BTP platform, their availability and individual contracts.

vlerenc commented 2 years ago

@wozniakjan I don't know what you are trying to say. Kyma clusters burn through 32+ cores, because API servers request up to 12 cores to process requests. What is your plan to change that - better yesterday than tomorrow?

Can you not help SAP and conceive a solution to get rid or sufficiently cut down the culprit? I believe, it's this one here from our investigation: data-privacy-integration-service-application (cc @dguendisch @ialidzhikov). It's a 178 kB service plan "toasting" the API server (see https://github.com/kubernetes/kubernetes/issues/109099 for more details what triggers the effect, CRD definition, nesting depth,... maybe you can help with/influence the service plan itself).

This creates high costs for SAP for no good reason and passively waiting for people to migrate away (or not), if that's what you said above, seems wrong.

wozniakjan commented 2 years ago

as far as I know, https://github.com/kubernetes/kubernetes/issues/109099 is not the only issue and by far not even the biggest that SAP has with service catalog. The migration is in progress for managed kymas, there is not going to be a service catalog in kyma anymore. This path was selected a little more than a year ago, currently being finally carried out.

Can you not help SAP and conceive a solution to get rid or sufficiently cut down the culprit?

I am an external contractor from kubermatic, on a temporary part-time contract at SAP, I have not been involved in setting the roadmap for kyma. On the other hand, judging from your title here at github as Tech Lead and Product Owner for Kubernetes/Gardener, Central Engineering, SAP SE, you should have a better and more direct path to the correct stakeholders at SAP internally.

vlerenc commented 2 years ago

you should have a better and more direct path to the correct stakeholders at SAP internally

@wozniakjan Nice one. :-)

Sometimes, I can only try to ask other departments/orgs. I am not responsible for all things Kubernetes at SAP, just working in/for Gardener, not Kyma or HANA or any other stakeholder.