Re: Review Request 39784: Upgrade Aurora to pants 0.0.55.

2015-11-10 Thread John Sirois 


> On Oct. 29, 2015, 4:57 p.m., Aurora ReviewBot wrote:
> > Master (bcb4774) is red with this patch.
> >   ./build-support/jenkins/build.sh
> > 
> > 22:56:59 00:00   [unpack-jars]
> > 22:56:59 00:00 [unpack-jars]
> > 22:56:59 00:00   [deferred-sources]
> > 22:56:59 00:00 [deferred-sources]
> > 22:56:59 00:00   [jvm-platform-validate]
> > 22:56:59 00:00 [jvm-platform-validate]
> > 22:56:59 00:00   [gen]
> > 22:56:59 00:00 [thrift]
> > 22:56:59 00:00 [protoc]
> > 22:56:59 00:00 [antlr]
> > 22:56:59 00:00 [ragel]
> > 22:56:59 00:00 [jaxb]
> > 22:57:00 00:01 [wire]
> > 22:57:00 00:01   [resolve]
> > 22:57:00 00:01 [ivy]
> > 22:57:00 00:01   [ivy-bootstrap]
> > 22:57:02 00:03   [bootstrap-nailgun-server]
> > 22:57:02 00:03   [compile]
> > 22:57:02 00:03 [compile]
> > 22:57:02 00:03 [jvm]
> > 22:57:02 00:03   [jvm-compilers]
> > 22:57:02 00:03 [zinc-pre]
> > 22:57:02 00:03 [zinc-post]
> > 22:57:03 00:04 [jvm-dep-check]
> > 22:57:03 00:04   [resources]
> > 22:57:03 00:04 [prepare]
> > 22:57:03 00:04 [services]
> > 22:57:03 00:04   [test]
> > 22:57:03 00:04 [run_prep_command]
> > 22:57:03 00:04 [test]
> > 22:57:03 00:04 [pytest]
> > 22:57:03 00:04   [run]
> >  
> > 22:57:07 00:08 [chroot]INFO] Attempting to fetch thrift binary 
> > from: 
> > https://dl.bintray.com/pantsbuild/bin/build-support/bin/thrift/linux/x86_64/0.9.1/thrift
> >  ...
> > INFO] Fetched thrift binary from: 
> > https://dl.bintray.com/pantsbuild/bin/build-support/bin/thrift/linux/x86_64/0.9.1/thrift
> >  .
> > 
> > 22:57:15 00:16   [complete]
> >FAILURE
> > Exception message: Ambiguous resolvable: thrift
> > 
> > 
> > 
> > I will refresh this build result if you post a review containing 
> > "@ReviewBot retry"
> 
> John Sirois wrote:
> I need to dig in here, this is unexpected.

This was pointed to a real issue, details on 
https://issues.apache.org/jira/browse/AURORA-1499.
The short is this RB is now blocked on pants 0.0.58 which will have a fix for 
this: https://github.com/pantsbuild/pants/issues/2533
Pants releases every Friday, so expect updates here Friday evening.


- John


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


On Nov. 9, 2015, 2:47 p.m., John Sirois wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39784/
> ---
> 
> (Updated Nov. 9, 2015, 2:47 p.m.)
> 
> 
> Review request for Aurora, Joe Smith, Bill Farner, and Zameer Manji.
> 
> 
> Bugs: AURORA-1499
> https://issues.apache.org/jira/browse/AURORA-1499
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> This also brings Aurora up to pex 1.1.0 and switches to the pants
> setup script; an equivalent to gradlew.  Of note, the script is checked
> in with chmod 555, its not intended to be edited.
> 
> Although pants now includes checkstyle built in, conversion to use it is
> left for follow-on work.
> 
> Also adapt make-pycharm-virtualenv which previously relied on the local
> bootstrapping of pants - now hidden away by the pants setup script.
> 
>  .pantsversion  |   1 -
>  3rdparty/python/requirements.txt   |   2 +-
>  BUILD  |  25 -
>  BUILD.tools|  19 --
>  build-support/jenkins/build.sh |   2 +-
>  build-support/pants_requirements.txt   |  30 ---
>  build-support/python/make-pycharm-virtualenv   |  11 +-
>  build-support/python/update-pants-requirements |  34 -
>  pants  | 102 
> +++---
>  pants.ini  |  63 
> ++-
>  src/main/python/apache/aurora/tools/BUILD  |   2 +-
>  src/main/python/apache/thermos/observer/BUILD  |   2 +-
>  12 files changed, 114 insertions(+), 179 deletions(-)
> 
> 
> Diffs
> -
> 
>   .pantsversion 78bae5bb6d254d014e35be0b828497f1509d80bd 
>   3rdparty/python/requirements.txt 27a2c77047c26ab380fc739e7c5c532bf590c6ee 
>   BUILD 7de0c74b03ba609576867ba96885858c0908f2e9 
>   BUILD.tools 75698a5ded7914a4d22ab7ae769d9ed1576531e4 
>   build-support/jenkins/build.sh 5606bb157cb117a588f363382d7c8841ae957138 
>   build-support/pants_requirements.txt 
> 6eefa7175f7bda7bea23466936e451113a2d86f7 
>   build-support/python/make-pycharm-virtualenv 
> d7bd41835bcd9a8fe62ea522a177bfa7830a897b 
>   build-support/python/update-pants-requirements 
> 82f7c5136fad52f4bc2a458beb5f34f0cc7f6cec 
>   pants

Re: Review Request 39784: Upgrade Aurora to pants 0.0.55.

2015-11-10 Thread Bill Farner 


> On Nov. 10, 2015, 12:18 p.m., John Sirois wrote:
> > @ReviewBot retry

FYI when you post a diff, the bot will pick it up.  The manual intervention 
should only be necessary for a flaked build.


- Bill


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


