Re: Review Request 29866: Upgrade pants to 0.0.27

2015-01-16 Thread Brian Wickman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29866/ --- (Updated Jan. 16, 2015, 7:16 p.m.) Review request for Aurora, Kevin Sweeney

Re: Review Request 29866: Upgrade pants to 0.0.27

2015-01-16 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29866/#review68469 --- Ship it! Master (1346c4f) is green with this patch.

Re: Review Request 29866: Upgrade pants to 0.0.27

2015-01-16 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29866/#review68494 --- Ship it! Thanks for doing this! Make sure you send a note to dev@

Re: Review Request 29866: Upgrade pants to 0.0.27

2015-01-14 Thread Brian Wickman
On Jan. 14, 2015, 12:49 a.m., Kevin Sweeney wrote: src/test/python/apache/aurora/executor/common/test_resource_manager_integration.py, line 21 https://reviews.apache.org/r/29866/diff/3/?file=820203#file820203line21 same, this doesn't appear to be related to a pants upgrade

Re: Review Request 29866: Upgrade pants to 0.0.27

2015-01-14 Thread Kevin Sweeney
On Jan. 13, 2015, 4:49 p.m., Kevin Sweeney wrote: src/test/python/apache/aurora/executor/common/test_resource_manager_integration.py, line 21 https://reviews.apache.org/r/29866/diff/3/?file=820203#file820203line21 same, this doesn't appear to be related to a pants upgrade

Re: Review Request 29866: Upgrade pants to 0.0.27

2015-01-14 Thread Joshua Cohen
On Jan. 14, 2015, 12:49 a.m., Kevin Sweeney wrote: src/test/python/apache/aurora/executor/common/test_resource_manager_integration.py, line 21 https://reviews.apache.org/r/29866/diff/3/?file=820203#file820203line21 same, this doesn't appear to be related to a pants upgrade

Re: Review Request 29866: Upgrade pants to 0.0.27

2015-01-14 Thread Brian Wickman
On Jan. 14, 2015, 12:49 a.m., Kevin Sweeney wrote: src/test/python/apache/aurora/executor/common/test_resource_manager_integration.py, line 21 https://reviews.apache.org/r/29866/diff/3/?file=820203#file820203line21 same, this doesn't appear to be related to a pants upgrade

Re: Review Request 29866: Upgrade pants to 0.0.27

2015-01-13 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29866/#review67988 --- src/main/python/apache/aurora/executor/common/resource_manager.py

Re: Review Request 29866: Upgrade pants to 0.0.27

2015-01-13 Thread Joe Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29866/#review67987 --- Ship it! Rad pants

Re: Review Request 29866: Upgrade pants to 0.0.27

2015-01-13 Thread Brian Wickman
On Jan. 14, 2015, 12:49 a.m., Kevin Sweeney wrote: src/test/python/apache/aurora/executor/common/test_resource_manager_integration.py, line 21 https://reviews.apache.org/r/29866/diff/3/?file=820203#file820203line21 same, this doesn't appear to be related to a pants upgrade this

Re: Review Request 29866: Upgrade pants to 0.0.27

2015-01-13 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29866/#review67996 --- Ship it! Master (a350982) is green with this patch.

Re: Review Request 29866: Upgrade pants to 0.0.27

2015-01-13 Thread Brian Wickman
On Jan. 14, 2015, 12:07 a.m., Maxim Khutornenko wrote: Please, update docs/developing-aurora-client.md as well. derp not sure how i missed that. - Brian --- This is an automatically generated e-mail. To reply, visit:

Re: Review Request 29866: Upgrade pants to 0.0.27

2015-01-13 Thread Brian Wickman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29866/#review67965 --- @ReviewBot retry - Brian Wickman On Jan. 13, 2015, 9:51 p.m.,

Re: Review Request 29866: Upgrade pants to 0.0.27

2015-01-13 Thread Brian Wickman
On Jan. 13, 2015, 11:01 p.m., Brian Wickman wrote: @ReviewBot retry n/m this is reproducible. just not caught by end to end test. - Brian --- This is an automatically generated e-mail. To reply, visit:

Review Request 29866: Upgrade pants to 0.0.27

2015-01-13 Thread Brian Wickman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29866/ --- Review request for Aurora, Kevin Sweeney and Maxim Khutornenko. Repository:

Re: Review Request 29866: Upgrade pants to 0.0.27

2015-01-13 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29866/#review67957 --- Master (a350982) is red with this patch.

Re: Review Request 29866: Upgrade pants to 0.0.27

2015-01-13 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29866/#review67967 --- build-support/python/make-pycharm-virtualenv

Re: Review Request 29866: Upgrade pants to 0.0.27

2015-01-13 Thread Brian Wickman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29866/ --- (Updated Jan. 13, 2015, 11:20 p.m.) Review request for Aurora, Kevin Sweeney

Re: Review Request 29866: Upgrade pants to 0.0.27

2015-01-13 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29866/#review67977 --- Ship it! Ship It! - Joshua Cohen On Jan. 13, 2015, 11:20 p.m.,

Re: Review Request 29866: Upgrade pants to 0.0.27

2015-01-13 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29866/#review67979 --- Ship it! Please, update docs/developing-aurora-client.md as well.

Re: Review Request 29866: Upgrade pants to 0.0.27

2015-01-13 Thread Brian Wickman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29866/ --- (Updated Jan. 14, 2015, 12:15 a.m.) Review request for Aurora, Kevin Sweeney

Re: Review Request 29866: Upgrade pants to 0.0.27

2015-01-13 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29866/#review67982 --- Ship it! Master (a350982) is green with this patch.