Re: Review Request 31022: Fixing aurora beta-update status command.

2015-02-13 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31022/#review72449 --- I'm over capacity on reviews at the moment, mind passing this one

Re: Review Request 31022: Fixing aurora beta-update status command.

2015-02-13 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31022/ --- (Updated Feb. 13, 2015, 10:15 p.m.) Review request for Aurora, Bill Farner and

Re: Review Request 31022: Fixing aurora beta-update status command.

2015-02-13 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31022/#review72443 --- src/test/python/apache/aurora/client/cli/test_supdate.py

Review Request 31022: Fixing aurora beta-update status command.

2015-02-13 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31022/ --- Review request for Aurora, Bill Farner and Zameer Manji. Repository: aurora

Re: Review Request 31022: Fixing aurora beta-update status command.

2015-02-13 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31022/ --- (Updated Feb. 13, 2015, 9:54 p.m.) Review request for Aurora, Bill Farner and

Re: Review Request 31022: Fixing aurora beta-update status command.

2015-02-13 Thread Maxim Khutornenko
On Feb. 13, 2015, 10:06 p.m., Zameer Manji wrote: src/test/python/apache/aurora/client/cli/test_supdate.py, line 120 https://reviews.apache.org/r/31022/diff/1/?file=863675#file863675line120 Shouldn't 'test' be self.TEST_CLUSTER or similar? Cluster parameter is not actually used in

Re: Review Request 31022: Fixing aurora beta-update status command.

2015-02-13 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31022/#review72445 --- Ship it! Ship It! - Zameer Manji On Feb. 13, 2015, 2:15 p.m.,

Re: Review Request 31022: Fixing aurora beta-update status command.

2015-02-13 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31022/#review72462 --- Ship it! Master (9da3ef0) is green with this patch.

Re: Review Request 31022: Fixing aurora beta-update status command.

2015-02-13 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31022/#review72459 --- Ship it! Ship It! - Joshua Cohen On Feb. 13, 2015, 11:27 p.m.,