Review Request 28170: AURORA-933 - python missing license headers

2014-11-18 Thread Jake Farrell
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28170/ --- Review request for Aurora, Kevin Sweeney and Bill Farner. Bugs: AURORA-933

Re: Review Request 28170: AURORA-933 - python missing license headers

2014-11-18 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28170/#review61931 --- Master (c44aa5c) is red with this patch.

Re: Review Request 28170: AURORA-933 - python missing license headers

2014-11-18 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28170/#review61950 --- Ship it! Ship It! - Joshua Cohen On Nov. 18, 2014, 3:46 p.m.,

Re: Review Request 28170: AURORA-933 - python missing license headers

2014-11-18 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28170/#review61952 --- Ship it! Ship It! - Kevin Sweeney On Nov. 18, 2014, 7:46 a.m.,

Re: Review Request 27757: Replace bindStartupAction with Service registration

2014-11-18 Thread Kevin Sweeney
On Nov. 11, 2014, 2:34 p.m., Bill Farner wrote: src/test/java/org/apache/aurora/scheduler/async/KillRetryTest.java, line 103 https://reviews.apache.org/r/27757/diff/1/?file=755113#file755113line103 remove fixed. On Nov. 11, 2014, 2:34 p.m., Bill Farner wrote:

Re: Review Request 27757: Replace bindStartupAction with Service registration

2014-11-18 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27757/#review61999 --- Master (c44aa5c) is red with this patch.

Re: Review Request 27757: Replace bindStartupAction with Service registration

2014-11-18 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27757/#review62001 --- @ReviewBot retry - Kevin Sweeney On Nov. 18, 2014, 12:26 p.m.,

Re: Review Request 27757: Replace bindStartupAction with Service registration

2014-11-18 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27757/#review62005 --- Master (c44aa5c) is red with this patch.

Re: Review Request 27757: Replace bindStartupAction with Service registration

2014-11-18 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27757/#review62007 --- @ReviewBot retry - Kevin Sweeney On Nov. 18, 2014, 12:26 p.m.,

Re: Review Request 27757: Replace bindStartupAction with Service registration

2014-11-18 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27757/#review62010 --- Master (c44aa5c) is red with this patch.

Re: Review Request 27757: Replace bindStartupAction with Service registration

2014-11-18 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27757/#review62011 --- @ReviewBot retry - Kevin Sweeney On Nov. 18, 2014, 12:26 p.m.,

Re: Review Request 27757: Replace bindStartupAction with Service registration

2014-11-18 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27757/#review62012 --- Master (c44aa5c) is red with this patch.

Re: Review Request 27757: Replace bindStartupAction with Service registration

2014-11-18 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27757/#review62015 --- @Reviewbot retry please? - Kevin Sweeney On Nov. 18, 2014, 12:26

Re: Review Request 27757: Replace bindStartupAction with Service registration

2014-11-18 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27757/#review62017 --- Master (c44aa5c) is red with this patch.

Re: Review Request 28026: Add more test coverage to SchedulerThriftInterface.

2014-11-18 Thread Maxim Khutornenko
On Nov. 14, 2014, 1:48 a.m., Zameer Manji wrote: src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java, line 1787 https://reviews.apache.org/r/28026/diff/1/?file=763222#file763222line1787 Can you file a JIRA for this? This is now tracked by

Review Request 28193: Prevent Aurora from creating zero sized Executor tasks.

2014-11-18 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28193/ --- Review request for Aurora, Maxim Khutornenko and Bill Farner. Bugs: AURORA-928

Re: Review Request 28193: Prevent Aurora from creating zero sized Executor tasks.

2014-11-18 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28193/#review62060 --- Ship it! Master (c44aa5c) is green with this patch.

Review Request 28198: Add an example on how to build with Docker.

2014-11-18 Thread Tony Dong
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28198/ --- Review request for Aurora. Repository: aurora Description --- Add an

Re: Review Request 28198: Add an example on how to build with Docker.

2014-11-18 Thread Tony Dong
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28198/ --- (Updated Nov. 19, 2014, 12:35 a.m.) Review request for Aurora and Benjamin

Re: Review Request 28198: Add an example on how to build with Docker.

2014-11-18 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28198/#review62078 --- Master (c44aa5c) is red with this patch.

Re: Review Request 28198: Add an example on how to build with Docker.

2014-11-18 Thread Tony Dong
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28198/#review62079 --- @ReviewBot retry - Tony Dong On Nov. 19, 2014, 12:35 a.m., Tony

Re: Review Request 28198: Add an example on how to build with Docker.

2014-11-18 Thread Tony Dong
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28198/ --- (Updated Nov. 19, 2014, 12:54 a.m.) Review request for Aurora, Benjamin

Re: Review Request 28198: Add an example on how to build with Docker.

2014-11-18 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28198/#review62083 --- Master (c44aa5c) is red with this patch.

Re: Review Request 28140: Drop argparse dependency; it's part of python 2.7

2014-11-18 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28140/#review62091 --- Ship it! Ship It! - David McLaughlin On Nov. 18, 2014, 12:29

Re: Review Request 28198: Add an example on how to build with Docker.

2014-11-18 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28198/#review62098 --- Please add a committer to this so it can be committed. I suggest