On Nov. 10, 2015, 12:17 p.m., John Sirois wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39784/
> ---
> 
> (Updated Nov. 10, 2015, 12:17 p.m.)
> 
> 
> Review request for Aurora, Joe Smith, Bill Farner, and Zameer Manji.
> 
> 
> Bugs: AURORA-1499
> https://issues.apache.org/jira/browse/AURORA-1499
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> See the changelog here:
>   https://pypi.python.org/pypi/pantsbuild.pants/0.0.57
>   
> This also brings Aurora up to pex 1.1.0 and switches to the pants
> setup script; an equivalent to gradlew.  Of note, the script is checked
> in with chmod 555, its not intended to be edited.
> 
> Although pants now includes checkstyle built in, conversion to use it is
> left for follow-on work.
> 
> Also adapt make-pycharm-virtualenv which previously relied on the local
> bootstrapping of pants - now hidden away by the pants setup script.
> 
>  .pantsversion  |   1 -
>  3rdparty/python/requirements.txt   |   2 +-
>  BUILD  |  25 -
>  BUILD.tools|  19 --
>  build-support/jenkins/build.sh |   2 +-
>  build-support/pants_requirements.txt   |  30 ---
>  build-support/python/make-pycharm-virtualenv   |  11 +-
>  build-support/python/update-pants-requirements |  34 -
>  pants  | 102 
> +++---
>  pants.ini  |  63 
> ++-
>  src/main/python/apache/aurora/tools/BUILD  |   2 +-
>  src/main/python/apache/thermos/observer/BUILD  |   2 +-
>  12 files changed, 114 insertions(+), 179 deletions(-)
> 
> 
> Diffs
> -
> 
>   .pantsversion 78bae5bb6d254d014e35be0b828497f1509d80bd 
>   3rdparty/python/requirements.txt 27a2c77047c26ab380fc739e7c5c532bf590c6ee 
>   BUILD 7de0c74b03ba609576867ba96885858c0908f2e9 
>   BUILD.tools 75698a5ded7914a4d22ab7ae769d9ed1576531e4 
>   api/src/main/thrift/org/apache/aurora/gen/BUILD 
> 2dcc46d9658044d96f4f1b3634f29ae1d446e0d7 
>   api/src/main/thrift/org/apache/thermos/BUILD 
> 6b13949aceb830b962a7098ae65462151119d752 
>   build-support/jenkins/build.sh 5606bb157cb117a588f363382d7c8841ae957138 
>   build-support/pants_requirements.txt 
> 6eefa7175f7bda7bea23466936e451113a2d86f7 
>   build-support/python/make-pycharm-virtualenv 
> d7bd41835bcd9a8fe62ea522a177bfa7830a897b 
>   build-support/python/update-pants-requirements 
> 82f7c5136fad52f4bc2a458beb5f34f0cc7f6cec 
>   pants 6f3526ef76fc37e3215b0673bcb1725d205a1c95 
>   pants.ini dd4ba668586ee5bd1c888f315429e661adb6a480 
>   src/main/python/apache/aurora/tools/BUILD 
> e5ac75838cbe98bbef4e35f6f300b6a6df5e7de5 
>   src/main/python/apache/thermos/observer/BUILD 
> d7eedabc0930711530b45ac98a1159e69d1a0c00 
> 
> Diff: https://reviews.apache.org/r/39784/diff/
> 
> 
> Testing
> ---
> 
> Locally: `./pants test.pytest --no-fast src/test/python:: -- -v`
> 
> Also generated a pycharm project via:
>   `./build-support/python/make-pycharm-virtualenv`
> Confirmed library source linking worked as did running unit tests
> via the IDE.
> 
> Also grepped for pants commands in the repo, found `binary` and `setup-py`
> and confirmed these worked.
> 
> 
> Thanks,
> 
> John Sirois
> 
>

Re: Review Request 39784: Upgrade Aurora to pants 0.0.55.

2015-11-10 Thread John Sirois 

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

(Updated Nov. 10, 2015, 1:17 p.m.)


Review request for Aurora, Joe Smith, Bill Farner, and Zameer Manji.


Bugs: AURORA-1499
https://issues.apache.org/jira/browse/AURORA-1499


Repository: aurora


Description (updated)
---

See the changelog here:
  https://pypi.python.org/pypi/pantsbuild.pants/0.0.57
  
This also brings Aurora up to pex 1.1.0 and switches to the pants
setup script; an equivalent to gradlew.  Of note, the script is checked
in with chmod 555, its not intended to be edited.

Although pants now includes checkstyle built in, conversion to use it is
left for follow-on work.

Also adapt make-pycharm-virtualenv which previously relied on the local
bootstrapping of pants - now hidden away by the pants setup script.

 .pantsversion  |   1 -
 3rdparty/python/requirements.txt   |   2 +-
 BUILD  |  25 -
 BUILD.tools|  19 --
 build-support/jenkins/build.sh |   2 +-
 build-support/pants_requirements.txt   |  30 ---
 build-support/python/make-pycharm-virtualenv   |  11 +-
 build-support/python/update-pants-requirements |  34 -
 pants  | 102 
+++---
 pants.ini  |  63 
++-
 src/main/python/apache/aurora/tools/BUILD  |   2 +-
 src/main/python/apache/thermos/observer/BUILD  |   2 +-
 12 files changed, 114 insertions(+), 179 deletions(-)


Diffs
-

  .pantsversion 78bae5bb6d254d014e35be0b828497f1509d80bd 
  3rdparty/python/requirements.txt 27a2c77047c26ab380fc739e7c5c532bf590c6ee 
  BUILD 7de0c74b03ba609576867ba96885858c0908f2e9 
  BUILD.tools 75698a5ded7914a4d22ab7ae769d9ed1576531e4 
  api/src/main/thrift/org/apache/aurora/gen/BUILD 
2dcc46d9658044d96f4f1b3634f29ae1d446e0d7 
  api/src/main/thrift/org/apache/thermos/BUILD 
6b13949aceb830b962a7098ae65462151119d752 
  build-support/jenkins/build.sh 5606bb157cb117a588f363382d7c8841ae957138 
  build-support/pants_requirements.txt 6eefa7175f7bda7bea23466936e451113a2d86f7 
  build-support/python/make-pycharm-virtualenv 
d7bd41835bcd9a8fe62ea522a177bfa7830a897b 
  build-support/python/update-pants-requirements 
82f7c5136fad52f4bc2a458beb5f34f0cc7f6cec 
  pants 6f3526ef76fc37e3215b0673bcb1725d205a1c95 
  pants.ini dd4ba668586ee5bd1c888f315429e661adb6a480 
  src/main/python/apache/aurora/tools/BUILD 
e5ac75838cbe98bbef4e35f6f300b6a6df5e7de5 
  src/main/python/apache/thermos/observer/BUILD 
d7eedabc0930711530b45ac98a1159e69d1a0c00 

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


Testing
---

Locally: `./pants test.pytest --no-fast src/test/python:: -- -v`

Also generated a pycharm project via:
  `./build-support/python/make-pycharm-virtualenv`
Confirmed library source linking worked as did running unit tests
via the IDE.

Also grepped for pants commands in the repo, found `binary` and `setup-py`
and confirmed these worked.


Thanks,

John Sirois

Re: Review Request 39784: Upgrade Aurora to pants 0.0.55.

2015-11-10 Thread John Sirois 

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

(Updated Nov. 10, 2015, 1:06 p.m.)


Review request for Aurora, Joe Smith, Bill Farner, and Zameer Manji.


Changes
---

Fixup aurora api thrift deps to work with commons.

