improve release notes/make brig's multiSFT optional (#4010)

wireapp · Apr 23, 2024 · ed8e838 · ed8e838
1 parent b94aea3
commit ed8e838
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 5 deletions.
diff --git a/changelog.d/0-release-notes/WPB-227 b/changelog.d/0-release-notes/WPB-227
@@ -1,12 +1,17 @@
-There is a new optional Boolean option, `multiSFT.enabled`, in `brig.yaml`,
-allowing calls between federated SFT servers. If provided, the field
-`is_federating` in the response of `/calls/config/v2` will reflect
-`multiSFT.enabled`'s value.
+There is a new optional Boolean in Brig's Helm chart, `config.multiSFT.enabled`,
+signalling whether calls between federated SFT servers are allowed.
+
+IMPORTANT: The value of this new option needs be set to the value of
+`multiSFT.enabled` in SFT's Helm chart. Otherwise federated SFT servers won't
+work.
+
+If provided, the field `is_federating` in the response of `/calls/config/v2`
+will reflect `multiSFT.enabled`'s value.
 
 Example:
 
 ```
-# [brig.yaml]
+# [brig/values.yaml]
 multiSFT:
   enabled: true
 ```

diff --git a/charts/brig/templates/configmap.yaml b/charts/brig/templates/configmap.yaml
@@ -57,7 +57,9 @@ data:
       host: gundeck
       port: 8080
 
+    {{- if .multiSFT }}
     multiSFT: {{ .multiSFT.enabled }}
+    {{- end }}
     {{- if .enableFederation }}
     # TODO remove this
     federator: