Skip to content

Commit

Permalink
[chart] Allow resources override for node DaemonSet
Browse files Browse the repository at this point in the history
CSI controller and node have different needs in a manner of capacity so
in this commit we enable users to define specific resources for the node
component. This will allow users not to reserve not needed resources on
all of their instances as node is a DaemonSet and may not need as much
CPU/Memory as the controller Pods.

Signed-off-by: dntosas <[email protected]>
  • Loading branch information
dntosas committed Feb 9, 2021
1 parent 4790c45 commit e81e584
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 4 deletions.
2 changes: 1 addition & 1 deletion charts/aws-ebs-csi-driver/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v1
appVersion: "0.9.0"
name: aws-ebs-csi-driver
description: A Helm chart for AWS EBS CSI Driver
version: 0.9.4
version: 0.9.5
kubeVersion: ">=1.17.0-0"
home: https://github.com/kubernetes-sigs/aws-ebs-csi-driver
sources:
Expand Down
6 changes: 3 additions & 3 deletions charts/aws-ebs-csi-driver/templates/node.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ spec:
timeoutSeconds: 3
periodSeconds: 10
failureThreshold: 5
{{- with .Values.resources }}
{{- with .Values.node.resources }}
resources: {{ toYaml . | nindent 12 }}
{{- end }}
- name: node-driver-registrar
Expand All @@ -103,7 +103,7 @@ spec:
mountPath: /csi
- name: registration-dir
mountPath: /registration
{{- with .Values.resources }}
{{- with .Values.node.resources }}
resources: {{ toYaml . | nindent 12 }}
{{- end }}
- name: liveness-probe
Expand All @@ -113,7 +113,7 @@ spec:
volumeMounts:
- name: plugin-dir
mountPath: /csi
{{- with .Values.resources }}
{{- with .Values.node.resources }}
resources: {{ toYaml . | nindent 12 }}
{{- end }}
volumes:
Expand Down
1 change: 1 addition & 0 deletions charts/aws-ebs-csi-driver/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -91,6 +91,7 @@ node:
podAnnotations: {}
tolerateAllTaints: true
tolerations: []
resources: {}

serviceAccount:
controller:
Expand Down

0 comments on commit e81e584

Please sign in to comment.