Enable federator in wire-server to ensure it's image is bundled (SER-…

…182) (#573) * Enable federator in wire-server to ensure it's image is bundled (SER-182) Federation is disabled by default. To include it's image into `list-helm-containers` results, it needs to be enabled. This approach is fragile: When the files of the Helm charts change, it may fail and would have to be adjusted. * Update offline/ci.sh --------- Co-authored-by: Julia Longtin <[email protected]>
wireapp · Jan 31, 2024 · eb2ffa5 · eb2ffa5
1 parent 2f70903
commit eb2ffa5
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/offline/ci.sh b/offline/ci.sh
@@ -133,12 +133,10 @@ done
 ####### DIRTY HACKS GO HERE #######
 ###################################
 
-# old hack? missin an undo step?
-#sed -i -Ee 's/useSharedFederatorSecret: false/useSharedFederatorSecret: true/' "$(pwd)"/charts/wire-server/charts/federator/values.yaml
-
 # Patch wire-server values.yaml to include federator
 # This is needed to bundle it's image.
 sed -i -Ee 's/federation: false/federation: true/' "$(pwd)"/values/wire-server/prod-values.example.yaml
+sed -i -Ee 's/useSharedFederatorSecret: false/useSharedFederatorSecret: true/' "$(pwd)"/charts/wire-server/charts/federator/values.yaml
 
 # Get and dump required containers from Helm charts. Omit integration test
 # containers (e.g. `quay.io_wire_galley-integration_4.22.0`.)
@@ -147,6 +145,7 @@ for chartPath in "$(pwd)"/charts/*; do
 done | list-helm-containers | grep -v "\-integration:" | create-container-dump containers-helm
 
 # Undo changes on wire-server values.yaml
+sed -i -Ee 's/useSharedFederatorSecret: true/useSharedFederatorSecret: false/' "$(pwd)"/charts/wire-server/charts/federator/values.yaml
 sed -i -Ee 's/federation: true/federation: false/' "$(pwd)"/values/wire-server/prod-values.example.yaml
 
 patch-ingress-controller-images "$(pwd)"