Previously the aurora api specified no thrift requirement and let pants
resolve it.  Upgrading pants leads to a conflict with twitter.commons
pinned `thrift==0.9.1` requirements so we work around by surfacing the
same pinning.  The resolution solution for aurora itself and any higher
level clients built on aurora libs is either unchanged or of no import:
+ If the 3rdparty has no thrift dep they don't care by definition, the
  new solution will pick exactly `0.9.1` instead of floating up to -
  right now `0.9.3` for example - a good thing.
+ If the 3rdparty pins their thrift requirement, nothing changes for
  them whether they use `pip` or `pex` or `pants`; their higher level
  dep trumps since all 3 tools do a breadth 1st search and 1st level
  wins.

Also restore a TODO to kill BUILD.tools and un-pin setuptools, let the
pants defaults (more modern setuptools) work.

 BUILD.tools | 2 ++
 api/src/main/thrift/org/apache/aurora/gen/BUILD | 7 +++
 api/src/main/thrift/org/apache/thermos/BUILD| 5 -
 pants.ini   | 7 ++-
 4 files changed, 15 insertions(+), 6 deletions(-)


Bugs: AURORA-1499
https://issues.apache.org/jira/browse/AURORA-1499


Repository: aurora


Description
---

This also brings Aurora up to pex 1.1.0 and switches to the pants
setup script; an equivalent to gradlew.  Of note, the script is checked
in with chmod 555, its not intended to be edited.

Although pants now includes checkstyle built in, conversion to use it is
left for follow-on work.

Also adapt make-pycharm-virtualenv which previously relied on the local
bootstrapping of pants - now hidden away by the pants setup script.

 .pantsversion  |   1 -
 3rdparty/python/requirements.txt   |   2 +-
 BUILD  |  25 -
 BUILD.tools|  19 --
 build-support/jenkins/build.sh |   2 +-
 build-support/pants_requirements.txt   |  30 ---
 build-support/python/make-pycharm-virtualenv   |  11 +-
 build-support/python/update-pants-requirements |  34 -
 pants  | 102 
+++---
 pants.ini  |  63 
++-
 src/main/python/apache/aurora/tools/BUILD  |   2 +-
 src/main/python/apache/thermos/observer/BUILD  |   2 +-
 12 files changed, 114 insertions(+), 179 deletions(-)


Diffs (updated)
-

  .pantsversion 78bae5bb6d254d014e35be0b828497f1509d80bd 
  3rdparty/python/requirements.txt 27a2c77047c26ab380fc739e7c5c532bf590c6ee 
  BUILD 7de0c74b03ba609576867ba96885858c0908f2e9 
  BUILD.tools 75698a5ded7914a4d22ab7ae769d9ed1576531e4 
  api/src/main/thrift/org/apache/aurora/gen/BUILD 
2dcc46d9658044d96f4f1b3634f29ae1d446e0d7 
  api/src/main/thrift/org/apache/thermos/BUILD 
6b13949aceb830b962a7098ae65462151119d752 
  build-support/jenkins/build.sh 5606bb157cb117a588f363382d7c8841ae957138 
  build-support/pants_requirements.txt 6eefa7175f7bda7bea23466936e451113a2d86f7 
  build-support/python/make-pycharm-virtualenv 
d7bd41835bcd9a8fe62ea522a177bfa7830a897b 
  build-support/python/update-pants-requirements 
82f7c5136fad52f4bc2a458beb5f34f0cc7f6cec 
  pants 6f3526ef76fc37e3215b0673bcb1725d205a1c95 
  pants.ini dd4ba668586ee5bd1c888f315429e661adb6a480 
  src/main/python/apache/aurora/tools/BUILD 
e5ac75838cbe98bbef4e35f6f300b6a6df5e7de5 
  src/main/python/apache/thermos/observer/BUILD 
d7eedabc0930711530b45ac98a1159e69d1a0c00 

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


Testing
---

Locally: `./pants test.pytest --no-fast src/test/python:: -- -v`

Also generated a pycharm project via:
  `./build-support/python/make-pycharm-virtualenv`
Confirmed library source linking worked as did running unit tests
via the IDE.

Also grepped for pants commands in the repo, found `binary` and `setup-py`
and confirmed these worked.


Thanks,

John Sirois

Re: Review Request 39784: Upgrade Aurora to pants 0.0.55.

2015-11-10 Thread Aurora ReviewBot 

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


Master (3ea1b26) is red with this patch.
  ./build-support/jenkins/build.sh

   Running packager against 
/home/jenkins/jenkins-slave/workspace/AuroraBot/dist/apache.aurora.thrift-0.11.0-SNAPSHOT
   Writing 
/home/jenkins/jenkins-slave/workspace/AuroraBot/dist/apache.aurora.thrift-0.11.0-SNAPSHOT.tar.gz
   Running packager against 
/home/jenkins/jenkins-slave/workspace/AuroraBot/dist/apache.thermos.thrift-0.11.0-SNAPSHOT
   Writing 
/home/jenkins/jenkins-slave/workspace/AuroraBot/dist/apache.thermos.thrift-0.11.0-SNAPSHOT.tar.gz
   Running packager against 
/home/jenkins/jenkins-slave/workspace/AuroraBot/dist/apache.thermos.common-0.11.0-SNAPSHOT
   Writing 
/home/jenkins/jenkins-slave/workspace/AuroraBot/dist/apache.thermos.common-0.11.0-SNAPSHOT.tar.gz
   Running packager against 
/home/jenkins/jenkins-slave/workspace/AuroraBot/dist/apache.thermos.monitoring-0.11.0-SNAPSHOT
   Writing 
/home/jenkins/jenkins-slave/workspace/AuroraBot/dist/apache.thermos.monitoring-0.11.0-SNAPSHOT.tar.gz
   Running packager against 
/home/jenkins/jenkins-slave/workspace/AuroraBot/dist/apache.thermos.config-0.11.0-SNAPSHOT
   Writing 
/home/jenkins/jenkins-slave/workspace/AuroraBot/dist/apache.thermos.config-0.11.0-SNAPSHOT.tar.gz
   Running packager against 
/home/jenkins/jenkins-slave/workspace/AuroraBot/dist/apache.thermos.core-0.11.0-SNAPSHOT
   Writing 
/home/jenkins/jenkins-slave/workspace/AuroraBot/dist/apache.thermos.core-0.11.0-SNAPSHOT.tar.gz
   Running packager against 
/home/jenkins/jenkins-slave/workspace/AuroraBot/dist/apache.aurora.common-0.11.0-SNAPSHOT
   Writing 
/home/jenkins/jenkins-slave/workspace/AuroraBot/dist/apache.aurora.common-0.11.0-SNAPSHOT.tar.gz
   Running packager against 
/home/jenkins/jenkins-slave/workspace/AuroraBot/dist/apache.aurora.config-0.11.0-SNAPSHOT
   Writing 
