other: Provide option to add environment variables to only supersetNo…

…de (apache#16627) * Add env variables for supersetNode in Helm template * Typo in template * Removed unnecessary if * Revert "Removed unnecessary if" This reverts commit af83568. * Bumping chart version to 0.3.8 Co-authored-by: Craig Rueda <[email protected]>
opus-42 · Nov 14, 2021 · 5d656c8 · 5d656c8
1 parent 37d706e
commit 5d656c8
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 1 deletion.
diff --git a/helm/superset/Chart.yaml b/helm/superset/Chart.yaml
@@ -22,7 +22,7 @@ maintainers:
   - name: craig-rueda
     email: [email protected]
     url: https://github.com/craig-rueda
-version: 0.3.7
+version: 0.3.8
 dependencies:
 - name: postgresql
   version: 10.2.0

diff --git a/helm/superset/templates/deployment.yaml b/helm/superset/templates/deployment.yaml
@@ -82,6 +82,12 @@ spec:
               value: {{ $value | quote }}
             {{- end }}
           {{- end }}
+          {{- if .Values.supersetNode.env }}
+            {{- range $key, $value := .Values.supersetNode.env }}
+            - name: {{ $key | quote}}
+              value: {{ $value | quote }}
+            {{- end }}
+          {{- end }}
           envFrom:
             - secretRef:
                 name: {{ tpl .Values.envFromSecret . | quote }}

diff --git a/helm/superset/values.yaml b/helm/superset/values.yaml
@@ -207,6 +207,7 @@ supersetNode:
     db_user: superset
     db_pass: superset
     db_name: superset
+  env: {}
   forceReload: false # If true, forces deployment to reload on each upgrade
   initContainers:
     - name: wait-for-postgres