pantsbuild / pants / 6213
0%
main: 93%

LAST BUILD BRANCH: upgrade_runson_to_ami-0a3c71582153db678
DEFAULT BRANCH: main
Ran
Jobs 10
Files 351
Run time 7min
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
6213

push

travis-ci

jsirois 
Detect new package when doing ownership checks.

Both the main release and the -o ownership listing now check to see if a
package is new.  The release script skips the ownership check for new
packages now since it will register the package and the ownership
listing just notes the package is new.

Testing Done:
Used this diff to test with:
```diff
$ git diff -U1 contrib/
diff --git a/contrib/release_packages.sh b/contrib/release_packages.sh
index d958e1d..9ac5c5e 100644
--- a/contrib/release_packages.sh
+++ b/contrib/release_packages.sh
@@ -17,3 +17,3 @@
 PKG_SCROOGE=(
-  "pantsbuild.pants.contrib.scrooge"
+  "pantsbuild.pants.contrib.scrooge.NEW!"
   "//contrib/scrooge/src/python/pants/contrib/scrooge:plugin"
```

For the owners listing this gives:
```console
$ ./build-support/bin/release.sh -o
...
Owners of pantsbuild.pants.testinfra:
  Patrick.Lawson
  stuhood
  ity
  ericzundel
  benjyw
  john.sirois

The pantsbuild.pants.contrib.scrooge.NEW! package is new!  There are no owners yet.

Owners of pantsbuild.pants.contrib.buildgen:
  Patrick.Lawson
  john.sirois
  benjyw
  stuhood
  ity
  ericzundel
...
```

And then for the release itself I tested with this diff:
```diff
$ git diff
diff --git a/build-support/bin/release.sh b/build-support/bin/release.sh
index 63e4cf1..088d50e 100755
--- a/build-support/bin/release.sh
+++ b/build-support/bin/release.sh
@@ -475,8 +475,9 @@ elif [[ "${test_release}" == "true" ]]; then
 else
   banner "Releasing packages to PyPi." && \
   (
-    check_origin && check_clean_master && check_pgp && check_owners && \
-      dry_run_install && publish_packages && tag_release && \
+    check_owners && \
+#    check_origin && check_clean_master && check_pgp && check_owners && \
+#      dry_run_install && publish_packages && tag_release && \
         banner "Successfully released packages to PyPi."
   ) || die "Failed to release packages to PyPi."
 fi
```

And got:
```console
$ ./build-support/bin/release.sh

[== 00:00 Releasing packages to PyPi. ==]

[== 00:00 Checking package ownership for pypi user john.sirois of 7 packages ... ==]

[== 00:00 [1/7] checking that john.sirois owns pantsbuild.pants... ==]

[== 00:05 [2/7] checking that john.sirois owns pantsbuild.pants.backend.android... ==]

[== 00:16 [3/7] checking that john.sirois owns pantsbuild.pants.testinfra... ==]

[== 00:25 [4/7] checking that john.sirois owns pantsbuild.pants.contrib.scrooge.NEW!... ==]

The pantsbuild.pants.contrib.scrooge.NEW! package is new!  There are no owners yet.

[== 00:25 [5/7] checking that john.sirois owns pantsbuild.pants.contrib.buildgen... ==]

[== 00:35 [6/7] checking that john.sirois owns pantsbuild.pants.contrib.spindle... ==]

[== 00:44 [7/7] checking that john.sirois owns pantsbuild.pants.contrib.go... ==]

Your pypi account john.sirois needs to be added as an owner for the
following packages:
  pantsbuild.pants.contrib.go

Failed to release packages to PyPi.
```

That last bit of output is a reminder to Stu to add the current crop of OWNERs to go.

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

Bugs closed: 1990, 1996

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

15703 of 21270 relevant lines covered (73.83%)

1.39 hits per line

Jobs
ID Job ID Ran Files Coverage
1 6213.1 (CI_FLAGS="-cjlpn 'Various pants self checks'") 0
Travis Job 6213.1
2 6213.2 (CI_FLAGS="-fkmsrcn -u 0/2 'Unit tests for pants and pants-plugins - shard 1'") 0
69.0
 Travis Job 6213.2
3 6213.3 (CI_FLAGS="-fkmsrcn -u 1/2 'Unit tests for pants and pants-plugins - shard 2'") 0
70.07
 Travis Job 6213.3
4 6213.4 (CI_FLAGS="-fkmsrcjlp 'Python contrib tests'") 0
Travis Job 6213.4
5 6213.5 (CI_FLAGS="-fkmsrjlpn -i 0/6 'Python integration tests for pants - shard 1'") 0
Travis Job 6213.5
6 6213.6 (CI_FLAGS="-fkmsrjlpn -i 1/6 'Python integration tests for pants - shard 2'") 0
Travis Job 6213.6
7 6213.7 (CI_FLAGS="-fkmsrjlpn -i 2/6 'Python integration tests for pants - shard 3'") 0
Travis Job 6213.7
8 6213.8 (CI_FLAGS="-fkmsrjlpn -i 3/6 'Python integration tests for pants - shard 4'") 0
Travis Job 6213.8
9 6213.9 (CI_FLAGS="-fkmsrjlpn -i 4/6 'Python integration tests for pants - shard 5'") 0
Travis Job 6213.9
10 6213.10 (CI_FLAGS="-fkmsrjlpn -i 5/6 'Python integration tests for pants - shard 6'") 0
Travis Job 6213.10
Source Files on build 6213
Detailed source file information is not available for this build.