openstack/openstack-ansible
Code Issues Proposed changes

Merge "Only build ARA report when test fails"

Browse Source
This commit is contained in:
Jenkins 2017-10-14 10:49:18 +00:00 committed by Gerrit Code Review
commit 1b1dc14297
1 changed files with 16 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
scripts/scripts-library.sh
View File

@ -123,6 +123,11 @@ function exit_fail {
}
function gate_job_exit_tasks {
# This environment variable captures the exit code
# which was present when the trap was initiated.
# This would be the success/failure of the test.
export TEST_EXIT_CODE=$?
# If this is a gate node from OpenStack-Infra Store all logs into the
# execution directory after gate run.
if [ "$GATE_EXIT_LOG_COPY" == true ]; then
@ -137,7 +142,17 @@ function gate_job_exit_tasks {
# Generate the ARA report if enabled
if [ "$GATE_EXIT_RUN_ARA" == true ]; then
/opt/ansible-runtime/bin/ara generate html "${GATE_LOG_DIR}/ara" || true
# In order to reduce the quantity of unnecessary log content
# being kept in OpenStack-Infra we only generate the ARA report
# when the test result is a failure.
if [[ "${TEST_EXIT_CODE}" != "0" ]]; then
echo "Generating ARA report due to non-zero exit code (${TEST_EXIT_CODE})."
/opt/ansible-runtime/bin/ara generate html "${GATE_LOG_DIR}/ara" || true
else
echo "Not generating ARA report due to test pass."
fi
# We still want the subunit report though, as that reflects
# success/failure in OpenStack Health
/opt/ansible-runtime/bin/ara generate subunit "${GATE_LOG_DIR}/ara/testrepository.subunit" || true
fi
# Compress the files gathered so that they do not take up too much space.