pantsbuild / pants / 6544
0%
main: 93%

LAST BUILD BRANCH: csb/pex-2.92.0
DEFAULT BRANCH: main
Ran
Jobs 10
Files 381
Run time 8min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
6544

push

travis-ci

jsirois 
Ensure pipeline failuires fail CI.

We use pipelines to collect filtered lists of targets for
several CI steps.  Previously these could fail when BUILD
files had sytax or other errors, but the CI run would proceed
with no targets.  This change turns on pipefail and lifts up
target collection steps so they are AND'ed with the steps
that use them such that pipefail actually triggers an overall
failure in those steps.

Testing Done:
Manually tested by introducing 2 BUILD file errors:
```diff
$ git diff -U1 pants-plugins/
diff --git a/pants-plugins/tests/python/internal_backend_test/BUILD b/pants-plugins/tests/python/internal_backend_test/BUILD
index b9577ff..7b0e067 100644
--- a/pants-plugins/tests/python/internal_backend_test/BUILD
+++ b/pants-plugins/tests/python/internal_backend_test/BUILD
@@ -3,3 +3,3 @@

-target(
+target2(
   name='all',

$ git diff -U1 -- tests/python/
diff --git a/tests/python/pants_test/BUILD b/tests/python/pants_test/BUILD
index 3a3cc05..a710ed0 100644
--- a/tests/python/pants_test/BUILD
+++ b/tests/python/pants_test/BUILD
@@ -3,2 +3,4 @@

+GARBAGE
+
 python_library(
```

Then tested each pipeline-using CI step.

Internal plugins:
```console
$ ./build-support/bin/ci.sh -fkmsrjpnca
...
Exception message: name 'target2' is not defined
 while executing BUILD file FilesystemBuildFile(/home/jsirois/dev/3rdparty/jsirois-pants/pants-plugins/tests/python/internal_backend_test/BUILD)
 Loading addresses from 'pants-plugins/tests/python/internal_backend_test' failed.
Invalid BUILD files for [pants-plugins/tests/python::]

INFO] Detected git repository at /home/jsirois/dev/3rdparty/jsirois-pants on branch jsirois/issues/2053

Internal backend python test failure
```

Python unit tests:
```console
$ ./build-support/bin/ci.sh -fkmsrjlnca
...
Exception message: name 'GARBAGE' is not defined
 while executing BUILD file FilesystemBuildFile(/home/jsirois/dev/3rdparty/jsirois-pants/tests/python/pants_test/BUILD)
 Loading addresses from 'tests/python/pants_test' failed.
Invalid BUILD files for [tests/python::]

INFO] Detected git repository at /home/jsirois/dev/3rdparty/jsirois-pants on branch jsirois/issues/2053

Core python test failure
```

Python integration tests:
```console
$ ./build-support/bin/ci.sh -fkmsrjlpna
...
Exception message: name 'GARBAGE' is not defined
 while executing BUILD file FilesystemBuildFile(/home/jsirois/dev/3rdparty/jsirois-pants/tests/python/pants_test/BUILD)
 Loading addresses from 'tests/python/pants_test' failed.
Invalid BUILD files for [tests/python::]

INFO] Detected git repository at /home/jsirois/dev/3rdparty/jsirois-pants on branch jsirois/issues/2053

Pants Integration test failure
```

CI went green here:
  https://travis-ci.org/pantsbuild/pants/builds/77969386

Bugs closed: 2053, 2096

Reviewed at https://rbcommons.com/s/twitter/r/2731/

16978 of 22690 relevant lines covered (74.83%)

1.41 hits per line

Jobs
ID Job ID Ran Files Coverage
1 6544.1 (CI_FLAGS="-cjlpn 'Various pants self checks'") 0
Travis Job 6544.1
2 6544.2 (CI_FLAGS="-fkmsrcn -u 0/2 'Unit tests for pants and pants-plugins - shard 1'") 0
70.89
 Travis Job 6544.2
3 6544.3 (CI_FLAGS="-fkmsrcn -u 1/2 'Unit tests for pants and pants-plugins - shard 2'") 0
70.1
 Travis Job 6544.3
4 6544.4 (CI_FLAGS="-fkmsrcjlp 'Python contrib tests'") 0
Travis Job 6544.4
5 6544.5 (CI_FLAGS="-fkmsrjlpn -i 0/6 'Python integration tests for pants - shard 1'") 0
Travis Job 6544.5
6 6544.6 (CI_FLAGS="-fkmsrjlpn -i 1/6 'Python integration tests for pants - shard 2'") 0
Travis Job 6544.6
7 6544.7 (CI_FLAGS="-fkmsrjlpn -i 2/6 'Python integration tests for pants - shard 3'") 0
Travis Job 6544.7
8 6544.8 (CI_FLAGS="-fkmsrjlpn -i 3/6 'Python integration tests for pants - shard 4'") 0
Travis Job 6544.8
9 6544.9 (CI_FLAGS="-fkmsrjlpn -i 4/6 'Python integration tests for pants - shard 5'") 0
Travis Job 6544.9
10 6544.10 (CI_FLAGS="-fkmsrjlpn -i 5/6 'Python integration tests for pants - shard 6'") 0
Travis Job 6544.10
Source Files on build 6544
Detailed source file information is not available for this build.