Skip to content

Commit

Permalink
fix: make secretMount parameters required (opensearch-project#22)
Browse files Browse the repository at this point in the history
This fixes the case where a parameter on one of the items is silently
missing.
  • Loading branch information
smlx authored Sep 10, 2021
1 parent 0ad19d5 commit 5ebed68
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions charts/opensearch/templates/statefulset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -138,9 +138,9 @@ spec:
terminationGracePeriodSeconds: {{ .Values.terminationGracePeriod }}
volumes:
{{- range .Values.secretMounts }}
- name: {{ .name }}
- name: {{ .name | required "secretMount .name is required" }}
secret:
secretName: {{ .secretName }}
secretName: {{ .secretName | required "secretMount .secretName is required" }}
{{- if .defaultMode }}
defaultMode: {{ .defaultMode }}
{{- end }}
Expand Down Expand Up @@ -359,8 +359,8 @@ spec:
{{- end }}
{{- end }}
{{- range .Values.secretMounts }}
- name: {{ .name }}
mountPath: {{ .path }}
- name: {{ .name | required "secretMount .name is required" }}
mountPath: {{ .path | required "secretMount .path is required" }}
{{- if .subPath }}
subPath: {{ .subPath }}
{{- end }}
Expand Down

0 comments on commit 5ebed68

Please sign in to comment.