crc-org / snc

Single Node Cluster creation scripts for OpenShift 4.x as used by CodeReady Containers
https://crc.dev
Apache License 2.0
101 stars 50 forks source link

Update resource requests/limits #272

Open guillaumerose opened 3 years ago

guillaumerose commented 3 years ago

Change

For Kube control plane components launched as static pods For pods across all the openshift namespaces

How

(a) Enabling pre-compiled admission controller ‘MutatingAdmissionWebhook’ to inject required resource requests/limits values for pods across all the openshift namespaces (admission-webhook.yaml) (b) Resources requests/limits are updated in the manifest files of Kube control plane components (launched as static pods by Kubelet) (update-kube-controlplane.sh)

Implemented in https://github.com/code-ready/snc/pull/246

Expected and mesured improvements

Pros

Cons

guillaumerose commented 3 years ago

Minikube has following requests:

  Namespace                  Name                                CPU Requests  CPU Limits  Memory Requests  Memory Limits  AGE
  ---------                  ----                                ------------  ----------  ---------------  -------------  ---
  kube-system                coredns-f9fd979d6-jpgtd             100m (5%)     0 (0%)      70Mi (1%)        170Mi (2%)     16m
  kube-system                etcd-minikube                       0 (0%)        0 (0%)      0 (0%)           0 (0%)         16m
  kube-system                kube-apiserver-minikube             250m (12%)    0 (0%)      0 (0%)           0 (0%)         16m
  kube-system                kube-controller-manager-minikube    200m (10%)    0 (0%)      0 (0%)           0 (0%)         16m
  kube-system                kube-proxy-8dcpm                    0 (0%)        0 (0%)      0 (0%)           0 (0%)         16m
  kube-system                kube-scheduler-minikube             100m (5%)     0 (0%)      0 (0%)           0 (0%)         16m
  kube-system                storage-provisioner                 0 (0%)        0 (0%)      0 (0%)           0 (0%)         16m