fix: don't set spec.updateStrategy.maxSurge of DaemonSet by default

postfinance · Oct 2, 2024 · 3facb9c · 3facb9c
1 parent cd56ef0
commit 3facb9c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/helm/kubenurse/templates/daemonset.yaml b/helm/kubenurse/templates/daemonset.yaml
@@ -13,7 +13,9 @@ spec:
       {{- include "kubenurse.selectorLabels" . | nindent 6 }}
   updateStrategy:
     rollingUpdate:
-      maxSurge: 0
+      {{- with .Values.daemonset.rollingUpdate.maxSurge }}
+      maxSurge: {{ . }}
+      {{- end }}
       maxUnavailable: {{ .Values.daemonset.rollingUpdate.maxUnavailable }}
     type: RollingUpdate
   template:

diff --git a/helm/kubenurse/values.yaml b/helm/kubenurse/values.yaml
@@ -22,6 +22,7 @@ daemonset:
   priorityClassName: ""
   rollingUpdate:
     maxUnavailable: 34%
+    # maxSurge:
 
 serviceMonitor:
   enabled: false