Revert "Mark individual step as failure"
This reverts commit 272a2bf76f0e00c5066aa6a05354d13ce5968727.
This commit mark every build red when there is only test failures. It
reduces the information and make the downstream projects harder to analyze.
To double down, it does not work, the whole build is marked as broken
but we cannot locate the failing step:
http://ci.bazel.io/blue/organizations/jenkins/Global%2Fbazel-tests/detail/bazel-tests/180/pipeline
Change-Id: Ic2aa30360d5d7e7dada7198f3aa9cc073f1a1906
diff --git a/jenkins/lib/src/build/bazel/ci/BazelUtils.groovy b/jenkins/lib/src/build/bazel/ci/BazelUtils.groovy
index 86db30b..e10236f 100644
--- a/jenkins/lib/src/build/bazel/ci/BazelUtils.groovy
+++ b/jenkins/lib/src/build/bazel/ci/BazelUtils.groovy
@@ -127,7 +127,8 @@
def status = bazelCommand("test ${filteredTests.replaceAll("\n", " ")}", true)
if (status == 3) {
// Bazel returns 3 if there was a test failures but no breakage, that is unstable
- script.error("`bazel test` returned status 3 which indicates test failures")
+ script.currentBuild.result = "UNSTABLE"
+ script.echo("`bazel test` returned status 3 which indicates test failures")
} else if (status != 0) {
// TODO(dmarting): capturing the output mark the wrong step at failure, there is
// no good way to do so, it would probably better to have better output in the failing
