From 9616172a1d8fb6cf15b46d0719d5ffde0b8fa6d6 Mon Sep 17 00:00:00 2001
From: Arie Peterson <arie@greenhost.nl>
Date: Thu, 2 Jun 2022 16:49:46 +0200
Subject: [PATCH 1/2] Upgrade zulip chart to 0.3.0

---
 flux2/apps/zulip/release.yaml                |  8 ++++++--
 flux2/apps/zulip/zulip-values-configmap.yaml | 13 ++++++++++---
 2 files changed, 16 insertions(+), 5 deletions(-)

diff --git a/flux2/apps/zulip/release.yaml b/flux2/apps/zulip/release.yaml
index 06ac1623..b1c9f149 100644
--- a/flux2/apps/zulip/release.yaml
+++ b/flux2/apps/zulip/release.yaml
@@ -9,7 +9,7 @@ spec:
   chart:
     spec:
       chart: zulip
-      version: 0.2.1
+      version: 0.3.0
       sourceRef:
         kind: HelmRepository
         name: zulip-helm-chart
@@ -19,7 +19,11 @@ spec:
     timeout: 30m
   upgrade:
     remediation:
-      retries: 2
+      retries: 1
+      # Uninstall the old version of zulip when upgrading fails.
+      # We needed this at least for the upgrade of the postgres dependency to
+      # chart version 11.
+      strategy: uninstall
   valuesFrom:
     - kind: ConfigMap
       name: stackspin-zulip-values
diff --git a/flux2/apps/zulip/zulip-values-configmap.yaml b/flux2/apps/zulip/zulip-values-configmap.yaml
index fe2c404c..b7be8a32 100644
--- a/flux2/apps/zulip/zulip-values-configmap.yaml
+++ b/flux2/apps/zulip/zulip-values-configmap.yaml
@@ -107,9 +107,14 @@ data:
     postgresql:
       image:
         tag: 14
-      persistence:
-        existingClaim: zulip-postgres
-      postgresqlPassword: "${postgresql_password}"
+      auth:
+        # Note: the old version (< 0.3.0) of this chart always used the
+        # postgresql admin password to log in. since 0.3.0 it uses a user
+        # password, but because we can't update the passwords in currently
+        # running postgresql instances through helm, we use the same password
+        # for both.
+        postgresqlPassword: "${postgresql_password}"
+        password: "${postgresql_password}"
       resources:
         limits:
           memory: 512Mi
@@ -117,6 +122,8 @@ data:
           cpu: 10m
           memory: 256Mi
       primary:
+        persistence:
+          existingClaim: zulip-postgres
         podAnnotations:
           backup.velero.io/backup-volumes: "data"
       commonLabels:
-- 
GitLab


From 02552cf4ead529761426b01d07c6482a2962b0f0 Mon Sep 17 00:00:00 2001
From: Arie Peterson <arie@greenhost.nl>
Date: Tue, 7 Jun 2022 16:54:22 +0200
Subject: [PATCH 2/2] Retry failed flux install of zulip

---
 flux2/apps/zulip/release.yaml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/flux2/apps/zulip/release.yaml b/flux2/apps/zulip/release.yaml
index b1c9f149..2fce805b 100644
--- a/flux2/apps/zulip/release.yaml
+++ b/flux2/apps/zulip/release.yaml
@@ -17,6 +17,8 @@ spec:
   interval: 1h
   install:
     timeout: 30m
+    remediation:
+      retries: 1
   upgrade:
     remediation:
       retries: 1
-- 
GitLab