fix: leaving storage class empty would result in disabling dynamic pr…

…ovisioning
wearefrank · Jan 4, 2024 · fde4df3 · fde4df3
1 parent 545b2f4
commit fde4df3
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 1 deletion.
diff --git a/charts/drill/Chart.yaml b/charts/drill/Chart.yaml
@@ -1,6 +1,6 @@
 apiVersion: v2
 name: drill
-version: 1.2.12
+version: 1.2.13
 appVersion: 1.21.1
 description: Helm Charts for deploying Apache Drill Clusters on Kubernetes
 icon: https://raw.githubusercontent.com/wearefrank/charts/master/charts/drill/icon.svg

diff --git a/charts/drill/templates/persistenvolumeclaim.yaml b/charts/drill/templates/persistenvolumeclaim.yaml
@@ -18,7 +18,13 @@ spec:
   selector:
     {{- toYaml . | nindent 6 }}
   {{- end }}
+  {{- if .Values.persistence.storageClass }}
+  {{- if eq "-" .Values.persistence.storageClass}}
+  storageClassName: ""
+  {{- else }}
   storageClassName: {{ .Values.persistence.storageClass | quote }}
+  {{- end }}
+  {{- end }}
   resources:
     requests:
       storage: {{ .Values.persistence.size | quote }}

diff --git a/charts/drill/templates/statefulset.yaml b/charts/drill/templates/statefulset.yaml
@@ -164,7 +164,13 @@ spec:
         selector:
           {{- toYaml . | nindent 10 }}
         {{- end }}
+        {{- if .Values.persistence.storageClass }}
+        {{- if eq "-" .Values.persistence.storageClass}}
+        storageClassName: ""
+        {{- else }}
         storageClassName: {{ .Values.persistence.storageClass | quote }}
+        {{- end }}
+        {{- end }}
         resources:
           requests:
             storage: {{ .Values.persistence.dataLogDir.size | quote }}