Merge pull request #435 from element-hq/bbz/missing-startupProbes

Add missing startupProbes

Author: benbz

charts/matrix-stack/templates/postgres/statefulset.yaml

@@ -65,6 +65,9 @@ spec:
           preStop:
             exec:
               command: ["pg_ctl", "stop", "-D", "/var/lib/postgres/data", "-w", "-t", "55", "-m", "fast"]
+        startupProbe:
+          exec:
+            command: ["psql", "-w", "-U", "postgres", "-d", "postgres", "-c", "SELECT 1"]
         readinessProbe:
           exec:
             command: ["psql", "-w", "-U", "postgres", "-d", "postgres", "-c", "SELECT 1"]
@@ -137,6 +140,9 @@ spec:
           preStop:
             exec:
               command: ["pg_ctl", "stop", "-D", "/var/lib/postgres/data", "-w", "-t", "55", "-m", "fast"]
+        startupProbe:
+          exec:
+            command: ["psql", "-w", "-U", "postgres", "-d", "postgres", "-c", "SELECT 1"]
         readinessProbe:
           exec:
             command: ["psql", "-w", "-U", "postgres", "-d", "postgres", "-c", "SELECT 1"]

charts/matrix-stack/templates/synapse/redis_deployment.yaml

@@ -66,6 +66,9 @@ spec:
         - containerPort: 6379
           name: redis
           protocol: TCP
+        startupProbe:
+          tcpSocket:
+            port: redis
         livenessProbe:
           tcpSocket:
             port: redis

newsfragments/435.changed.1.md

@@ -0,0 +1 @@
+Ensure Synapse's Redis has a startupProbe.

newsfragments/435.changed.md

@@ -0,0 +1 @@
+Ensure all Postgres containers have a startupProbe.