Skip to content

Commit

Permalink
Adjust resource requests and autoscaling of calico-node side car
Browse files Browse the repository at this point in the history
containers.

The side car containers use negligible amount of resources compared to
`calico-node`. Therefore, requests likely just waste resources.
Similarly, they fall below the minimum threshold of VPA. Hence,
autoscaling them does not make sense.
`minAllowed` and `maxAllowed` of `calico-node` also are not that useful.
Therefore, they are removed as well.
  • Loading branch information
ScheererJ committed Sep 19, 2024
1 parent 9364c26 commit 5097a0f
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 28 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -171,10 +171,6 @@ spec:
env:
- name: POD_CIDR
value: {{ .Values.global.podCIDR }}
resources:
requests:
cpu: 10m
memory: 50Mi
command:
- /bin/sh
- -c
Expand Down Expand Up @@ -238,10 +234,6 @@ spec:
valueFrom:
fieldRef:
fieldPath: spec.nodeName
resources:
requests:
cpu: 1m
memory: 20Mi
command:
- /bin/sh
- -c
Expand Down
20 changes: 0 additions & 20 deletions charts/internal/calico/templates/node/vpa.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,25 +14,5 @@ spec:
resourcePolicy:
containerPolicies:
- containerName: calico-node
minAllowed:
memory: 50Mi
maxAllowed:
memory: 2800Mi
controlledValues: RequestsOnly
- containerName: "add-snat-rule-to-upstream-dns"
minAllowed:
cpu: 10m
memory: 50Mi
maxAllowed:
cpu: 1
memory: 1G
controlledValues: RequestsOnly
- containerName: "network-unavailable-condition-ensurer"
minAllowed:
cpu: 1m
memory: 20Mi
maxAllowed:
cpu: 10m
memory: 100Mi
controlledValues: RequestsOnly
{{- end }}

0 comments on commit 5097a0f

Please sign in to comment.