Review Request 29971: Fix path to stylesheet in slaves and utilization templates.

2015-01-16 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29971/ --- Review request for Aurora, David McLaughlin and Bill Farner. Bugs: AURORA-1019

Re: Review Request 29901: De-flakify resource_manager_integration test

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

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 29971: Fix path to stylesheet in slaves and utilization templates.

2015-01-16 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29971/#review68457 --- Ship it! Ship It! - David McLaughlin On Jan. 16, 2015, 7:08

Re: Review Request 29971: Fix path to stylesheet in slaves and utilization templates.

2015-01-16 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29971/#review68461 --- Master (1346c4f) is red with this patch.

Re: Review Request 29901: De-flakify resource_manager_integration test

2015-01-16 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29901/#review68470 --- This patch does not apply cleanly on master (1346c4f), do you need

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 29971: Fix path to stylesheet in slaves and utilization templates.

2015-01-16 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29971/#review68477 --- @ReviewBot retry - Joshua Cohen On Jan. 16, 2015, 7:08 p.m.,

Review Request 29984: Minor clean up to mock usage in resource manager integration tests.

2015-01-16 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29984/ --- Review request for Aurora, Kevin Sweeney and Brian Wickman. Repository: aurora

Re: Review Request 29942: Updates so client will run in PyCharm.

2015-01-16 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29942/#review68486 --- Ship it! This is now on origin/master: ``` $ git log -1

Re: Review Request 29971: Fix path to stylesheet in slaves and utilization templates.

2015-01-16 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29971/#review68489 --- Ship it! Ship It! - Bill Farner On Jan. 16, 2015, 7:08 p.m.,

Re: Review Request 29971: Fix path to stylesheet in slaves and utilization templates.

2015-01-16 Thread Bill Farner
On Jan. 16, 2015, 10:29 p.m., Bill Farner wrote: Ship It! This is now on master: ``` $ git log -1 origin/master commit da74e282befa2dcd9e2084120c31a60bcfc5ba33 Author: Joshua Cohen jco...@twopensource.com Date: Fri Jan 16 14:29:31 2015 -0800 Fix path to stylesheet in slaves and

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@

Review Request 30000: Bump the minimum allocation for thermos to 256MB of RAM.

2015-01-16 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3/ --- Review request for Aurora and Bill Farner. Bugs: AURORA-1021

Re: Review Request 29464: Add option to override local scheduler address published into ZooKeeper

2015-01-16 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29464/#review68523 --- Ship it! Sorry for the delay on this - i actually thought it was

Re: Review Request 30000: Bump the minimum allocation for thermos to 256MB of RAM.

2015-01-16 Thread Zameer Manji
On Jan. 16, 2015, 5:14 p.m., Bill Farner wrote: LGTM, but i really think we should reject unreasonably-small tasks rather than silently modifying them. Please file a ticket to address that way. This has the bonus of reducing complexity in our code.