kyma-project / btp-manager

BTP Manager installs SAP BTP Service Operator as a Kyma module
Apache License 2.0
6 stars 17 forks source link

No priority class for BTP Manager #765

Closed ralikio closed 2 months ago

ralikio commented 3 months ago

An invalid K8S cronjob with jobs that do not properly close its containers may result in cluster being filled with running workloads that unnecessarily consume cluster resources. If there are many of such pods than K8S scheduler is not scheduling BTP Manager's pods. Add a priority class that will put enough priority on btp-manager-controller-manager and sap-btp-operator-controller-manager so that in similar cases they have more chance to be scheduled than problematic workloads. See Istio Module's priority class configuration for reference.

➜  ui git:(features/enable-to-load-test-data) ✗ kubectl get deployments -A -o custom-columns="NAME:.metadata.name,PRIORITY_CLASS:.spec.template.spec.priorityClassName"
NAME                                  PRIORITY_CLASS
btp-manager-controller-manager        <none>
sap-btp-operator-controller-manager   <none>

AC

MarekMichali commented 2 months ago

Most modules use a priority class with a value of 2000000 or 2100000.

MarekMichali commented 2 months ago

We decided to use the name btp-manager-kyma-priority and value 2100000.