Skip to content

Commit

Permalink
[Release-v0.26] resources, Update minimum cpu / memory requests (#321)
Browse files Browse the repository at this point in the history
* resources, Update cert-manager cpu / memory requests

According empiric use measures.

See https://bugzilla.redhat.com/show_bug.cgi?id=1935218

Signed-off-by: Or Shoval <[email protected]>

* mac-controller-manager, Update minimum memory request

Update minimum memory request according empiric measures,
in order to reduce idle usage and minimum required memory
for scheduling kubemacpool.

Signed-off-by: Or Shoval <[email protected]>

* Update test/release folder

By running Make generate.

Signed-off-by: Or Shoval <[email protected]>

* cert-manager, Update minimum CPU request

In order to allow the cert-manager to give faster SLA,
as the webhook (and handler) depends on it,
increase minimum memory and cpu requests.

See nmstate/kubernetes-nmstate#781 (comment)

Signed-off-by: Or Shoval <[email protected]>
  • Loading branch information
oshoval authored Jul 14, 2021
1 parent 34127b0 commit 43d107a
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 3 deletions.
6 changes: 5 additions & 1 deletion config/default/manager/manager.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ spec:
resources:
requests:
cpu: 100m
memory: 300Mi
memory: 100Mi
ports:
- containerPort: 8000
name: webhook-server
Expand Down Expand Up @@ -149,6 +149,10 @@ spec:
image: quay.io/kubevirt/kubemacpool:latest
imagePullPolicy: Always
name: manager
resources:
requests:
cpu: "30m"
memory: "30Mi"
env:
- name: RUN_CERT_MANAGER
value: ""
Expand Down
6 changes: 5 additions & 1 deletion config/release/kubemacpool.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -273,6 +273,10 @@ spec:
image: quay.io/kubevirt/kubemacpool:latest
imagePullPolicy: Always
name: manager
resources:
requests:
cpu: 30m
memory: 30Mi
priorityClassName: system-cluster-critical
restartPolicy: Always
terminationGracePeriodSeconds: 5
Expand Down Expand Up @@ -361,7 +365,7 @@ spec:
resources:
requests:
cpu: 100m
memory: 300Mi
memory: 100Mi
volumeMounts:
- mountPath: /tmp/k8s-webhook-server/serving-certs/
name: tls-key-pair
Expand Down
6 changes: 5 additions & 1 deletion config/test/kubemacpool.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -274,6 +274,10 @@ spec:
image: registry:5000/kubevirt/kubemacpool:latest
imagePullPolicy: Always
name: manager
resources:
requests:
cpu: 30m
memory: 30Mi
priorityClassName: system-cluster-critical
restartPolicy: Always
terminationGracePeriodSeconds: 5
Expand Down Expand Up @@ -362,7 +366,7 @@ spec:
resources:
requests:
cpu: 100m
memory: 300Mi
memory: 100Mi
volumeMounts:
- mountPath: /tmp/k8s-webhook-server/serving-certs/
name: tls-key-pair
Expand Down

0 comments on commit 43d107a

Please sign in to comment.