From 26d82dd5ab0690a55be2740339cecb065e66d78c Mon Sep 17 00:00:00 2001
From: Andrei Pohilko <andrei.pokhilko@gmail.com>
Date: Tue, 11 Apr 2023 12:10:30 +0100
Subject: [PATCH] Omit storageClassName if it is null

Fixes #226
---
 charts/helm-dashboard/templates/pvc.yaml | 2 ++
 charts/helm-dashboard/values.yaml        | 2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/charts/helm-dashboard/templates/pvc.yaml b/charts/helm-dashboard/templates/pvc.yaml
index 96d7336..68e0df3 100644
--- a/charts/helm-dashboard/templates/pvc.yaml
+++ b/charts/helm-dashboard/templates/pvc.yaml
@@ -14,8 +14,10 @@ spec:
   {{- if .Values.dashboard.persistence.hostPath }}
   storageClassName: ""
   {{- else }}
+  {{- if kindIs "string" .Values.dashboard.persistence.storageClass }}
   storageClassName: "{{ .Values.dashboard.persistence.storageClass }}"
   {{- end }}
+  {{- end }}
   accessModes:
   {{- if not (empty .Values.dashboard.persistence.accessModes) }}
   {{- range .Values.dashboard.persistence.accessModes }}
diff --git a/charts/helm-dashboard/values.yaml b/charts/helm-dashboard/values.yaml
index aa795db..15a308c 100644
--- a/charts/helm-dashboard/values.yaml
+++ b/charts/helm-dashboard/values.yaml
@@ -43,7 +43,7 @@ dashboard:
     ##   set, choosing the default provisioner.  (gp2 on AWS, standard on
     ##   GKE, AWS & OpenStack)
     ##
-    storageClass: ""
+    storageClass: null
 
     ## Helm Dashboard Persistent Volume access modes
     ## Must match those of existing PV or dynamic provisioner