Re: Review Request 19061: Disable kill of production jobs without force flag

2014-03-12 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19061/ --- (Updated March 12, 2014, 2:12 p.m.) Review request for Aurora, Kevin Sweeney

Review Request 19143: Catch errors thrown by authentication modules.

2014-03-12 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19143/ --- Review request for Aurora, Kevin Sweeney and Brian Wickman. Bugs: aurora-259

Re: Review Request 18979: Add an updated version of the clientv2 doc to apache.

2014-03-12 Thread Mark Chu-Carroll
On March 11, 2014, 3:16 p.m., Tom Galloway wrote: Line 148: Each noun will be separate component - Each noun will be a separate component noun verb syntax; are you wedded to this? aurora get quota and aurora create job are more English (if Tarzan sounding) like than aurora quota

Re: Review Request 18979: Add an updated version of the clientv2 doc to apache.

2014-03-12 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18979/ --- (Updated March 12, 2014, 5:44 p.m.) Review request for Aurora, Dave Lester and

Re: Review Request 19143: Catch errors thrown by authentication modules.

2014-03-12 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19143/#review36996 --- src/main/python/apache/aurora/client/api/scheduler_client.py

Re: Review Request 19159: Add killall.

2014-03-12 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19159/#review37005 --- src/main/python/apache/aurora/client/commands/core.py

Re: Review Request 19159: Add killall.

2014-03-12 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19159/#review37008 --- src/main/python/apache/aurora/client/commands/core.py

Re: Review Request 19159: Add killall.

2014-03-12 Thread Maxim Khutornenko
On March 12, 2014, 10:25 p.m., Mark Chu-Carroll wrote: src/main/python/apache/aurora/client/commands/core.py, line 399 https://reviews.apache.org/r/19159/diff/1/?file=517771#file517771line399 This is deliberate: the kill command doesn't have a force option. So this ensures that

Re: Review Request 19143: Catch errors thrown by authentication modules.

2014-03-12 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19143/ --- (Updated March 12, 2014, 6:59 p.m.) Review request for Aurora, Kevin Sweeney