[superset] branch master updated: Force pod restart on config changes (#13056)

[cr...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

craigrueda pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/superset.git


The following commit(s) were added to refs/heads/master by this push:
     new acca3a4  Force pod restart on config changes (#13056)
acca3a4 is described below

commit acca3a4431c5dd46ee52f4fb3b0e836fac9ccc4b
Author: Yann Jouanique <ya...@oneacrefund.org>
AuthorDate: Sat Feb 13 02:30:03 2021 +0200

    Force pod restart on config changes (#13056)
---
 helm/superset/templates/deployment.yaml | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/helm/superset/templates/deployment.yaml b/helm/superset/templates/deployment.yaml
index 12a71fa..b105334 100644
--- a/helm/superset/templates/deployment.yaml
+++ b/helm/superset/templates/deployment.yaml
@@ -31,11 +31,17 @@ spec:
       release: {{ .Release.Name }}
   template:
     metadata:
-      {{ if .Values.supersetNode.forceReload }}
       annotations:
-        # Optionally force the thing to reload
+        # Force reload on config changes
+        checksum/superset_config.py: {{ include "superset-config" . | sha256sum }}
+        checksum/superset_init.sh: {{ tpl .Values.init.initscript . | sha256sum }}
+        checksum/superset_bootstrap.sh: {{ include "superset-bootstrap" . | sha256sum }}
+        checksum/connections: {{ .Values.supersetNode.connections | toYaml | sha256sum }}
+        checksum/extraConfigs: {{ .Values.extraConfigs | toYaml | sha256sum }}
+        {{- if .Values.supersetNode.forceReload }}
+        # Optionally force the thing to reload unconditionally
         force-reload: {{ randAlphaNum 5 | quote }}
-      {{ end }}
+       {{- end }}
       labels:
         app: {{ template "superset.name" . }}
         release: {{ .Release.Name }}