-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/53836/
-----------------------------------------------------------

(Updated Dec. 6, 2016, 5:30 p.m.)


Review request for Aurora, Joshua Cohen, Stephan Erb, and Zameer Manji.


Changes
-------

DRY up thrift dep, bump to latest pants and kill all default target names and 
globs.

 api/src/main/thrift/org/apache/aurora/gen/BUILD         | 10 +++-------
 api/src/main/thrift/org/apache/thermos/BUILD            |  4 +---
 build-support/thrift/BUILD                              |  7 +++++++
 pants.ini                                               |  2 +-
 src/main/python/apache/aurora/admin/BUILD               |  1 -
 src/main/python/apache/aurora/client/BUILD              |  1 -
 src/main/python/apache/aurora/common/BUILD              |  1 -
 src/main/python/apache/aurora/config/BUILD              |  1 -
 src/main/python/apache/aurora/executor/BUILD            |  1 -
 src/main/python/apache/aurora/kerberos/BUILD            |  1 -
 src/main/python/apache/aurora/tools/BUILD               |  1 -
 src/main/python/apache/thermos/cli/BUILD                |  1 -
 src/main/python/apache/thermos/common/BUILD             |  1 -
 src/main/python/apache/thermos/config/BUILD             |  1 -
 src/main/python/apache/thermos/core/BUILD               |  1 -
 src/main/python/apache/thermos/monitoring/BUILD         |  1 -
 src/main/python/apache/thermos/observer/BUILD           |  1 -
 src/main/python/apache/thermos/runner/BUILD             |  1 -
 src/main/python/apache/thermos/testing/BUILD            |  1 -
 src/test/python/apache/aurora/BUILD                     |  1 -
 src/test/python/apache/aurora/admin/BUILD               |  2 --
 src/test/python/apache/aurora/client/BUILD              |  2 --
 src/test/python/apache/aurora/client/api/BUILD          |  2 --
 src/test/python/apache/aurora/client/cli/BUILD          |  2 --
 src/test/python/apache/aurora/client/docker/BUILD       |  2 --
 src/test/python/apache/aurora/client/hooks/BUILD        |  3 +--
 src/test/python/apache/aurora/common/BUILD              |  2 --
 src/test/python/apache/aurora/common/health_check/BUILD |  2 --
 src/test/python/apache/aurora/config/BUILD              |  2 --
 src/test/python/apache/aurora/executor/BUILD            |  2 --
 src/test/python/apache/aurora/executor/bin/BUILD        |  2 --
 src/test/python/apache/aurora/executor/common/BUILD     |  2 --
 src/test/python/apache/aurora/tools/BUILD               |  2 --
 src/test/python/apache/thermos/cli/BUILD                |  2 --
 src/test/python/apache/thermos/cli/commands/BUILD       |  2 --
 src/test/python/apache/thermos/common/BUILD             |  2 --
 src/test/python/apache/thermos/config/BUILD             |  2 --
 src/test/python/apache/thermos/core/BUILD               |  2 --
 src/test/python/apache/thermos/monitoring/BUILD         |  2 --
 src/test/python/apache/thermos/observer/BUILD           |  2 --
 src/test/python/apache/thermos/observer/http/BUILD      |  2 --
 41 files changed, 13 insertions(+), 69 deletions(-)


Repository: aurora


Description
-------

This required an upgrade to the latest pants dev release to correct
an issue with the setup.py binary packager we use to generate sdists.

This is for sanity sake, and, once the TODO in
`build-support/thrift/prepare_binary.sh` is addressed, it will also
allow pants-patch-free addition of new platforms (thinking ARM).

 api/src/main/thrift/org/apache/aurora/gen/BUILD                        |  3 +++
 api/src/main/thrift/org/apache/thermos/BUILD                           |  1 +
 build-support/thrift/.gitignore                                        |  1 +
 build-support/thrift/AURORA-1727.compiler.cpp.Makefile.in.patch        | 42 
+++++++++++++------------------
 build-support/thrift/AURORA-1727.lib.py.setup.py.patch                 | 45 
+++++++++++++++++++++++++++++++++
 {api/src/main/thrift/org/apache/thermos => build-support/thrift}/BUILD | 18 
++++---------
 build-support/thrift/Makefile                                          |  2 +-
 build-support/thrift/prepare_binary.sh                                 | 68 
++++++++++++++++++++++++++++++++++++++++++++++++++
 build-support/thrift/thriftw                                           | 12 
+++++++--
 pants.ini                                                              |  7 
+++++-
 10 files changed, 158 insertions(+), 41 deletions(-)


Diffs (updated)
-----

  api/src/main/thrift/org/apache/aurora/gen/BUILD 
f4004a3ca36078c6d24991db4beb68903a05652c 
  api/src/main/thrift/org/apache/thermos/BUILD 
cb655a2fd35d22f7d7e80c4311742bad763d8614 
  build-support/thrift/.gitignore 9a3adb69210ba3dbf3c1f408895561da37e8f4c3 
  build-support/thrift/AURORA-1727.compiler.cpp.Makefile.in.patch 
b69e3fef137cd73c6f2b73201463a0705ef8082a 
  build-support/thrift/AURORA-1727.lib.py.setup.py.patch PRE-CREATION 
  build-support/thrift/BUILD PRE-CREATION 
  build-support/thrift/Makefile 48b174ad622288d2738a5fa37bbb72385fcc3a27 
  build-support/thrift/prepare_binary.sh PRE-CREATION 
  build-support/thrift/thriftw 50d6dfdeb16ca8bf14aaff7aa826e3d69c5e13f0 
  pants.ini cecdb277f327f77b2652f76a30fc8d4ffd9ff1db 
  src/main/python/apache/aurora/admin/BUILD 
