[HELM]: Added checksum config annotation in stateful set for broker, …

…controller and server (#13059) * Added checksum config anotation in statful set for broker, controller and server * Updated as per the PR comment. * remove unneccessry testing file.
apache · May 16, 2024 · 340286b · 340286b
1 parent 49fa66c
commit 340286b
Show file tree

Hide file tree

Showing 5 changed files with 31 additions and 4 deletions.
diff --git a/helm/pinot/templates/broker/statefulset.yaml b/helm/pinot/templates/broker/statefulset.yaml
@@ -37,7 +37,10 @@ spec:
       labels:
         {{- include "pinot.brokerLabels" . | nindent 8 }}
       annotations:
-{{ toYaml .Values.broker.podAnnotations | indent 8 }}
+        {{- if .Values.broker.automaticReload.enabled }}
+        checksum/config: {{- include (print $.Template.BasePath "/broker/configmap.yaml") . | sha256sum }}
+        {{- end }}
+{{ toYaml .Values.broker.podAnnotations | nindent 8 }}
     spec:
       terminationGracePeriodSeconds: {{ .Values.terminationGracePeriodSeconds }}
       serviceAccountName: {{ include "pinot.serviceAccountName" . }}

diff --git a/helm/pinot/templates/controller/statefulset.yaml b/helm/pinot/templates/controller/statefulset.yaml
@@ -37,7 +37,10 @@ spec:
       labels:
         {{- include "pinot.controllerLabels" . | nindent 8 }}
       annotations:
-{{ toYaml .Values.controller.podAnnotations | indent 8 }}
+        {{- if .Values.controller.automaticReload.enabled }}
+        checksum/config: {{- include (print $.Template.BasePath "/controller/configmap.yaml") . | sha256sum }}
+        {{- end }}
+{{ toYaml .Values.controller.podAnnotations | nindent 8 }}
     spec:
       terminationGracePeriodSeconds: {{ .Values.terminationGracePeriodSeconds }}
       serviceAccountName: {{ include "pinot.serviceAccountName" . }}

diff --git a/helm/pinot/templates/minion/statefulset.yaml b/helm/pinot/templates/minion/statefulset.yaml
@@ -38,7 +38,10 @@ spec:
       labels:
         {{- include "pinot.minionLabels" . | nindent 8 }}
       annotations:
-{{ toYaml .Values.minion.podAnnotations | indent 8 }}
+        {{- if .Values.minion.automaticReload.enabled }}
+        checksum/config: {{ include (print $.Template.BasePath "/minion/configmap.yaml") . | sha256sum}}
+        {{- end }}
+{{ toYaml .Values.minion.podAnnotations | nindent 8 }}
     spec:
       terminationGracePeriodSeconds: {{ .Values.terminationGracePeriodSeconds }}
       serviceAccountName: {{ include "pinot.serviceAccountName" . }}

diff --git a/helm/pinot/templates/server/statefulset.yaml b/helm/pinot/templates/server/statefulset.yaml
@@ -37,7 +37,10 @@ spec:
       labels:
         {{- include "pinot.serverLabels" . | nindent 8 }}
       annotations:
-{{ toYaml .Values.server.podAnnotations | indent 8 }}
+        {{- if .Values.server.automaticReload.enabled }}
+        checksum/config: {{- include (print $.Template.BasePath "/server/configmap.yaml") . | sha256sum }}
+        {{- end }}
+{{ toYaml .Values.server.podAnnotations | nindent 8 }}
     spec:
       terminationGracePeriodSeconds: {{ .Values.terminationGracePeriodSeconds }}
       serviceAccountName: {{ include "pinot.serviceAccountName" . }}

diff --git a/helm/pinot/values.yaml b/helm/pinot/values.yaml
@@ -164,6 +164,10 @@ controller:
 
   podAnnotations: {}
 
+  # set enabled as true, to automatically roll controller stateful set for configmap change
+  automaticReload:
+    enabled: false
+
   updateStrategy:
     type: RollingUpdate
 
@@ -269,6 +273,10 @@ broker:
 
   podAnnotations: {}
 
+  # set enabled as true, to automatically roll broker stateful set for configmap change
+  automaticReload:
+    enabled: false
+
   updateStrategy:
     type: RollingUpdate
 
@@ -362,6 +370,10 @@ server:
 
   podAnnotations: {}
 
+  # set enabled as true, to automatically roll server stateful set for configmap change
+  automaticReload:
+    enabled: false
+
   updateStrategy:
     type: RollingUpdate
 
@@ -452,6 +464,9 @@ minion:
 
   podAnnotations: {}
 
+  automaticReload:
+    enabled: false
+
   updateStrategy:
     type: RollingUpdate