fix(demo-wiab): wpb-17321 allow pre_chart_process to be aware of values and fix demo build to pre_clean_values properly

Author: mohitrajain

offline/default-build/build.sh

@@ -38,7 +38,7 @@ cp -r "${ROOT_DIR}"/values "${OUTPUT_DIR}"/
 cp -r "${ROOT_DIR}"/dashboards "${OUTPUT_DIR}"/
 
 # all basic chart pre-processing tasks
-"${TASKS_DIR}"/pre_chart_process_0.sh "${OUTPUT_DIR}"
+"${TASKS_DIR}"/pre_chart_process_0.sh OUTPUT_DIR="${OUTPUT_DIR}"
 
 # all extra pre chart processing tasks for this profile should come here
 # pre_chart_process_1.sh 

offline/demo-build/build.sh

@@ -32,18 +32,18 @@ cp -r "${ROOT_DIR}"/values "${OUTPUT_DIR}"/
 cp -r "${ROOT_DIR}"/dashboards "${OUTPUT_DIR}"/
 
 # removing the values/$chart directories in values directory if not required
-"${TASKS_DIR}"/pre_clean_values_0.sh VALUES_DIR="${OUTPUT_DIR}/values" HELM_CHART_EXCLUDE_LIST="${HELM_CHART_EXCLUDE_LIST}"
+"${TASKS_DIR}"/pre_clean_values_0.sh VALUES_DIR="${OUTPUT_DIR}/values" HELM_CHART_EXCLUDE_LIST="${HELM_CHART_EXCLUDE_LIST}" VALUES_TYPE="demo"
 
 # all basic chart pre-processing tasks
-"${TASKS_DIR}"/pre_chart_process_0.sh "${OUTPUT_DIR}"
+"${TASKS_DIR}"/pre_chart_process_0.sh OUTPUT_DIR="${OUTPUT_DIR}" VALUES_TYPE="demo"
 
 # processing the charts
 # here we also filter the images post processing the helm charts
 # pass the image names to be filtered as arguments as regex #IMAGE_EXCLUDE_LIST='brig|galley'
 "${TASKS_DIR}"/process_charts.sh OUTPUT_DIR="${OUTPUT_DIR}" IMAGE_EXCLUDE_LIST="quay.io/wire/federator" VALUES_TYPE="demo"
 
 # all basic chart pre-processing tasks
-"${TASKS_DIR}"/post_chart_process_0.sh "${OUTPUT_DIR}"
+"${TASKS_DIR}"/post_chart_process_0.sh "${OUTPUT_DIR}" VALUES_TYPE="demo"
 
 # --------------------------
 # building admin host containers, has dependenct on the helm charts

offline/min-build/build.sh

@@ -40,7 +40,7 @@ cp -r "${ROOT_DIR}"/values "${OUTPUT_DIR}"/
 "${TASKS_DIR}"/pre_clean_values_0.sh VALUES_DIR="${OUTPUT_DIR}/values" HELM_CHART_EXCLUDE_LIST="${HELM_CHART_EXCLUDE_LIST}"
 
 # all basic chart pre-processing tasks
-"${TASKS_DIR}"/pre_chart_process_0.sh "${OUTPUT_DIR}"
+"${TASKS_DIR}"/pre_chart_process_0.sh OUTPUT_DIR="${OUTPUT_DIR}"
 
 # all extra pre chart processing tasks for this profile should come here
 # pre_chart_process_1.sh

offline/tasks/pre_chart_process_0.sh

@@ -1,18 +1,40 @@
 #!/usr/bin/env bash
-set -euo pipefail
+set -exuo pipefail
 
-if [[ ! $# -eq 1 ]]; then
-  echo "usage: $0 OUTPUT-DIR" >&2
+# Default output dir
+OUTPUT_DIR=""
+# Default values type will expect to use prod values
+VALUES_TYPE="prod"
+
+# Parse the arguments
+for arg in "$@"
+do
+  case $arg in
+    OUTPUT_DIR=*)
+      OUTPUT_DIR="${arg#*=}"
+      ;;
+    VALUES_TYPE=*)
+      VALUES_TYPE="${arg#*=}"
+      ;;
+    *)
+      echo "Unknown argument: $arg" >&2
+      exit 1
+      ;;
+  esac
+done
+
+
+# Check if OUTPUT_DIR is set
+if [[ -z "$OUTPUT_DIR" ]]; then
+  echo "usage: $0 OUTPUT_DIR=\"values-dir\" [VALUES_TYPE=\"prod|demo\"]" >&2
   exit 1
 fi
 
-OUTPUT_DIR="$1"
-
-echo "Running pre-chart process script 0 in dir $OUTPUT_DIR ..."
+echo "Running pre-chart process script 0 in dir $OUTPUT_DIR with values type $VALUES_TYPE"
 
 # Patch wire-server values.yaml to include federator
 # This is needed to bundle it's image.
-sed -i -Ee 's/federation: false/federation: true/' "${OUTPUT_DIR}"/values/wire-server/prod-values.example.yaml
+sed -i -Ee 's/federation: false/federation: true/' "${OUTPUT_DIR}/values/wire-server/${VALUES_TYPE}-values.example.yaml"
 sed -i -Ee 's/useSharedFederatorSecret: false/useSharedFederatorSecret: true/' "${OUTPUT_DIR}"/charts/wire-server/charts/federator/values.yaml
 
 # drop step-certificates/.../test-connection.yaml because it lacks an image tag