aboutsummaryrefslogtreecommitdiffstats
path: root/jjb/global-macros.yaml
diff options
context:
space:
mode:
authorAnil Belur <abelur@linuxfoundation.org>2017-08-01 14:31:17 +1000
committerAnil Belur <abelur@linuxfoundation.org>2017-08-01 14:31:17 +1000
commita1c66436bdfc4778b066e65ddaa1a228181becec (patch)
treea48b92f74b45e229c19d078cad2e7c2b2475ddba /jjb/global-macros.yaml
parent8f551bf3eb8a3af4a4754b2966451745fb100bde (diff)
Improve packer verify and build jobs
- AGL uses multiple user credentials some of which are missing. JJB does not support setting "Ignore missing credentials" through JJB, therefore create a separate wrapper to fix the issue. - Improve the packer validation builder to exclude non-packer templates. Change-Id: I16524cbee14add2028d8c9e30b0f6c9315928031 Signed-off-by: Anil Belur <abelur@linuxfoundation.org>
Diffstat (limited to 'jjb/global-macros.yaml')
-rw-r--r--jjb/global-macros.yaml6
1 files changed, 4 insertions, 2 deletions
diff --git a/jjb/global-macros.yaml b/jjb/global-macros.yaml
index c73bd861..19a794b0 100644
--- a/jjb/global-macros.yaml
+++ b/jjb/global-macros.yaml
@@ -456,17 +456,19 @@
templates="../packer/templates/*"
provision="../packer/provision/*.sh"
for v in $varfiles; do
- [[ "${v##*/}" =~ ^(cloud-env.*|README.*)$ ]] && continue
+ [[ ${v##*/} =~ ^(cloud-env.*|lava-env.*|README.md)$ ]] && continue
for t in $templates; do
+ [[ ${t##*/} =~ ^(cloud-env.*|lava-env.*|README.md)$ ]] && continue
export PACKER_LOG="yes" && \
export PACKER_LOG_PATH="packer-validate-${v##*/}-${t##*/}.log" && \
packer.io validate -var-file=$CLOUDENV \
-var-file=$v $t
if [ $? -ne 0 ]; then
- break
+ exit 1
fi
done
done
+ # todo: migrate to linting with shellcheck
for p in $provision; do
/bin/bash -n $p > provision-validate-${p##*/}.log 2>&1
done