Re: Review Request 49872: Upgrade pants to 1.1.0-rc7.

2016-07-11 Thread John Sirois

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

(Updated July 11, 2016, 6:21 p.m.)


Review request for Aurora, Stephan Erb and Zameer Manji.


Repository: aurora


Description (updated)
---

This helps test the pants release candidate to pave our way to upgrade to
the 1.1.0 release upgrade and it now limits our backends to python-only
for real. This means ivy is no longer bootstrapped to run python tests
for example.

 pants.ini | 7 ++-
 1 file changed, 6 insertions(+), 1 deletion(-)


Diffs
-

  pants.ini dc11d5014c1412cd4a852ddba11a37e178ff02b7 

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


Testing
---

Locally green:
```
$ ./pants test src/{main,test}/python::
$ ./build-support/release/make-python-sdists
$ ./src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh
```


Thanks,

John Sirois



Re: Review Request 49872: Upgrade pants to 1.1.0-rc7.

2016-07-11 Thread John Sirois

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



Noting that 1.1.0-rc7 is actually 1.1.0-rc0 - the RC process has not been as 
fraught as an rc7 might lead you to believe.
Long story short, pants-devel did not do its homework when switching to the new 
version scheme, and in pip's eyes 1.1.0-pre0 == 1.1.0-rc0; so pants needed to 
move past the last pre (6), thus we're at 1.1.0-rc7.

- John Sirois


On July 10, 2016, 4:43 p.m., John Sirois wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/49872/
> ---
> 
> (Updated July 10, 2016, 4:43 p.m.)
> 
> 
> Review request for Aurora, Stephan Erb and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> This helps test our the release candidate to pave our way to upgrade to
> the 1.1.0 release upgrade and it now limits our backends to python-only
> for real. This means ivy is no longer bootstrapped to run python tests
> for example.
> 
>  pants.ini | 7 ++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
> 
> 
> Diffs
> -
> 
>   pants.ini dc11d5014c1412cd4a852ddba11a37e178ff02b7 
> 
> Diff: https://reviews.apache.org/r/49872/diff/
> 
> 
> Testing
> ---
> 
> Locally green:
> ```
> $ ./pants test src/{main,test}/python::
> $ ./build-support/release/make-python-sdists
> $ ./src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh
> ```
> 
> 
> Thanks,
> 
> John Sirois
> 
>



Re: Review Request 49872: Upgrade pants to 1.1.0-rc7.

2016-07-10 Thread Aurora ReviewBot

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


Ship it!




Master (c41420f) is green with this patch.
  ./build-support/jenkins/build.sh

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

- Aurora ReviewBot


On July 10, 2016, 10:43 p.m., John Sirois wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/49872/
> ---
> 
> (Updated July 10, 2016, 10:43 p.m.)
> 
> 
> Review request for Aurora, Stephan Erb and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> This helps test our the release candidate to pave our way to upgrade to
> the 1.1.0 release upgrade and it now limits our backends to python-only
> for real. This means ivy is no longer bootstrapped to run python tests
> for example.
> 
>  pants.ini | 7 ++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
> 
> 
> Diffs
> -
> 
>   pants.ini dc11d5014c1412cd4a852ddba11a37e178ff02b7 
> 
> Diff: https://reviews.apache.org/r/49872/diff/
> 
> 
> Testing
> ---
> 
> Locally green:
> ```
> $ ./pants test src/{main,test}/python::
> $ ./build-support/release/make-python-sdists
> $ ./src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh
> ```
> 
> 
> Thanks,
> 
> John Sirois
> 
>



Re: Review Request 49872: Upgrade pants to 1.1.0-rc7.

2016-07-10 Thread John Sirois

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



NB: I won't wait on Zameer's review since he may no longer be active, but I 
thought he'd be happy to see the resolution of 
https://github.com/pantsbuild/pants/issues/940 at last applied to the Aurora 
repo.

- John Sirois


On July 10, 2016, 4:43 p.m., John Sirois wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/49872/
> ---
> 
> (Updated July 10, 2016, 4:43 p.m.)
> 
> 
> Review request for Aurora, Stephan Erb and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> This helps test our the release candidate to pave our way to upgrade to
> the 1.1.0 release upgrade and it now limits our backends to python-only
> for real. This means ivy is no longer bootstrapped to run python tests
> for example.
> 
>  pants.ini | 7 ++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
> 
> 
> Diffs
> -
> 
>   pants.ini dc11d5014c1412cd4a852ddba11a37e178ff02b7 
> 
> Diff: https://reviews.apache.org/r/49872/diff/
> 
> 
> Testing
> ---
> 
> Locally green:
> ```
> $ ./pants test src/{main,test}/python::
> $ ./build-support/release/make-python-sdists
> $ ./src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh
> ```
> 
> 
> Thanks,
> 
> John Sirois
> 
>



Review Request 49872: Upgrade pants to 1.1.0-rc7.

2016-07-10 Thread John Sirois

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

Review request for Aurora, Stephan Erb and Zameer Manji.


Repository: aurora


Description
---

This helps test our the release candidate to pave our way to upgrade to
the 1.1.0 release upgrade and it now limits our backends to python-only
for real. This means ivy is no longer bootstrapped to run python tests
for example.

 pants.ini | 7 ++-
 1 file changed, 6 insertions(+), 1 deletion(-)


Diffs
-

  pants.ini dc11d5014c1412cd4a852ddba11a37e178ff02b7 

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


Testing
---

Locally green:
```
$ ./pants test src/{main,test}/python::
$ ./build-support/release/make-python-sdists
$ ./src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh
```


Thanks,

John Sirois