From ef286cd57fa08c73779ba7a76aac6fed37e62278 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Thorsten=20Ro=C3=9Fner?= <thorsten.rossner.extern@zendis.de>
Date: Tue, 12 Nov 2024 08:08:54 +0100
Subject: [PATCH] fix(jitsi-web): Restart on deployment to ensure changes are
 picked up.

---
 charts/opendesk-jitsi/templates/configmap.yaml |  3 ++-
 charts/opendesk-jitsi/templates/role.yaml      | 16 ++++++++++++----
 charts/opendesk-jitsi/values.yaml              |  2 +-
 3 files changed, 15 insertions(+), 6 deletions(-)

diff --git a/charts/opendesk-jitsi/templates/configmap.yaml b/charts/opendesk-jitsi/templates/configmap.yaml
index b259db1..3e6756a 100644
--- a/charts/opendesk-jitsi/templates/configmap.yaml
+++ b/charts/opendesk-jitsi/templates/configmap.yaml
@@ -93,6 +93,7 @@ data:
     kubectl patch svc jitsi-jvb --type json --patch \
       "[{ \"op\": \"replace\", \"path\": \"/spec/ports/0/targetPort\", \"value\": ${port} }]"
     {{- end }}
-    kubectl rollout restart deployment jitsi-jvb
+    # As the jitsi-web subchart does not realize when files in the folder /files/web are changed, we restart jitsi-web forcefully.
+    kubectl rollout restart deployment jitsi-web
 {{- end }}
 ...
diff --git a/charts/opendesk-jitsi/templates/role.yaml b/charts/opendesk-jitsi/templates/role.yaml
index 47a987d..075abdc 100644
--- a/charts/opendesk-jitsi/templates/role.yaml
+++ b/charts/opendesk-jitsi/templates/role.yaml
@@ -11,7 +11,7 @@ metadata:
   labels:
     {{- include "common.labels.standard" . | nindent 4 }}
 rules:
-  # Get IP/Port from service.
+  # Get IP/Port from service
   # Update target port for nodePort
   - apiGroups: [""]
     resources:
@@ -22,7 +22,7 @@ rules:
       - "get"
       - "update"
       - "patch"
-  # Update configmap for env vars.
+  # Update configmap for env vars
   - apiGroups: [""]
     resources:
       - "configmaps"
@@ -32,7 +32,7 @@ rules:
       - "get"
       - "update"
       - "patch"
-  # Restart JVB after patch.
+  # Get resources for jitsi-jvb
   - apiGroups: ["apps"]
     resources:
       - "deployments"
@@ -40,7 +40,15 @@ rules:
       - "jitsi-jvb"
     verbs:
       - "get"
-      - "update"
+      - "patch"
+  # Restart jitsi-web to pickup changes even when templated in one of the configMap files
+  - apiGroups: ["apps"]
+    resources:
+      - "deployments"
+    resourceNames:
+      - "jitsi-web"
+    verbs:
+      - "get"
       - "patch"
 ...
 {{- end }}
diff --git a/charts/opendesk-jitsi/values.yaml b/charts/opendesk-jitsi/values.yaml
index 273f850..ba7d4b9 100644
--- a/charts/opendesk-jitsi/values.yaml
+++ b/charts/opendesk-jitsi/values.yaml
@@ -208,7 +208,7 @@ patchJVB:
 imagePullSecrets: []
 
 jitsi:
-  publicURL: "https://jitsi.my-deployment.de"
+  publicURL: "https://jitsi.mydomain.tld"
   enableAuth: true
   enableGuests: true
 
-- 
GitLab