From ba8d3297e7aa890448cb32e6fb781c11045556d1 Mon Sep 17 00:00:00 2001
From: Marcin Antas <antas.marcin@gmail.com>
Date: Mon, 15 Apr 2024 12:11:00 +0200
Subject: [PATCH] Set updateStrategy rolling update max unavailable to 100%

---
 .cicd/test.sh        | 3 ++-
 weaviate/values.yaml | 5 +++++
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/.cicd/test.sh b/.cicd/test.sh
index f02565c..9768879 100755
--- a/.cicd/test.sh
+++ b/.cicd/test.sh
@@ -221,7 +221,8 @@ function check_creates_template() {
   check_string_existence "--set containerSecurityContext.allowPrivilegeEscalation=false " "allowPrivilegeEscalation: false"
   check_string_existence "" "imagePullPolicy: IfNotPresent"
   check_setting_has_value "--set image.pullSecrets[0]=weaviate-image-pull-secret" "imagePullSecrets" "name: weaviate-image-pull-secret"
-  check_setting_has_value "--set updateStrategy.type=OnDelete" "updateStrategy" "type: OnDelete"
+  check_string_existence "--set updateStrategy.type=OnDelete" "type: OnDelete"
+  check_setting_has_value "--set updateStrategy.rollingUpdate.maxUnavailable=10%" "rollingUpdate" "maxUnavailable: 10%"
   check_string_existence "--set priorityClassName=critical" "priorityClassName: critical"
   check_string_existence "--set globalPriorityClassName=high" "priorityClassName: high"
 
diff --git a/weaviate/values.yaml b/weaviate/values.yaml
index 4a0eb54..0281eb8 100644
--- a/weaviate/values.yaml
+++ b/weaviate/values.yaml
@@ -69,6 +69,11 @@ replicas: 1
 podManagementPolicy: Parallel
 updateStrategy:
   type: RollingUpdate
+  # This setting is only available in K8s v1.24 and higher.
+  # Setting maxUnavailable to 100% results in removing all of the pods
+  # and re-creating them in parallel all at once.
+  # rollingUpdate:
+  #   maxUnavailable: 100%
 resources: {}
   # requests:
   #   cpu: '500m'