fix label asignment

functions should assign values to variables instead of try to return the
output by echoing them. We can still use the echo for debugging
purposes, but the variable should be assigned by the function and used
in the other. Note that variables are not locally scoped.

chaos-workers/chaos-experiments/scripts/utils.sh

@@ -18,20 +18,22 @@ function getBrokerLabels() {
   if [ "${CHAOS_SETUP}" == "cloud" ]; then
     # For backwards compatability the brokers kept the gateway labels, for a statefulset the labels are not modifiable
     # To still be able to distinguish the standalone gateway with the broker, the gateway got a new label.
-    echo "-l app.kubernetes.io/app=zeebe -l app.kubernetes.io/component=gateway"
+    labels="app.kubernetes.io/app=zeebe app.kubernetes.io/component=gateway"
   else # helm
-    echo "-l app.kubernetes.io/component=zeebe-broker"
+    labels="app.kubernetes.io/component=zeebe-broker"
   fi
+  echo "$labels"
 }
 
 function getGatewayLabels() {
   if [ "${CHAOS_SETUP}" == "cloud" ]; then
     # For backwards compatability the brokers kept the gateway labels, for a statefulset the labels are not modifiable
     # To still be able to distinguish the standalone gateway with the broker, the gateway got a new label.
-    echo "-l app.kubernetes.io/app=zeebe -l app.kubernetes.io/component=standalone-gateway"
+    labels="app.kubernetes.io/app=zeebe app.kubernetes.io/component=standalone-gateway"
   else # helm
-    echo "-l app.kubernetes.io/component=zeebe-gateway"
+    labels="app.kubernetes.io/component=zeebe-gateway"
   fi
+  echo "$labels"
 }
 
 function runOnAllBrokers()