/home/jenkins/jenkins-slave/workspace/AuroraBot/dist/apache.aurora.config-0.11.0-SNAPSHOT.tar.gz
   Running packager against 
/home/jenkins/jenkins-slave/workspace/AuroraBot/dist/apache.aurora.executor-0.11.0-SNAPSHOT
   Writing 
/home/jenkins/jenkins-slave/workspace/AuroraBot/dist/apache.aurora.executor-0.11.0-SNAPSHOT.tar.gz
20:17:56 00:03   [complete]
   SUCCESS
INFO] Detected git repository at 
/home/jenkins/jenkins-slave/workspace/AuroraBot on branch None

20:17:56 00:00 [main]
   (To run a reporting server: ./pants server)
20:17:56 00:00   [bootstrap]
20:17:56 00:00   [setup]
20:17:56 00:00 [parse]
   Executing tasks in goals: setup-py
20:17:56 00:00   [setup-py]
20:17:56 00:00 [setup-py]
   Creating setup.py project for 
PythonLibrary(BuildFileAddress(FilesystemBuildFile(/home/jenkins/jenkins-slave/workspace/AuroraBot/src/main/python/apache/aurora/kerberos/BUILD),
 kerberos))
   Creating setup.py project for 
PythonLibrary(BuildFileAddress(FilesystemBuildFile(/home/jenkins/jenkins-slave/workspace/AuroraBot/src/main/python/apache/aurora/common/BUILD),
 common))
   Creating setup.py project for 
PythonLibrary(BuildFileAddress(FilesystemBuildFile(/home/jenkins/jenkins-slave/workspace/AuroraBot/api/src/main/thrift/org/apache/aurora/gen/BUILD),
 gen))
   Creating setup.py project for 
PythonLibrary(BuildFileAddress(FilesystemBuildFile(/home/jenkins/jenkins-slave/workspace/AuroraBot/src/main/python/apache/aurora/admin/BUILD),
 admin))
   Creating setup.py project for 
PythonLibrary(BuildFileAddress(FilesystemBuildFile(/home/jenkins/jenkins-slave/workspace/AuroraBot/src/main/python/apache/aurora/client/BUILD),
 client))
FAILURE: No exported target owner found for 
PythonLibrary(BuildFileAddress(FilesystemBuildFile(/home/jenkins/jenkins-slave/workspace/AuroraBot/src/main/resources/apache/aurora/client/cli/BUILD),
 auroraversion))


20:17:57 00:01   [complete]
   FAILURE


I will refresh this build result if you post a review containing "@ReviewBot 
retry"

- Aurora ReviewBot


On Nov. 10, 2015, 8:17 p.m., John Sirois wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39784/
> ---
> 
>

Re: Review Request 39784: Upgrade Aurora to pants 0.0.55.

2015-11-09 Thread John Sirois 

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


@ReviewBot retry

- John Sirois


On Nov. 9, 2015, 2:47 p.m., John Sirois wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39784/
> ---
> 
> (Updated Nov. 9, 2015, 2:47 p.m.)
> 
> 
> Review request for Aurora, Joe Smith, Bill Farner, and Zameer Manji.
> 
> 
> Bugs: AURORA-1499
> https://issues.apache.org/jira/browse/AURORA-1499
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> This also brings Aurora up to pex 1.1.0 and switches to the pants
> setup script; an equivalent to gradlew.  Of note, the script is checked
> in with chmod 555, its not intended to be edited.
> 
> Although pants now includes checkstyle built in, conversion to use it is
> left for follow-on work.
> 
> Also adapt make-pycharm-virtualenv which previously relied on the local
> bootstrapping of pants - now hidden away by the pants setup script.
> 
>  .pantsversion  |   1 -
>  3rdparty/python/requirements.txt   |   2 +-
>  BUILD  |  25 -
>  BUILD.tools|  19 --
>  build-support/jenkins/build.sh |   2 +-
>  build-support/pants_requirements.txt   |  30 ---
>  build-support/python/make-pycharm-virtualenv   |  11 +-
>  build-support/python/update-pants-requirements |  34 -
>  pants  | 102 
> +++---
>  pants.ini  |  63 
> ++-
>  src/main/python/apache/aurora/tools/BUILD  |   2 +-
>  src/main/python/apache/thermos/observer/BUILD  |   2 +-
>  12 files changed, 114 insertions(+), 179 deletions(-)
> 
> 
> Diffs
> -
> 
>   .pantsversion 78bae5bb6d254d014e35be0b828497f1509d80bd 
>   3rdparty/python/requirements.txt 27a2c77047c26ab380fc739e7c5c532bf590c6ee 
>   BUILD 7de0c74b03ba609576867ba96885858c0908f2e9 
>   BUILD.tools 75698a5ded7914a4d22ab7ae769d9ed1576531e4 
>   build-support/jenkins/build.sh 5606bb157cb117a588f363382d7c8841ae957138 
>   build-support/pants_requirements.txt 
> 6eefa7175f7bda7bea23466936e451113a2d86f7 
>   build-support/python/make-pycharm-virtualenv 
> d7bd41835bcd9a8fe62ea522a177bfa7830a897b 
>   build-support/python/update-pants-requirements 
> 82f7c5136fad52f4bc2a458beb5f34f0cc7f6cec 
>   pants 6f3526ef76fc37e3215b0673bcb1725d205a1c95 
>   pants.ini dd4ba668586ee5bd1c888f315429e661adb6a480 
>   src/main/python/apache/aurora/tools/BUILD 
> e5ac75838cbe98bbef4e35f6f300b6a6df5e7de5 
>   src/main/python/apache/thermos/observer/BUILD 
> d7eedabc0930711530b45ac98a1159e69d1a0c00 
> 
> Diff: https://reviews.apache.org/r/39784/diff/
> 
> 
> Testing
> ---
> 
> Locally: `./pants test.pytest --no-fast src/test/python:: -- -v`
> 
> Also generated a pycharm project via:
>   `./build-support/python/make-pycharm-virtualenv`
> Confirmed library source linking worked as did running unit tests
> via the IDE.
> 
> Also grepped for pants commands in the repo, found `binary` and `setup-py`
> and confirmed these worked.
> 
> 
> Thanks,
> 
> John Sirois
> 
>

Re: Review Request 39784: Upgrade Aurora to pants 0.0.55.

2015-11-09 Thread John Sirois 

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


@ReviewBot retry

- John Sirois


