chore(helm): add rolling update strategy (#398)

Because - we need to add the maxSurge and maxUnavailable values into the deployment This commit - add rolling update strategy
instill-ai · Dec 15, 2023 · 1dcb0c7 · 1dcb0c7
1 parent 9d56fca
commit 1dcb0c7
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 2 deletions.
diff --git a/charts/vdp/templates/controller-vdp/deployment.yaml b/charts/vdp/templates/controller-vdp/deployment.yaml
@@ -10,7 +10,11 @@ metadata:
 spec:
   strategy:
     type: {{ .Values.updateStrategy.type }}
-    {{- if eq .Values.updateStrategy.type "Recreate" }}
+    {{- if eq .Values.updateStrategy.type "RollingUpdate" }}
+    rollingUpdate:
+      maxSurge: {{ .Values.updateStrategy.rollingUpdate.maxSurge }}
+      maxUnavailable: {{ .Values.updateStrategy.rollingUpdate.maxUnavailable }}
+    {{- else}}
     rollingUpdate: null
     {{- end }}
   {{- if not .Values.controllerVDP.autoscaling.enabled }}

diff --git a/charts/vdp/templates/pipeline-backend/deployment.yaml b/charts/vdp/templates/pipeline-backend/deployment.yaml
@@ -13,7 +13,11 @@ metadata:
 spec:
   strategy:
     type: {{ .Values.updateStrategy.type }}
-    {{- if eq .Values.updateStrategy.type "Recreate" }}
+    {{- if eq .Values.updateStrategy.type "RollingUpdate" }}
+    rollingUpdate:
+      maxSurge: {{ .Values.updateStrategy.rollingUpdate.maxSurge }}
+      maxUnavailable: {{ .Values.updateStrategy.rollingUpdate.maxUnavailable }}
+    {{- else}}
     rollingUpdate: null
     {{- end }}
   {{- if not .Values.pipelineBackend.autoscaling.enabled }}

diff --git a/charts/vdp/values.yaml b/charts/vdp/values.yaml
@@ -8,6 +8,9 @@ fullnameOverride: ""
 # Set it as "Recreate" when "RWM" for volumes isn't supported
 updateStrategy:
   type: RollingUpdate
+  rollingUpdate:
+    maxSurge: 1
+    maxUnavailable: 0
 # -- Logging level: debug, info, warning, error or fatal
 logLevel: info
 # -- Enable development mode