Review Request 21142: Avoid sending tasks in SANDBOX_DELETED state to the GC executor.

2014-05-11 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21142/ --- Review request for Aurora and Maxim Khutornenko. Bugs: AURORA-395 https://i

Re: Review Request 21247: Add config grouping visualisation to job page

2014-05-11 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21247/#review42617 --- Ship it! src/main/resources/org/apache/aurora/scheduler/http/ui/gr

Re: Review Request 21223: Bugfix: make "restart" command generate a correct error message in some cases.

2014-05-11 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21223/#review42590 --- Ship it! src/test/python/apache/aurora/client/api/test_restarter.p

Re: Review Request 20394: Add return code to task/process pages

2014-05-11 Thread David McLaughlin
> On May 6, 2014, 6:50 p.m., Brian Wickman wrote: > > david -- should i submit this? i just realized that you likely don't have > > commit privileges, so i think this is up to me to submit. Yup, that would be cool. I'm not a committer yet. - David -

Re: Review Request 21142: Avoid sending tasks in SANDBOX_DELETED state to the GC executor.

2014-05-11 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21142/ --- (Updated May 7, 2014, 1:24 a.m.) Review request for Aurora and Maxim Khutornenk

Review Request 21232: Add a strict mode for commands that specify instances.

2014-05-11 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21232/ --- Review request for Aurora, David McLaughlin and Suman Karumuri. Bugs: aurora-40

Re: Review Request 21136: Add python import order wrapper/commit hook

2014-05-11 Thread Joe Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21136/#review42421 --- Ship it! fwiw, lgtm as-is - Joe Smith On May 6, 2014, 4:10 p.m.,

Review Request 21297: Adding UpdateConfig value checks.

2014-05-11 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21297/ --- Review request for Aurora and Brian Wickman. Bugs: AURORA-404 https://issue

Re: Review Request 21232: Add a strict mode for commands that specify instances.

2014-05-11 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21232/#review42616 --- Ship it! Ship It! - Suman Karumuri On May 8, 2014, 6:54 p.m., Ma

Re: Review Request 21232: Add a strict mode for commands that specify instances.

2014-05-11 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21232/#review42615 --- - Suman Karumuri On May 8, 2014, 6:54 p.m., Mark Chu-Carroll wrote

Re: Review Request 21144: Always show resource consumption table on Job page.

2014-05-11 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21144/#review42583 --- Ship it! Ship It! - David McLaughlin On May 7, 2014, 1:39 a.m.,

Re: Review Request 19788: Add a clientv2 version of the e2e test.

2014-05-11 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19788/ --- (Updated May 7, 2014, 3:47 p.m.) Review request for Aurora, David McLaughlin an

Review Request 21235: Now showing SANDBOX_DELETED message in the header.

2014-05-11 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21235/ --- Review request for Aurora. Bugs: AURORA-402 https://issues.apache.org/jira/

Re: Review Request 21235: Now showing SANDBOX_DELETED message in the header.

2014-05-11 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21235/ --- (Updated May 8, 2014, 9:14 p.m.) Review request for Aurora and Maxim Khutornenk

Review Request 21205: Bugfix: restart doesn't notify user about invalid max_total_failures option.

2014-05-11 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21205/ --- Review request for Aurora, David McLaughlin and Suman Karumuri. Bugs: aurora-39