On Oct. 29, 2015, 4:05 p.m., John Sirois wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39784/
> ---
> 
> (Updated Oct. 29, 2015, 4:05 p.m.)
> 
> 
> Review request for Aurora, Joe Smith, Bill Farner, and Zameer Manji.
> 
> 
> Bugs: AURORA-1499
> https://issues.apache.org/jira/browse/AURORA-1499
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> This also brings Aurora up to pex 1.1.0 and switches to the pants
> setup script; an equivalent to gradlew.  Of note, the script is checked
> in with chmod 555, its not intended to be edited.
> 
> Although pants now includes checkstyle built in, conversion to use it is
> left for follow-on work.
> 
> Also adapt make-pycharm-virtualenv which previously relied on the local
> bootstrapping of pants - now hidden away by the pants setup script.
> 
>  .pantsversion  |   1 -
>  3rdparty/python/requirements.txt   |   2 +-
>  BUILD  |  25 -
>  BUILD.tools|  19 --
>  build-support/jenkins/build.sh |   2 +-
>  build-support/pants_requirements.txt   |  30 ---
>  build-support/python/make-pycharm-virtualenv   |  11 +-
>  build-support/python/update-pants-requirements |  34 -
>  pants  | 102 
> +++---
>  pants.ini  |  63 
> ++-
>  src/main/python/apache/aurora/tools/BUILD  |   2 +-
>  src/main/python/apache/thermos/observer/BUILD  |   2 +-
>  12 files changed, 114 insertions(+), 179 deletions(-)
> 
> 
> Diffs
> -
> 
>   .pantsversion 78bae5bb6d254d014e35be0b828497f1509d80bd 
>   3rdparty/python/requirements.txt 1eeb36dee1a0ebd33999bbb3327338a51cba00d7 
>   BUILD 7de0c74b03ba609576867ba96885858c0908f2e9 
>   BUILD.tools 75698a5ded7914a4d22ab7ae769d9ed1576531e4 
>   build-support/jenkins/build.sh 5606bb157cb117a588f363382d7c8841ae957138 
>   build-support/pants_requirements.txt 
> fad8da5ce4c03f25554394bc628d4fbb0fe6cd69 
>   build-support/python/make-pycharm-virtualenv 
> d7bd41835bcd9a8fe62ea522a177bfa7830a897b 
>   build-support/python/update-pants-requirements 
> 82f7c5136fad52f4bc2a458beb5f34f0cc7f6cec 
>   pants 6f3526ef76fc37e3215b0673bcb1725d205a1c95 
>   pants.ini dd4ba668586ee5bd1c888f315429e661adb6a480 
>   src/main/python/apache/aurora/tools/BUILD 
> e5ac75838cbe98bbef4e35f6f300b6a6df5e7de5 
>   src/main/python/apache/thermos/observer/BUILD 
> d7eedabc0930711530b45ac98a1159e69d1a0c00 
> 
> Diff: https://reviews.apache.org/r/39784/diff/
> 
> 
> Testing
> ---
> 
> Locally: `./pants test.pytest --no-fast src/test/python:: -- -v`
> 
> Also generated a pycharm project via:
>   `./build-support/python/make-pycharm-virtualenv`
> Confirmed library source linking worked as did running unit tests
> via the IDE.
> 
> Also grepped for pants commands in the repo, found `binary` and `setup-py`
> and confirmed these worked.
> 
> 
> Thanks,
> 
> John Sirois
> 
>

Re: Review Request 39784: Upgrade Aurora to pants 0.0.55.

2015-11-02 Thread John Sirois 


> On Oct. 29, 2015, 4:15 p.m., Zameer Manji wrote:
> > John, would you mind filing a ticket to track the followup work to use the 
> > built in pants checkstyle?
> 
> Joe Smith wrote:
> https://issues.apache.org/jira/browse/AURORA-1532

Thanks Joe.


- John


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


On Oct. 29, 2015, 4:05 p.m., John Sirois wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39784/
> ---
> 
> (Updated Oct. 29, 2015, 4:05 p.m.)
> 
> 
> Review request for Aurora, Joe Smith, Bill Farner, and Zameer Manji.
> 
> 
> Bugs: AURORA-1499
> https://issues.apache.org/jira/browse/AURORA-1499
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> This also brings Aurora up to pex 1.1.0 and switches to the pants
> setup script; an equivalent to gradlew.  Of note, the script is checked
> in with chmod 555, its not intended to be edited.
> 
> Although pants now includes checkstyle built in, conversion to use it is
> left for follow-on work.
> 
> Also adapt make-pycharm-virtualenv which previously relied on the local
> bootstrapping of pants - now hidden away by the pants setup script.
> 
>  .pantsversion  |   1 -
>  3rdparty/python/requirements.txt   |   2 +-
>  BUILD  |  25 -
>  BUILD.tools|  19 --
>  build-support/jenkins/build.sh |   2 +-
>  build-support/pants_requirements.txt   |  30 ---
>  build-support/python/make-pycharm-virtualenv   |  11 +-
>  build-support/python/update-pants-requirements |  34 -
>  pants  | 102 
> +++---
>  pants.ini  |  63 
> ++-
>  src/main/python/apache/aurora/tools/BUILD  |   2 +-
>  src/main/python/apache/thermos/observer/BUILD  |   2 +-
>  12 files changed, 114 insertions(+), 179 deletions(-)
> 
> 
> Diffs
> -
> 
>   .pantsversion 78bae5bb6d254d014e35be0b828497f1509d80bd 
>   3rdparty/python/requirements.txt 1eeb36dee1a0ebd33999bbb3327338a51cba00d7 
>   BUILD 7de0c74b03ba609576867ba96885858c0908f2e9 
>   BUILD.tools 75698a5ded7914a4d22ab7ae769d9ed1576531e4 
>   build-support/jenkins/build.sh 5606bb157cb117a588f363382d7c8841ae957138 
>   build-support/pants_requirements.txt 
> fad8da5ce4c03f25554394bc628d4fbb0fe6cd69 
>   build-support/python/make-pycharm-virtualenv 
> d7bd41835bcd9a8fe62ea522a177bfa7830a897b 
>   build-support/python/update-pants-requirements 
> 82f7c5136fad52f4bc2a458beb5f34f0cc7f6cec 
>   pants 6f3526ef76fc37e3215b0673bcb1725d205a1c95 
>   pants.ini dd4ba668586ee5bd1c888f315429e661adb6a480 
>   src/main/python/apache/aurora/tools/BUILD 
> e5ac75838cbe98bbef4e35f6f300b6a6df5e7de5 
>   src/main/python/apache/thermos/observer/BUILD 
> d7eedabc0930711530b45ac98a1159e69d1a0c00 
> 
> Diff: https://reviews.apache.org/r/39784/diff/
> 
> 
> Testing
> ---
> 
> Locally: `./pants test.pytest --no-fast src/test/python:: -- -v`
> 
> Also generated a pycharm project via:
>   `./build-support/python/make-pycharm-virtualenv`
> Confirmed library source linking worked as did running unit tests
> via the IDE.
> 
> Also grepped for pants commands in the repo, found `binary` and `setup-py`
> and confirmed these worked.
> 
> 
> Thanks,
> 
> John Sirois
> 
>

