From 09447f11b490887c4e548e5a19b0aea6e1c4a9b4 Mon Sep 17 00:00:00 2001 From: danc094codetogether Date: Wed, 26 Nov 2025 13:01:50 -0600 Subject: [PATCH] feat(helm): add warning log when deprecated AV settings are detected Fixes: #192 - Added a startup warning when any Audio/Video related values (av.enabled, av.serverIP, av.stunServers, av.jitsiUrl) are present in Helm chart configurations. - Ensures deployments are notified that built in A/V support is no longer available. - No functional behavior changed, only logs a warning at container start. --- charts/collab/templates/deployment.yaml | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/charts/collab/templates/deployment.yaml b/charts/collab/templates/deployment.yaml index ff2fde7..a4e77d8 100644 --- a/charts/collab/templates/deployment.yaml +++ b/charts/collab/templates/deployment.yaml @@ -110,6 +110,15 @@ spec: {{- if or (eq .Values.codetogether.mode "direct") (eq .Values.codetogether.mode "locator-edge") }} - name: CT_AV_ENABLED value: {{ .Values.av.enabled | quote }} + + # ------------------------------ + # A/V DEPRECATION NOTICE + # ------------------------------ + {{- if or .Values.av.enabled .Values.av.serverIP .Values.av.stunServers.enabled .Values.av.jitsiUrl }} + - name: CT_AV_DEPRECATED_WARNING + value: "Audio/Video options detected in Helm values, but A/V is no longer supported and will be ignored." + {{- end }} + {{- if .Values.av.enabled }} - name: CT_AV_LAN_IP value: {{ .Values.av.serverIP | quote }}