Merge branch 'feat/fix-nodeport-job' into 'main'

fix(sovereign-workplace-jitsi): Fix NodePort bootstrap patch command

See merge request souvap/tooling/charts/sovereign-workplace-jitsi!2

charts/sovereign-workplace-jitsi/templates/configmap.yaml

@@ -23,6 +23,8 @@ metadata:
   name: {{ include "common.names.fullname" . }}
 data:
   init.sh: |
+    #!/bin/sh
+    set -e
     {{- if (eq .Values.jitsi.jvb.service.type "LoadBalancer") }}
     counter=0
     maxRetries=30
@@ -44,8 +46,8 @@ data:
     port=$(kubectl get svc jitsi-jvb -o jsonpath='{.spec.ports[0].nodePort}');
     echo "Patching JVB to use port: ${port}"
     kubectl patch cm jitsi-jvb --type json --patch \
-      "[{ \"op\": \"add\", \"path\": \"/data/JVB_PORT\", \"value\": \"${port}\" }"
+      "[{ \"op\": \"add\", \"path\": \"/data/JVB_PORT\", \"value\": \"${port}\" }]"
     {{- end }}
     kubectl rollout restart deployment jitsi-jvb
-{{- end }}
+    {{- end }}
 ...

charts/sovereign-workplace-jitsi/templates/job.yaml

@@ -17,7 +17,7 @@ metadata:
 spec:
   template:
     spec:
-      restartPolicy: "Never"
+      restartPolicy: "OnFailure"
       {{- if or .Values.patchJVB.imagePullSecrets .Values.global.imagePullSecrets }}
       imagePullSecrets:
         {{- range .Values.global.imagePullSecrets }}