diff options
author | Daniel Díaz <daniel.diaz@linaro.org> | 2016-10-07 18:46:07 -0500 |
---|---|---|
committer | Fathi Boudra <fathi.boudra@linaro.org> | 2016-10-11 06:30:58 +0000 |
commit | 3cf845944ebd1f12786b1c334e6f73af00385079 (patch) | |
tree | bf54e842303f88ad8694a4585dc4f0b9382b60cf /linaro-art-tip-build-ARTVixlTest.yaml | |
parent | 5990c324d36cd2617ac4492c97b6e4936eb91c51 (diff) |
Inject Gerrit variables before anything fails.
Building upon d11f7fb, it was noticed that a build step can
fail and Jenkins will not execute the rest of the build
steps, heading straight into the post-build steps.
Since during the build a fatal error may occur, the Gerrit
variables may not get defined at all before reaching the
email-ext plug-in.
This commit moves the recipient list to the beginning of
the job definition, so that it gets executed regardless of
the build result.
Change-Id: I8fe03cd5cbc7eebc176e151b71b890387bc26818
Signed-off-by: Daniel Díaz <daniel.diaz@linaro.org>
Diffstat (limited to 'linaro-art-tip-build-ARTVixlTest.yaml')
-rw-r--r-- | linaro-art-tip-build-ARTVixlTest.yaml | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/linaro-art-tip-build-ARTVixlTest.yaml b/linaro-art-tip-build-ARTVixlTest.yaml index 9e2c991fe8..21946e0793 100644 --- a/linaro-art-tip-build-ARTVixlTest.yaml +++ b/linaro-art-tip-build-ARTVixlTest.yaml @@ -71,6 +71,14 @@ variable: PRIVATE_KEY builders: - shell: | + CI_MAIL_RECIPIENTS="${GERRIT_CHANGE_OWNER_EMAIL},${GERRIT_EVENT_ACCOUNT_EMAIL}" + CI_MAIL_RECIPIENTS="${CI_MAIL_RECIPIENTS},vishal.bhoj@linaro.org" + CI_MAIL_RECIPIENTS="${CI_MAIL_RECIPIENTS},daniel.diaz@linaro.org" + CI_MAIL_RECIPIENTS="${CI_MAIL_RECIPIENTS},julien.duraj@linaro.org" + echo "CI_MAIL_RECIPIENTS=${CI_MAIL_RECIPIENTS}" > env_var_parameters + - inject: + properties-file: env_var_parameters + - shell: | #!/bin/bash -xe # Some Docker instances mount ${HOME}/.ssh/config from the host. @@ -94,14 +102,9 @@ cd ${ANDROID_BUILD_DIR} perl ${ART_BUILD_SCRIPTS_DIR}/jenkins/test_launcher.pl \ ${ART_BUILD_SCRIPTS_DIR}/tests/test_art_vixl.sh - - shell: | - echo "GERRIT_CHANGE_OWNER_EMAIL=${GERRIT_CHANGE_OWNER_EMAIL}" > env_var_parameters - echo "GERRIT_EVENT_ACCOUNT_EMAIL=${GERRIT_EVENT_ACCOUNT_EMAIL}" >> env_var_parameters - - inject: - properties-file: env_var_parameters publishers: - email-ext: - recipients: '${GERRIT_CHANGE_OWNER_EMAIL},${GERRIT_EVENT_ACCOUNT_EMAIL},vishal.bhoj@linaro.org,daniel.diaz@linaro.org,julien.duraj@linaro.org' + recipients: '${CI_MAIL_RECIPIENTS}' content-type: text body: | Build Triggered by: ${GERRIT_CHANGE_SUBJECT}: ${GERRIT_CHANGE_URL} |