Re: Review Request 39784: Upgrade Aurora to pants 0.0.55.

2015-11-02 Thread John Sirois 


> On Oct. 29, 2015, 4:57 p.m., Aurora ReviewBot wrote:
> > Master (bcb4774) is red with this patch.
> >   ./build-support/jenkins/build.sh
> > 
> > 22:56:59 00:00   [unpack-jars]
> > 22:56:59 00:00 [unpack-jars]
> > 22:56:59 00:00   [deferred-sources]
> > 22:56:59 00:00 [deferred-sources]
> > 22:56:59 00:00   [jvm-platform-validate]
> > 22:56:59 00:00 [jvm-platform-validate]
> > 22:56:59 00:00   [gen]
> > 22:56:59 00:00 [thrift]
> > 22:56:59 00:00 [protoc]
> > 22:56:59 00:00 [antlr]
> > 22:56:59 00:00 [ragel]
> > 22:56:59 00:00 [jaxb]
> > 22:57:00 00:01 [wire]
> > 22:57:00 00:01   [resolve]
> > 22:57:00 00:01 [ivy]
> > 22:57:00 00:01   [ivy-bootstrap]
> > 22:57:02 00:03   [bootstrap-nailgun-server]
> > 22:57:02 00:03   [compile]
> > 22:57:02 00:03 [compile]
> > 22:57:02 00:03 [jvm]
> > 22:57:02 00:03   [jvm-compilers]
> > 22:57:02 00:03 [zinc-pre]
> > 22:57:02 00:03 [zinc-post]
> > 22:57:03 00:04 [jvm-dep-check]
> > 22:57:03 00:04   [resources]
> > 22:57:03 00:04 [prepare]
> > 22:57:03 00:04 [services]
> > 22:57:03 00:04   [test]
> > 22:57:03 00:04 [run_prep_command]
> > 22:57:03 00:04 [test]
> > 22:57:03 00:04 [pytest]
> > 22:57:03 00:04   [run]
> >  
> > 22:57:07 00:08 [chroot]INFO] Attempting to fetch thrift binary 
> > from: 
> > https://dl.bintray.com/pantsbuild/bin/build-support/bin/thrift/linux/x86_64/0.9.1/thrift
> >  ...
> > INFO] Fetched thrift binary from: 
> > https://dl.bintray.com/pantsbuild/bin/build-support/bin/thrift/linux/x86_64/0.9.1/thrift
> >  .
> > 
> > 22:57:15 00:16   [complete]
> >FAILURE
> > Exception message: Ambiguous resolvable: thrift
> > 
> > 
> > 
> > I will refresh this build result if you post a review containing 
> > "@ReviewBot retry"

I need to dig in here, this is unexpected.


- John


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


On Oct. 29, 2015, 4:05 p.m., John Sirois wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39784/
> ---
> 
> (Updated Oct. 29, 2015, 4:05 p.m.)
> 
> 
> Review request for Aurora, Joe Smith, Bill Farner, and Zameer Manji.
> 
> 
> Bugs: AURORA-1499
> https://issues.apache.org/jira/browse/AURORA-1499
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> This also brings Aurora up to pex 1.1.0 and switches to the pants
> setup script; an equivalent to gradlew.  Of note, the script is checked
> in with chmod 555, its not intended to be edited.
> 
> Although pants now includes checkstyle built in, conversion to use it is
> left for follow-on work.
> 
> Also adapt make-pycharm-virtualenv which previously relied on the local
> bootstrapping of pants - now hidden away by the pants setup script.
> 
>  .pantsversion  |   1 -
>  3rdparty/python/requirements.txt   |   2 +-
>  BUILD  |  25 -
>  BUILD.tools|  19 --
>  build-support/jenkins/build.sh |   2 +-
>  build-support/pants_requirements.txt   |  30 ---
>  build-support/python/make-pycharm-virtualenv   |  11 +-
>  build-support/python/update-pants-requirements |  34 -
>  pants  | 102 
> +++---
>  pants.ini  |  63 
> ++-
>  src/main/python/apache/aurora/tools/BUILD  |   2 +-
>  src/main/python/apache/thermos/observer/BUILD  |   2 +-
>  12 files changed, 114 insertions(+), 179 deletions(-)
> 
> 
> Diffs
> -
> 
>   .pantsversion 78bae5bb6d254d014e35be0b828497f1509d80bd 
>   3rdparty/python/requirements.txt 1eeb36dee1a0ebd33999bbb3327338a51cba00d7 
>   BUILD 7de0c74b03ba609576867ba96885858c0908f2e9 
>   BUILD.tools 75698a5ded7914a4d22ab7ae769d9ed1576531e4 
>   build-support/jenkins/build.sh 5606bb157cb117a588f363382d7c8841ae957138 
>   build-support/pants_requirements.txt 
> fad8da5ce4c03f25554394bc628d4fbb0fe6cd69 
>   build-support/python/make-pycharm-virtualenv 
> d7bd41835bcd9a8fe62ea522a177bfa7830a897b 
>   build-support/python/update-pants-requirements 
> 82f7c5136fad52f4bc2a458beb5f34f0cc7f6cec 
>   pants 6f3526ef76fc37e3215b0673bcb1725d205a1c95 
>   pants.ini dd4ba668586ee5bd1c888f315429e661adb6a480 
>   src/main/python/apache/aurora/tools/BUILD 
> e5ac75838cbe98bbef4e35f6f300b6a6df5e7de5 
>   src/main/python/apache/thermos/observer/BUILD 
> d7eedabc0930711530b45ac98a1159e69d1a0c00 
> 
> Diff: https://reviews.apache.org/r/39784/diff/

Re: Review Request 39784: Upgrade Aurora to pants 0.0.55.

2015-11-02 Thread John Sirois 


> On Oct. 29, 2015, 4:24 p.m., Zameer Manji wrote:
> > Would you also mind updating the commit message to point to the pants 
> > changelog?

Will do.


> On Oct. 29, 2015, 4:24 p.m., Zameer Manji wrote:
> > BUILD.tools, line 1
> > 
> >
> > Aurora only uses pants for Python code. If we need to specify these JVM 
> > artifacts for it to work right now, can you please maintain the reference 
> > to issue 867?

Did not mean to nuke that - will restore.


> On Oct. 29, 2015, 4:24 p.m., Zameer Manji wrote:
> > pants.ini, line 14
> > 
> >
> > Is this comment relevant anymore? If not mind removing it?

I'll kill it, its not a lie, but not particularaly useful.


- John


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


