Skip to content

Commit

Permalink
make volume and volume mounts configurable through helm
Browse files Browse the repository at this point in the history
  • Loading branch information
eljohnson92 committed Oct 16, 2024
1 parent 708ee98 commit f158daf
Show file tree
Hide file tree
Showing 2 changed files with 23 additions and 6 deletions.
12 changes: 9 additions & 3 deletions deploy/chart/templates/daemonset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,9 @@ spec:
volumeMounts:
- mountPath: /etc/kubernetes
name: k8s
{{- with .Values.volumeMounts}}
{{- toYaml . | nindent 12 }}
{{- end}}
env:
- name: LINODE_API_TOKEN
valueFrom:
Expand All @@ -65,10 +68,13 @@ spec:
secretKeyRef:
name: {{ if .Values.secretRef }}{{ .Values.secretRef.name | default "ccm-linode" }}{{ else }}"ccm-linode"{{ end }}
key: {{ if .Values.secretRef }}{{ .Values.secretRef.regionRef | default "region" }}{{ else }}"region"{{ end }}
{{if .Values.env}}
{{- toYaml .Values.env | nindent 12 }}
{{end}}
{{- with .Values.env}}
{{- toYaml . | nindent 12 }}
{{- end}}
volumes:
- name: k8s
hostPath:
path: /etc/kubernetes
{{- with .Values.volumes}}
{{- toYaml . | nindent 8 }}
{{- end}}
17 changes: 14 additions & 3 deletions deploy/chart/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,17 @@ tolerations:
# This section adds the ability to pass environment variables to adjust CCM defaults
# https://github.com/linode/linode-cloud-controller-manager/blob/master/cloud/linode/loadbalancers.go
# LINODE_HOSTNAME_ONLY_INGRESS type bool is supported
# env:
# - name: EXAMPLE_ENV_VAR
# value: "true"
env:
- name: EXAMPLE_ENV_VAR

Check failure on line 62 in deploy/chart/values.yaml

View workflow job for this annotation

GitHub Actions / helm-test

62:4 [indentation] wrong indentation: expected 2 but found 3
value: "true"

# This section adds the ability to pass volumes to the CCM DaemonSet
volumes:
- name: test-volume

Check failure on line 67 in deploy/chart/values.yaml

View workflow job for this annotation

GitHub Actions / helm-test

67:4 [indentation] wrong indentation: expected 2 but found 3
emptyDir:
sizeLimit: 500Mi

# this section adds the ability to pass volumeMounts to the CCM container
volumeMounts:
- mountPath: /tmp/

Check failure on line 73 in deploy/chart/values.yaml

View workflow job for this annotation

GitHub Actions / helm-test

73:4 [indentation] wrong indentation: expected 2 but found 3
name: test-volume

0 comments on commit f158daf

Please sign in to comment.