Hello Lars Volker, Michael Brown, Nithya Janarthanan, Philip Zeyliger, Joe McDonnell, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11257 to look at the new patch set (#9). Change subject: IMPALA-7399: Emit a junit xml report when trapping errors ...................................................................... IMPALA-7399: Emit a junit xml report when trapping errors This patch will cause a junitxml file to be emitted in the case of errors in build scripts. Instead of simply echoing a message to the console, we set up a trap function that also writes out the the junit xml report that can be consumed by jenkins.impala.io. Main things to pay attention to: - New file that gets sourced by all bash scripts when trapping within bash scripts: https://gerrit.cloudera.org/c/11257/1/bin/report_build_error.sh - Installation of the python lib into impala-python venv for use from within python files: https://gerrit.cloudera.org/c/11257/1/bin/impala-python-common.sh - Change to the generate_junitxml.py file itself, for ease of https://gerrit.cloudera.org/c/11257/1/lib/python/impala_py_lib/jenkins/generate_junitxml.py Most of the other changes are to source the new report_build_error.sh script to set up the trap function. Change-Id: Idd62045bb43357abc2b89a78afff499149d3c3fc --- M bin/clean-cmake.sh M bin/clean.sh M bin/create-test-configuration.sh M bin/create_testdata.sh M bin/distcc/distcc_server_setup.sh M bin/impala-python-common.sh M bin/jenkins/all-tests.sh M bin/jenkins/build-all-flag-combinations.sh M bin/jenkins/build-only.sh M bin/make_impala.sh A bin/report_build_error.sh M bin/run-all-tests.sh M bin/run-backend-tests.sh M bin/start-catalogd.sh M bin/start-impalad.sh M bin/start-statestored.sh M buildall.sh M infra/python/bootstrap_virtualenv.py M lib/python/impala_py_lib/jenkins/generate_junitxml.py M shell/make_shell_tarball.sh M testdata/bin/check-schema-diff.sh M testdata/bin/compute-table-stats.sh M testdata/bin/copy-data-sources.sh M testdata/bin/copy-udfs-udas.sh M testdata/bin/create-load-data.sh M testdata/bin/create-table-many-blocks.sh M testdata/bin/generate-load-nested.sh M testdata/bin/kill-all.sh M testdata/bin/kill-hbase.sh M testdata/bin/kill-hive-server.sh M testdata/bin/kill-java-service.sh M testdata/bin/kill-sentry-service.sh M testdata/bin/load-hive-builtins.sh M testdata/bin/load-metastore-snapshot.sh M testdata/bin/load-test-warehouse-snapshot.sh M testdata/bin/run-all.sh M testdata/bin/run-hbase.sh M testdata/bin/run-hive-server.sh M testdata/bin/run-mini-dfs.sh M testdata/bin/run-sentry-service.sh M testdata/bin/setup-hdfs-env.sh M testdata/cluster/admin M testdata/datasets/tpcds/preload M testdata/datasets/tpch/preload M tests/run-custom-cluster-tests.sh M tests/run-process-failure-tests.sh 46 files changed, 159 insertions(+), 72 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/57/11257/9 -- To view, visit http://gerrit.cloudera.org:8080/11257 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Idd62045bb43357abc2b89a78afff499149d3c3fc Gerrit-Change-Number: 11257 Gerrit-PatchSet: 9 Gerrit-Owner: David Knupp <dkn...@cloudera.com> Gerrit-Reviewer: David Knupp <dkn...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Lars Volker <l...@cloudera.com> Gerrit-Reviewer: Michael Brown <mi...@cloudera.com> Gerrit-Reviewer: Nithya Janarthanan <njanartha...@cloudera.com> Gerrit-Reviewer: Philip Zeyliger <phi...@cloudera.com>