On Oct. 29, 2015, 4:05 p.m., John Sirois wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39784/
> ---
> 
> (Updated Oct. 29, 2015, 4:05 p.m.)
> 
> 
> Review request for Aurora, Joe Smith, Bill Farner, and Zameer Manji.
> 
> 
> Bugs: AURORA-1499
> https://issues.apache.org/jira/browse/AURORA-1499
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> This also brings Aurora up to pex 1.1.0 and switches to the pants
> setup script; an equivalent to gradlew.  Of note, the script is checked
> in with chmod 555, its not intended to be edited.
> 
> Although pants now includes checkstyle built in, conversion to use it is
> left for follow-on work.
> 
> Also adapt make-pycharm-virtualenv which previously relied on the local
> bootstrapping of pants - now hidden away by the pants setup script.
> 
>  .pantsversion  |   1 -
>  3rdparty/python/requirements.txt   |   2 +-
>  BUILD  |  25 -
>  BUILD.tools|  19 --
>  build-support/jenkins/build.sh |   2 +-
>  build-support/pants_requirements.txt   |  30 ---
>  build-support/python/make-pycharm-virtualenv   |  11 +-
>  build-support/python/update-pants-requirements |  34 -
>  pants  | 102 
> +++---
>  pants.ini  |  63 
> ++-
>  src/main/python/apache/aurora/tools/BUILD  |   2 +-
>  src/main/python/apache/thermos/observer/BUILD  |   2 +-
>  12 files changed, 114 insertions(+), 179 deletions(-)
> 
> 
> Diffs
> -
> 
>   .pantsversion 78bae5bb6d254d014e35be0b828497f1509d80bd 
>   3rdparty/python/requirements.txt 1eeb36dee1a0ebd33999bbb3327338a51cba00d7 
>   BUILD 7de0c74b03ba609576867ba96885858c0908f2e9 
>   BUILD.tools 75698a5ded7914a4d22ab7ae769d9ed1576531e4 
>   build-support/jenkins/build.sh 5606bb157cb117a588f363382d7c8841ae957138 
>   build-support/pants_requirements.txt 
> fad8da5ce4c03f25554394bc628d4fbb0fe6cd69 
>   build-support/python/make-pycharm-virtualenv 
> d7bd41835bcd9a8fe62ea522a177bfa7830a897b 
>   build-support/python/update-pants-requirements 
> 82f7c5136fad52f4bc2a458beb5f34f0cc7f6cec 
>   pants 6f3526ef76fc37e3215b0673bcb1725d205a1c95 
>   pants.ini dd4ba668586ee5bd1c888f315429e661adb6a480 
>   src/main/python/apache/aurora/tools/BUILD 
> e5ac75838cbe98bbef4e35f6f300b6a6df5e7de5 
>   src/main/python/apache/thermos/observer/BUILD 
> d7eedabc0930711530b45ac98a1159e69d1a0c00 
> 
> Diff: https://reviews.apache.org/r/39784/diff/
> 
> 
> Testing
> ---
> 
> Locally: `./pants test.pytest --no-fast src/test/python:: -- -v`
> 
> Also generated a pycharm project via:
>   `./build-support/python/make-pycharm-virtualenv`
> Confirmed library source linking worked as did running unit tests
> via the IDE.
> 
> Also grepped for pants commands in the repo, found `binary` and `setup-py`
> and confirmed these worked.
> 
> 
> Thanks,
> 
> John Sirois
> 
>

Re: Review Request 39784: Upgrade Aurora to pants 0.0.55.

2015-10-30 Thread Joe Smith 


> On Oct. 29, 2015, 3:15 p.m., Zameer Manji wrote:
> > John, would you mind filing a ticket to track the followup work to use the 
> > built in pants checkstyle?

https://issues.apache.org/jira/browse/AURORA-1532


- Joe


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


On Oct. 29, 2015, 3:05 p.m., John Sirois wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39784/
> ---
> 
> (Updated Oct. 29, 2015, 3:05 p.m.)
> 
> 
> Review request for Aurora, Joe Smith, Bill Farner, and Zameer Manji.
> 
> 
> Bugs: AURORA-1499
> https://issues.apache.org/jira/browse/AURORA-1499
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> This also brings Aurora up to pex 1.1.0 and switches to the pants
> setup script; an equivalent to gradlew.  Of note, the script is checked
> in with chmod 555, its not intended to be edited.
> 
> Although pants now includes checkstyle built in, conversion to use it is
> left for follow-on work.
> 
> Also adapt make-pycharm-virtualenv which previously relied on the local
> bootstrapping of pants - now hidden away by the pants setup script.
> 
>  .pantsversion  |   1 -
>  3rdparty/python/requirements.txt   |   2 +-
>  BUILD  |  25 -
>  BUILD.tools|  19 --
>  build-support/jenkins/build.sh |   2 +-
>  build-support/pants_requirements.txt   |  30 ---
>  build-support/python/make-pycharm-virtualenv   |  11 +-
>  build-support/python/update-pants-requirements |  34 -
>  pants  | 102 
> +++---
>  pants.ini  |  63 
> ++-
>  src/main/python/apache/aurora/tools/BUILD  |   2 +-
>  src/main/python/apache/thermos/observer/BUILD  |   2 +-
>  12 files changed, 114 insertions(+), 179 deletions(-)
> 
> 
> Diffs
> -
> 
>   .pantsversion 78bae5bb6d254d014e35be0b828497f1509d80bd 
>   3rdparty/python/requirements.txt 1eeb36dee1a0ebd33999bbb3327338a51cba00d7 
>   BUILD 7de0c74b03ba609576867ba96885858c0908f2e9 
>   BUILD.tools 75698a5ded7914a4d22ab7ae769d9ed1576531e4 
>   build-support/jenkins/build.sh 5606bb157cb117a588f363382d7c8841ae957138 
>   build-support/pants_requirements.txt 
> fad8da5ce4c03f25554394bc628d4fbb0fe6cd69 
>   build-support/python/make-pycharm-virtualenv 
> d7bd41835bcd9a8fe62ea522a177bfa7830a897b 
>   build-support/python/update-pants-requirements 
> 82f7c5136fad52f4bc2a458beb5f34f0cc7f6cec 
>   pants 6f3526ef76fc37e3215b0673bcb1725d205a1c95 
>   pants.ini dd4ba668586ee5bd1c888f315429e661adb6a480 
>   src/main/python/apache/aurora/tools/BUILD 
> e5ac75838cbe98bbef4e35f6f300b6a6df5e7de5 
>   src/main/python/apache/thermos/observer/BUILD 
> d7eedabc0930711530b45ac98a1159e69d1a0c00 
> 
> Diff: https://reviews.apache.org/r/39784/diff/
> 
> 
> Testing
> ---
> 
> Locally: `./pants test.pytest --no-fast src/test/python:: -- -v`
> 
> Also generated a pycharm project via:
>   `./build-support/python/make-pycharm-virtualenv`
> Confirmed library source linking worked as did running unit tests
> via the IDE.
> 
> Also grepped for pants commands in the repo, found `binary` and `setup-py`
> and confirmed these worked.
> 
> 
> Thanks,
> 
> John Sirois
> 
>

