Skip to content

Commit 26d82dd

Browse files
committed
Omit storageClassName if it is null
Fixes #226
1 parent b1294cb commit 26d82dd

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

charts/helm-dashboard/templates/pvc.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,10 @@ spec:
1414
{{- if .Values.dashboard.persistence.hostPath }}
1515
storageClassName: ""
1616
{{- else }}
17+
{{- if kindIs "string" .Values.dashboard.persistence.storageClass }}
1718
storageClassName: "{{ .Values.dashboard.persistence.storageClass }}"
1819
{{- end }}
20+
{{- end }}
1921
accessModes:
2022
{{- if not (empty .Values.dashboard.persistence.accessModes) }}
2123
{{- range .Values.dashboard.persistence.accessModes }}

charts/helm-dashboard/values.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ dashboard:
4343
## set, choosing the default provisioner. (gp2 on AWS, standard on
4444
## GKE, AWS & OpenStack)
4545
##
46-
storageClass: ""
46+
storageClass: null
4747

4848
## Helm Dashboard Persistent Volume access modes
4949
## Must match those of existing PV or dynamic provisioner

0 commit comments

Comments
 (0)