Re: Review Request 17823: Add a utility to list missing shipits.

2014-02-14 Thread Brian Wickman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17823/#review34508 --- Ship it! Ship It! - Brian Wickman On Feb. 13, 2014, 10:59 p.m.,

Re: Review Request 17877: AURORA-200: set_quota should die if given invalid arguments

2014-02-14 Thread Dan Norris
On Feb. 11, 2014, 4:33 p.m., Mark Chu-Carroll wrote: src/main/python/apache/aurora/client/commands/admin.py, line 174 https://reviews.apache.org/r/17877/diff/1/?file=481150#file481150line174 Die expects a message string, not an exception object. die('Invalid unit specification')

Re: Review Request 17752: Add task noun, supporting run and ssh verbs.

2014-02-14 Thread Brian Wickman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17752/#review34515 --- src/main/python/apache/aurora/client/cli/options.py

Re: Review Request 17948: Implement help message generation for the noun/verb framework.

2014-02-14 Thread Brian Wickman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17948/#review34509 --- src/main/python/apache/aurora/client/cli/__init__.py

Re: Review Request 18042: Adding sla get_task_up_count command.

2014-02-14 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18042/ --- (Updated Feb. 14, 2014, 6 p.m.) Review request for Aurora, Mark Chu-Carroll

Re: Review Request 18141: Prepare and launch GC executor tasks asynchronously.

2014-02-14 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18141/#review34542 --- src/main/java/org/apache/aurora/scheduler/MesosSchedulerImpl.java

Re: Review Request 18149: Cron predictor conformance test.

2014-02-14 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18149/#review34555 --- Ship it! This is great, thanks!! - Bill Farner On Feb. 14,

Re: Review Request 18157: Compute task host attribute aggregates once when scheduling tasks.

2014-02-14 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18157/#review34561 --- Ship it!