Skip to content

Commit

Permalink
[prometheus-node-exporter] Fix variable scope for sidecar volume moun…
Browse files Browse the repository at this point in the history
…ts (#2825)

* Fix variable scope for sidecar volume mounts

Signed-off-by: jsolbrig <[email protected]>

* Bump chart version to 4.8.1

Signed-off-by: jsolbrig <[email protected]>

Signed-off-by: jsolbrig <[email protected]>
Co-authored-by: André Bauer <[email protected]>
  • Loading branch information
jsolbrig and monotek authored Dec 21, 2022
1 parent 9d8807d commit d3858c1
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion charts/prometheus-node-exporter/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ keywords:
- prometheus
- exporter
type: application
version: 4.8.0
version: 4.8.1
appVersion: 1.5.0
home: https://github.com/prometheus/node_exporter/
sources:
Expand Down
6 changes: 3 additions & 3 deletions charts/prometheus-node-exporter/templates/daemonset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -147,14 +147,14 @@ spec:
{{- end }}
{{- with .Values.sidecars }}
{{- toYaml . | nindent 8 }}
{{- if or .Values.sidecarVolumeMount .Values.sidecarHostVolumeMounts }}
{{- if or $.Values.sidecarVolumeMount $.Values.sidecarHostVolumeMounts }}
volumeMounts:
{{- range $_, $mount := .Values.sidecarVolumeMount }}
{{- range $_, $mount := $.Values.sidecarVolumeMount }}
- name: {{ $mount.name }}
mountPath: {{ $mount.mountPath }}
readOnly: {{ $mount.readOnly }}
{{- end }}
{{- range $_, $mount := .Values.sidecarHostVolumeMounts }}
{{- range $_, $mount := $.Values.sidecarHostVolumeMounts }}
- name: {{ $mount.name }}
mountPath: {{ $mount.mountPath }}
readOnly: {{ $mount.readOnly }}
Expand Down

0 comments on commit d3858c1

Please sign in to comment.