commit d47bfb2ba49c8633f790645f0f6b6b44ba1b446b
author Matteo Scandolo <matteo.scandolo@gmail.com> Mon May 24 09:40:05 2021 -0700
committer Matteo Scandolo <matteo.scandolo@gmail.com> Mon May 24 09:40:11 2021 -0700
tree 58b3bb55a5bc4e3487dce9b52b5b880f4662a91b
parent 5ef090e2595140f7481a8a3c4bb27c2e9fb76db1

[VOL-4127] Adding extraHelmFlag parameter to physical build jobs

Change-Id: I1b7edff8e6a1910285cd4498ddde85e9f42972f6

jjb/pipeline/voltha/master/physical-build.groovy

diff --git a/jjb/pipeline/voltha/master/physical-build.groovy b/jjb/pipeline/voltha/master/physical-build.groovy
index 1b66c03..6f6f9f9 100644
--- a/jjb/pipeline/voltha/master/physical-build.groovy
+++ b/jjb/pipeline/voltha/master/physical-build.groovy
@@ -93,30 +93,34 @@
             }
 
             // should the config file be suffixed with the workflow? see "deployment_config"
-            def extraHelmFlags = "-f $WORKSPACE/${configBaseDir}/${configKubernetesDir}/voltha/${configFileName}.yml --set global.log_level=${logLevel} "
+            def localHelmFlags = "-f $WORKSPACE/${configBaseDir}/${configKubernetesDir}/voltha/${configFileName}.yml --set global.log_level=${logLevel} "
 
             if (workFlow.toLowerCase() == "dt") {
-              extraHelmFlags += " --set radius.enabled=false "
+              localHelmFlags += " --set radius.enabled=false "
             }
             if (workFlow.toLowerCase() == "tt") {
-              extraHelmFlags += " --set radius.enabled=false --set global.incremental_evto_update=true "
+              localHelmFlags += " --set radius.enabled=false --set global.incremental_evto_update=true "
             }
 
             // NOTE temporary workaround expose ONOS node ports (pod-config needs to be updated to contain these values)
-            extraHelmFlags = extraHelmFlags + " --set onos-classic.onosSshPort=30115 " +
+            localHelmFlags = localHelmFlags + " --set onos-classic.onosSshPort=30115 " +
             "--set onos-classic.onosApiPort=30120 " +
             "--set onos-classic.onosOfPort=31653 " +
             "--set onos-classic.individualOpenFlowNodePorts=true "
 
+
             def bbsimReplica = 0
             if (installBBSim.toBoolean()) {
               bbsimReplica = 1
-              extraHelmFlags = extraHelmFlags + " --set onu=${onuNumber},pon=${ponNumber} "
+              localHelmFlags = localHelmFlags + " --set onu=${onuNumber},pon=${ponNumber} "
             }
 
+            // adding user specified helm flags at the end so they'll have priority over everything else
+            localHelmFlags = localHelmFlags += " ${extraHelmFlags}"
+
             volthaDeploy([
               workflow: workFlow.toLowerCase(),
-              extraHelmFlags: extraHelmFlags,
+              extraHelmFlags: localHelmFlags,
               localCharts: localCharts,
               kubeconfig: "$WORKSPACE/${configBaseDir}/${configKubernetesDir}/${configFileName}.conf",
               onosReplica: params.NumOfOnos,