switch to spreadServiceAcrossNodes setting

epoch8 · Aug 27, 2024 · 724821b · 724821b
1 parent 62febc8
commit 724821b
Show file tree

Hide file tree

Showing 6 changed files with 36 additions and 7 deletions.
diff --git a/charts/simple-app/CHANGELOG.md b/charts/simple-app/CHANGELOG.md
@@ -1,3 +1,7 @@
+# 0.13.0
+
+* Add `spreadServiceAcrossNodes` setting
+
 # 0.12.0, 0.12.1
 
 * Add `monitoringCoreos` setting for prometheus-operator managed metrics

diff --git a/charts/simple-app/Chart.yaml b/charts/simple-app/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.12.1"
+version: "0.13.0"
 
 # 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-app/templates/deployment.yaml b/charts/simple-app/templates/deployment.yaml
@@ -131,10 +131,13 @@ spec:
       tolerations:
         {{- toYaml . | nindent 8 }}
       {{- end }}
+      {{- if .Values.spreadServiceAcrossNodes }}
       topologySpreadConstraints:
       - labelSelector:
           matchLabels:
-            app.kubernetes.io/instance: {{ .Release.Name }}
-        maxSkew: {{ .Values.skew }}
+          {{- include "simple-app.selectorLabels" . | nindent 12 }}
+        maxSkew: 1
         topologyKey: "kubernetes.io/hostname"
-        whenUnsatisfiable: {{ .Values.skewPolicy }}
+        nodeTaintsPolicy: Honor
+        whenUnsatisfiable: ScheduleAnyway
+      {{- end }}
diff --git a/charts/simple-app/tests/simple_test.yaml b/charts/simple-app/tests/simple_test.yaml
@@ -308,3 +308,22 @@ tests:
       - equal:
           path: metadata.name
           value: simple
+
+  - it: Spread service across nodes
+    template: deployment.yaml
+    set:
+      spreadServiceAcrossNodes: true
+    asserts:
+      - isKind:
+          of: Deployment
+      - equal:
+          path: spec.template.spec.topologySpreadConstraints
+          value:
+            - maxSkew: 1
+              topologyKey: kubernetes.io/hostname
+              nodeTaintsPolicy: Honor
+              whenUnsatisfiable: ScheduleAnyway
+              labelSelector:
+                matchLabels:
+                  app.kubernetes.io/instance: simple
+                  app.kubernetes.io/name: simple-app
diff --git a/charts/simple-app/values.schema.json b/charts/simple-app/values.schema.json
@@ -196,6 +196,9 @@
         "affinity": {
             "type": "object"
         },
+        "spreadServiceAcrossNodes": {
+            "type": "boolean"
+        },
         "gke": {
             "type": "object",
             "properties": {

diff --git a/charts/simple-app/values.yaml b/charts/simple-app/values.yaml
@@ -4,9 +4,6 @@
 # This is a YAML-formatted file.
 # Declare variables to be passed into your templates.
 
-skew: 1
-skewPolicy: ScheduleAnyway
-
 nameOverride: ""
 fullnameOverride: ""
 
@@ -181,6 +178,9 @@ tolerations: []
 
 affinity: {}
 
+# If set, create topologySpreadConstraints to spread the pods across nodes
+spreadServiceAcrossNodes: false
+
 # Settings specific to GKE deployment
 gke:
   # If enabled and gateway.enabled is true, creates GCPBackendPolicy and