adding strategy to deployment spec

evilgn0me · Mar 19, 2024 · cb4d7c3 · vsimon · Apr 1, 2024 · cb4d7c3
1 parent 44ff5e8
commit cb4d7c3
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 1 deletion.
diff --git a/charts/simple-ssh-bastion/Chart.yaml b/charts/simple-ssh-bastion/Chart.yaml
@@ -15,7 +15,7 @@ type: application
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version.
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: 0.1.7
+version: 0.1.8
 
 # This is the version number of the application being deployed. This version number should be
 # incremented each time you make changes to the application. Versions are not expected to

diff --git a/charts/simple-ssh-bastion/templates/deployment.yaml b/charts/simple-ssh-bastion/templates/deployment.yaml
@@ -6,6 +6,12 @@ metadata:
     {{- include "simple-ssh-bastion.labels" . | nindent 4 }}
 spec:
   replicas: {{ .Values.replicaCount }}
+  {{- if .Values.strategy }}
+  strategy:
+    {{- with .Values.strategy }}
+      {{- toYaml . | nindent 2 }}
+    {{- end }}
+  {{- end }}
   selector:
     matchLabels:
       {{- include "simple-ssh-bastion.selectorLabels" . | nindent 6 }}

diff --git a/charts/simple-ssh-bastion/values.yaml b/charts/simple-ssh-bastion/values.yaml
@@ -38,6 +38,12 @@ persistence:
   accessMode: ReadWriteOnce
   size: 1Gi
 
+# deployment strategy, since by default we are creating persistance this should be set to 100 max unavailable.
+strategy:
+  rollingUpdate:
+    maxSurge: 100%
+    maxUnavailable: 100%
+
 imagePullSecrets: []
 nameOverride: ""
 fullnameOverride: ""