diff options
| author |   Anil Belur <abelur@linuxfoundation.org> | 2017-08-01 14:31:17 +1000 |
|---|---|---|
| committer | Anil Belur <abelur@linuxfoundation.org> | 2017-08-01 14:31:17 +1000 |
| commit | a1c66436bdfc4778b066e65ddaa1a228181becec (patch) | |
| tree | a48b92f74b45e229c19d078cad2e7c2b2475ddba /jjb | |
| parent | 8f551bf3eb8a3af4a4754b2966451745fb100bde (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')
| -rw-r--r-- | jjb/ci-management/ci-management.yaml | 28 | ||||
| -rw-r--r-- | jjb/global-macros.yaml | 6 |
2 files changed, 26 insertions, 8 deletions
diff --git a/jjb/ci-management/ci-management.yaml b/jjb/ci-management/ci-management.yaml index 07811ea5..5407073f 100644 --- a/jjb/ci-management/ci-management.yaml +++ b/jjb/ci-management/ci-management.yaml @@ -113,8 +113,17 @@ choosing-strategy: 'gerrit' wrappers: - - agl-infra-wrappers: - build-timeout: '{build-timeout}' + - timeout: + type: absolute + timeout: '{build-timeout}' + timeout-var: 'BUILD_TIMEOUT' + fail: true + - ssh-agent-credentials: + users: + - 'd28b1f3b-3603-49bc-b837-bb0a2e85d4b8' + - ansicolor + - openstack: + single-use: false triggers: - gerrit-trigger-patch-submitted: @@ -153,22 +162,29 @@ choosing-strategy: 'gerrit' wrappers: + - timeout: + type: absolute + timeout: '{build-timeout}' + timeout-var: 'BUILD_TIMEOUT' + fail: true - ssh-agent-credentials: users: - - '{ssh-credentials}' - - agl-infra-wrappers: - build-timeout: '{build-timeout}' + - 'd28b1f3b-3603-49bc-b837-bb0a2e85d4b8' + - ansicolor + - openstack: + single-use: false triggers: - timed: '00 10 1 * *' - gerrit: + server-name: 'gerrit.automotivelinux.org' trigger-on: - change-merged-event - comment-added-contains-event: comment-contains-value: 'remerge' projects: - project-compare-type: 'ANT' - project-pattern: 'ci-management' + project-pattern: '{name}' branches: - branch-compare-type: 'ANT' branch-pattern: '**/{branch}' 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 |