From: Cédric Ollivier <cedric.ollivier@orange.com>
Date: Sat, 5 Jan 2019 12:56:30 +0000 (+0100)
Subject: Allow conditional docker build args
X-Git-Url: https://gerrit.opnfv.org/gerrit/gitweb?p=releng.git;a=commitdiff_plain;h=ab4b5a257813b172aebd409f68c6483d6383bb99

Allow conditional docker build args

It allows managing xtesting and functest jjbs via the same role.
functest-component doesn't take BRANCH into account.

Change-Id: If487e8350bca6b37d585a4ed16214e77df6170ae
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
---

diff --git a/jjb/functest/xtesting.yaml b/jjb/functest/xtesting.yaml
index 8e2b1e82c..267514a24 100644
--- a/jjb/functest/xtesting.yaml
+++ b/jjb/functest/xtesting.yaml
@@ -73,8 +73,13 @@
           else
             image={repo}:{port}/{container}:{tag}
           fi
+          if [ "{ref_arg}" = "None" ]; then
+            build_arg=""
+          else
+            build_arg="--build-arg {ref_arg}={ref}"
+          fi
           cd {path}
-          sudo docker build --build-arg {ref_arg}={ref} \
+          sudo docker build $build_arg \
             --pull=false --no-cache --force-rm=true \
             -t $image .