Re: Review Request 28398: Adding cron check into aurora beta-update start.

2014-11-25 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28398/ --- (Updated Nov. 25, 2014, 6 p.m.) Review request for Aurora, David McLaughlin,

Re: Review Request 28398: Adding cron check into aurora beta-update start.

2014-11-25 Thread Maxim Khutornenko
On Nov. 24, 2014, 7:48 p.m., David McLaughlin wrote: src/main/python/apache/aurora/client/cli/cron.py, line 79 https://reviews.apache.org/r/28398/diff/1/?file=774538#file774538line79 s/existing // Done. - Maxim --- This is

Re: Review Request 28398: Adding cron check into aurora beta-update start.

2014-11-24 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28398/#review62851 --- Ship it! Ship It! - Bill Farner On Nov. 24, 2014, 7:19 p.m.,

Re: Review Request 28398: Adding cron check into aurora beta-update start.

2014-11-24 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28398/#review62853 --- Ship it! src/main/python/apache/aurora/client/cli/cron.py

Re: Review Request 28398: Adding cron check into aurora beta-update start.

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