Skip to content

Commit

Permalink
chore(helm): add rolling update strategy
Browse files Browse the repository at this point in the history
  • Loading branch information
Sarthak-instill committed Dec 6, 2023
1 parent fb15ee3 commit 31702da
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 2 deletions.
5 changes: 4 additions & 1 deletion charts/vdp/templates/controller-vdp/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,10 @@ metadata:
spec:
strategy:
type: {{ .Values.updateStrategy.type }}
{{- if eq .Values.updateStrategy.type "Recreate" }}
rollingUpdate:
maxSurge: {{ .Values.updateStrategy.rollingUpdate.maxSurge }}
maxUnavailable: {{ .Values.updateStrategy.rollingUpdate.maxUnavailable }}
{{- else}}
rollingUpdate: null
{{- end }}
{{- if not .Values.controllerVDP.autoscaling.enabled }}
Expand Down
5 changes: 4 additions & 1 deletion charts/vdp/templates/pipeline-backend/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,10 @@ metadata:
spec:
strategy:
type: {{ .Values.updateStrategy.type }}
{{- if eq .Values.updateStrategy.type "Recreate" }}
rollingUpdate:
maxSurge: {{ .Values.updateStrategy.rollingUpdate.maxSurge }}
maxUnavailable: {{ .Values.updateStrategy.rollingUpdate.maxUnavailable }}
{{- else}}
rollingUpdate: null
{{- end }}
{{- if not .Values.pipelineBackend.autoscaling.enabled }}
Expand Down
3 changes: 3 additions & 0 deletions charts/vdp/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 31702da

Please sign in to comment.