Review Request 39784: Upgrade Aurora to pants 0.0.55.

2015-10-29 Thread John Sirois 

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

Review request for Aurora, Joe Smith, Bill Farner, and Zameer Manji.


Bugs: AURORA-1499
https://issues.apache.org/jira/browse/AURORA-1499


Repository: aurora


Description
---

This also brings Aurora up to pex 1.1.0 and switches to the pants
setup script; an equivalent to gradlew.  Of note, the script is checked
in with chmod 555, its not intended to be edited.

Although pants now includes checkstyle built in, conversion to use it is
left for follow-on work.

Also adapt make-pycharm-virtualenv which previously relied on the local
bootstrapping of pants - now hidden away by the pants setup script.

 .pantsversion  |   1 -
 3rdparty/python/requirements.txt   |   2 +-
 BUILD  |  25 -
 BUILD.tools|  19 --
 build-support/jenkins/build.sh |   2 +-
 build-support/pants_requirements.txt   |  30 ---
 build-support/python/make-pycharm-virtualenv   |  11 +-
 build-support/python/update-pants-requirements |  34 -
 pants  | 102 
+++---
 pants.ini  |  63 
++-
 src/main/python/apache/aurora/tools/BUILD  |   2 +-
 src/main/python/apache/thermos/observer/BUILD  |   2 +-
 12 files changed, 114 insertions(+), 179 deletions(-)


Diffs
-

  .pantsversion 78bae5bb6d254d014e35be0b828497f1509d80bd 
  3rdparty/python/requirements.txt 1eeb36dee1a0ebd33999bbb3327338a51cba00d7 
  BUILD 7de0c74b03ba609576867ba96885858c0908f2e9 
  BUILD.tools 75698a5ded7914a4d22ab7ae769d9ed1576531e4 
  build-support/jenkins/build.sh 5606bb157cb117a588f363382d7c8841ae957138 
  build-support/pants_requirements.txt fad8da5ce4c03f25554394bc628d4fbb0fe6cd69 
  build-support/python/make-pycharm-virtualenv 
d7bd41835bcd9a8fe62ea522a177bfa7830a897b 
  build-support/python/update-pants-requirements 
82f7c5136fad52f4bc2a458beb5f34f0cc7f6cec 
  pants 6f3526ef76fc37e3215b0673bcb1725d205a1c95 
  pants.ini dd4ba668586ee5bd1c888f315429e661adb6a480 
  src/main/python/apache/aurora/tools/BUILD 
e5ac75838cbe98bbef4e35f6f300b6a6df5e7de5 
  src/main/python/apache/thermos/observer/BUILD 
d7eedabc0930711530b45ac98a1159e69d1a0c00 

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


Testing
---

Locally: `./pants test.pytest --no-fast src/test/python:: -- -v`

Also generated a pycharm project via:
  `./build-support/python/make-pycharm-virtualenv`
Confirmed library source linking worked as did running unit tests
via the IDE.

Also grepped for pants commands in the repo, found `binary` and `setup-py`
and confirmed these worked.


Thanks,

John Sirois

Re: Review Request 39784: Upgrade Aurora to pants 0.0.55.

2015-10-29 Thread Zameer Manji 

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


John, would you mind filing a ticket to track the followup work to use the 
built in pants checkstyle?

- Zameer Manji


On Oct. 29, 2015, 3:05 p.m., John Sirois wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39784/
> ---
> 
> (Updated Oct. 29, 2015, 3:05 p.m.)
> 
> 
> Review request for Aurora, Joe Smith, Bill Farner, and Zameer Manji.
> 
> 
> Bugs: AURORA-1499
> https://issues.apache.org/jira/browse/AURORA-1499
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> This also brings Aurora up to pex 1.1.0 and switches to the pants
> setup script; an equivalent to gradlew.  Of note, the script is checked
> in with chmod 555, its not intended to be edited.
> 
> Although pants now includes checkstyle built in, conversion to use it is
> left for follow-on work.
> 
> Also adapt make-pycharm-virtualenv which previously relied on the local
> bootstrapping of pants - now hidden away by the pants setup script.
> 
>  .pantsversion  |   1 -
>  3rdparty/python/requirements.txt   |   2 +-
>  BUILD  |  25 -
>  BUILD.tools|  19 --
>  build-support/jenkins/build.sh |   2 +-
>  build-support/pants_requirements.txt   |  30 ---
>  build-support/python/make-pycharm-virtualenv   |  11 +-
>  build-support/python/update-pants-requirements |  34 -
>  pants  | 102 
> +++---
>  pants.ini  |  63 
> ++-
>  src/main/python/apache/aurora/tools/BUILD  |   2 +-
>  src/main/python/apache/thermos/observer/BUILD  |   2 +-
>  12 files changed, 114 insertions(+), 179 deletions(-)
> 
> 
> Diffs
> -
> 
>   .pantsversion 78bae5bb6d254d014e35be0b828497f1509d80bd 
>   3rdparty/python/requirements.txt 1eeb36dee1a0ebd33999bbb3327338a51cba00d7 
>   BUILD 7de0c74b03ba609576867ba96885858c0908f2e9 
>   BUILD.tools 75698a5ded7914a4d22ab7ae769d9ed1576531e4 
>   build-support/jenkins/build.sh 5606bb157cb117a588f363382d7c8841ae957138 
>   build-support/pants_requirements.txt 
> fad8da5ce4c03f25554394bc628d4fbb0fe6cd69 
>   build-support/python/make-pycharm-virtualenv 
> d7bd41835bcd9a8fe62ea522a177bfa7830a897b 
>   build-support/python/update-pants-requirements 
> 82f7c5136fad52f4bc2a458beb5f34f0cc7f6cec 
>   pants 6f3526ef76fc37e3215b0673bcb1725d205a1c95 
>   pants.ini dd4ba668586ee5bd1c888f315429e661adb6a480 
>   src/main/python/apache/aurora/tools/BUILD 
> e5ac75838cbe98bbef4e35f6f300b6a6df5e7de5 
>   src/main/python/apache/thermos/observer/BUILD 
> d7eedabc0930711530b45ac98a1159e69d1a0c00 
> 
> Diff: https://reviews.apache.org/r/39784/diff/
> 
> 
> Testing
> ---
> 
> Locally: `./pants test.pytest --no-fast src/test/python:: -- -v`
> 
> Also generated a pycharm project via:
>   `./build-support/python/make-pycharm-virtualenv`
> Confirmed library source linking worked as did running unit tests
> via the IDE.
> 
> Also grepped for pants commands in the repo, found `binary` and `setup-py`
> and confirmed these worked.
> 
> 
> Thanks,
> 
> John Sirois
> 
>