b5d37f718a25d43d7ac07e30f789cf526bdfdcc2 
  src/main/python/apache/aurora/client/BUILD 
1411765ed47d4286e3ba0636bc3f3c1b29afac02 
  src/main/python/apache/aurora/common/BUILD 
0e4c51020946dc21953493bd43b944177c444c28 
  src/main/python/apache/aurora/config/BUILD 
12e7fe973f456d0847ce63d3b293131a7f4c3bdd 
  src/main/python/apache/aurora/executor/BUILD 
486230db34a22ea5dd0f68da911c0afb1afbcac0 
  src/main/python/apache/aurora/kerberos/BUILD 
847852fdf7762789579f8818c677dc1f79a76c73 
  src/main/python/apache/aurora/tools/BUILD 
6717f92c65d7bc1d71c67e37f57750c28f10fda3 
  src/main/python/apache/thermos/cli/BUILD 
a4932b2757a53ab8654c5d9c0fbae4446c9c5383 
  src/main/python/apache/thermos/common/BUILD 
0adabbfa3f0230dae95aca5002c128832916fdd6 
  src/main/python/apache/thermos/config/BUILD 
6e52c4bbe111529e14327de699218d9340e66fa9 
  src/main/python/apache/thermos/core/BUILD 
82448cec3380e237001f2fb02a28df4c5e5adb30 
  src/main/python/apache/thermos/monitoring/BUILD 
65ba7088f65e7baa5d30744736ba456b46a55e86 
  src/main/python/apache/thermos/observer/BUILD 
95b8dcd7d123d3a2bcb22df7efc60374ef919bf7 
  src/main/python/apache/thermos/runner/BUILD 
085b6ce35e8f67d9db0f6838bafeeb525191e6e9 
  src/main/python/apache/thermos/testing/BUILD 
24261701c5334999431a6cd41a6a711753180a89 
  src/test/python/apache/aurora/BUILD 626f1108a24e25747f2c57b4a9f21acbc32fd2d5 
  src/test/python/apache/aurora/admin/BUILD 
093537ed13406be3e8dff3245283b4d8d53481a3 
  src/test/python/apache/aurora/client/BUILD 
0dc7b8ed19e1216d027095afc43e063d17bedad1 
  src/test/python/apache/aurora/client/api/BUILD 
e9742240ec4847b4256b554a01dd2369f6fb62f5 
  src/test/python/apache/aurora/client/cli/BUILD 
16c4eb5bfbb751ba307f6f7b4335c2dbb99acb46 
  src/test/python/apache/aurora/client/docker/BUILD 
5ea2cf791cdec6011978770fc0cffe90e4e7735d 
  src/test/python/apache/aurora/client/hooks/BUILD 
47c86d98ef8e3eaaef3eba1b2e5179848c256fb9 
  src/test/python/apache/aurora/common/BUILD 
b3991ee6086269811cffbe5a65bf92fc74002379 
  src/test/python/apache/aurora/common/health_check/BUILD 
e73f0c0579d81ed1da8fa663e6a3af5dbc866c73 
  src/test/python/apache/aurora/config/BUILD 
ec7909d9442c986225f075ba3ec8e4bfce50dc85 
  src/test/python/apache/aurora/executor/BUILD 
d5724746dd0f8b4ef95abbe5a1e75e22548c1b55 
  src/test/python/apache/aurora/executor/bin/BUILD 
c9dcb9f1ed98dc98b52a5a5f0acd66845b0fffb3 
  src/test/python/apache/aurora/executor/common/BUILD 
5b794304659d6f80f05a140166c19856c8feef10 
  src/test/python/apache/aurora/tools/BUILD 
bf3bbf976968967873c7332708e254a7c2e2a14b 
  src/test/python/apache/thermos/cli/BUILD 
401bdba8c89ca47dfd65dc2142fcdb13d544a413 
  src/test/python/apache/thermos/cli/commands/BUILD 
7ed1d6ce067682c298e1a56d28c1b29de1e05086 
  src/test/python/apache/thermos/common/BUILD 
b570759b5829c144b7c90744dd5f86cd9f490289 
  src/test/python/apache/thermos/config/BUILD 
a507157a3f8df6a705339a91d6e82411d997bdf6 
  src/test/python/apache/thermos/core/BUILD 
957a116ca0ff9e951bbd49b25d0a6a199d994a2b 
  src/test/python/apache/thermos/monitoring/BUILD 
8f2b39336dce6c7b580e6ba0009f60afdcb89179 
  src/test/python/apache/thermos/observer/BUILD 
ec391f24344eefa7ad6ce50145458692bb8b1d97 
  src/test/python/apache/thermos/observer/http/BUILD 
708f09bef0755baebb64759eb4e920a1e213765b 

Diff: https://reviews.apache.org/r/53836/diff/


Testing
-------

Locally green:
```
rm -rf ~/.cache/pants .cache/
git clean -fdx build-support/thrift
./pants clean-all
./build-support/jenkins/build.sh

vagrant ssh --command "rm -rf ~/.cache/pants && cd aurora && rm -rf .cache/ && 
./pants clean-all"
./src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh

./build-support/python/make-pycharm-virtualenv
```


Thanks,

John Sirois

Reply via email to