commit ed1afdde06bd21c01f1148529630392c234a9605
author Matteo Scandolo <matteo.scandolo@gmail.com> Fri Apr 02 16:25:45 2021 -0700
committer Matteo Scandolo <matteo.scandolo@gmail.com> Fri Apr 02 16:25:45 2021 -0700
tree cf9b8add5a7fe79c82e5bcf78f47fea1fda91c04
parent 660e4af8935f46321397a95d7797955c9fa65f34

Always pass extraHelmFlag as last parameter in helm install commands

Change-Id: I2b251f954fd703095595a9dbcec25e5020a23a87

vars/volthaInfraDeploy.groovy

diff --git a/vars/volthaInfraDeploy.groovy b/vars/volthaInfraDeploy.groovy
index e7af5e2..4f828fd 100644
--- a/vars/volthaInfraDeploy.groovy
+++ b/vars/volthaInfraDeploy.groovy
@@ -48,8 +48,8 @@
     """
     // TODO support multiple replicas
     sh """
-    helm upgrade --install --create-namespace -n ${cfg.infraNamespace} voltha-infra ${volthaInfraChart} ${cfg.extraHelmFlags} \
+    helm upgrade --install --create-namespace -n ${cfg.infraNamespace} voltha-infra ${volthaInfraChart} \
           --set onos-classic.replicas=${cfg.onosReplica},onos-classic.atomix.replicas=${cfg.atomixReplica} \
-          -f $WORKSPACE/voltha-helm-charts/examples/${cfg.workflow}-values.yaml
+          -f $WORKSPACE/voltha-helm-charts/examples/${cfg.workflow}-values.yaml ${cfg.extraHelmFlags}
     """
 }

vars/volthaStackDeploy.groovy

diff --git a/vars/volthaStackDeploy.groovy b/vars/volthaStackDeploy.groovy
index 0b0d116..b39c84f 100644
--- a/vars/volthaStackDeploy.groovy
+++ b/vars/volthaStackDeploy.groovy
@@ -33,18 +33,20 @@
     println "Deploying VOLTHA Stack with the following parameters: ${cfg}."
 
     sh """
-    helm upgrade --install --create-namespace -n ${cfg.volthaNamespace} ${cfg.stackName} ${volthaStackChart} ${cfg.extraHelmFlags} \
+    helm upgrade --install --create-namespace -n ${cfg.volthaNamespace} ${cfg.stackName} ${volthaStackChart} \
           --set global.stack_name=${cfg.stackName} \
           --set global.voltha_infra_name=voltha-infra \
           --set global.voltha_infra_namespace=${cfg.infraNamespace} \
+          ${cfg.extraHelmFlags}
     """
 
     for(int i = 0;i<cfg.bbsimReplica;i++) {
       // TODO differentiate olt_id between different stacks
        sh """
-         helm upgrade --install --create-namespace -n ${cfg.volthaNamespace} bbsim${i} onf/bbsim ${cfg.extraHelmFlags} \
+         helm upgrade --install --create-namespace -n ${cfg.volthaNamespace} bbsim${i} onf/bbsim \
          --set olt_id="${cfg.stackId}${i}" \
-         -f $WORKSPACE/voltha-helm-charts/examples/${cfg.workflow}-values.yaml
+         -f $WORKSPACE/voltha-helm-charts/examples/${cfg.workflow}-values.yaml \
+         ${cfg.extraHelmFlags}
        """
     }