Re: Review Request 32171: Change update list subcommand to accept a hierarchy.

2015-03-19 Thread Bill Farner
On March 19, 2015, 9:13 p.m., Maxim Khutornenko wrote: src/main/python/apache/aurora/client/cli/update.py, line 250 https://reviews.apache.org/r/32171/diff/4/?file=900349#file900349line250 How about a special blocked group to display only the updates blocked due to lack of

Re: Review Request 32171: Change update list subcommand to accept a hierarchy.

2015-03-19 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32171/#review77124 --- Master (6396410) is red with this patch.

Re: Review Request 32171: Change update list subcommand to accept a hierarchy.

2015-03-19 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32171/ --- (Updated March 20, 2015, 12:29 a.m.) Review request for Aurora, David

Re: Review Request 32171: Change update list subcommand to accept a hierarchy.

2015-03-19 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32171/ --- (Updated March 19, 2015, 10:27 p.m.) Review request for Aurora, David

Re: Review Request 32171: Change update list subcommand to accept a hierarchy.

2015-03-19 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32171/#review77137 --- @ReviewBot retry I think i may have outpaced the bot here, the

Re: Review Request 32171: Change update list subcommand to accept a hierarchy.

2015-03-19 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32171/#review77140 --- @ReviewBot retry Forgive me for generating noise, i'm trying to

Re: Review Request 32171: Change update list subcommand to accept a hierarchy.

2015-03-19 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32171/#review77141 --- Master (465086e) is red with this patch.

Re: Review Request 32171: Change update list subcommand to accept a hierarchy.

2015-03-19 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32171/#review77144 --- Ship it! Master (465086e) is green with this patch.

Re: Review Request 32171: Change update list subcommand to accept a hierarchy.

2015-03-19 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32171/#review77138 --- Master (6396410) is red with this patch.

Re: Review Request 32171: Change update list subcommand to accept a hierarchy.

2015-03-19 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32171/ --- (Updated March 19, 2015, 8:22 p.m.) Review request for Aurora, David

Re: Review Request 32171: Change update list subcommand to accept a hierarchy.

2015-03-19 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32171/#review77102 --- Ship it! src/main/python/apache/aurora/client/cli/update.py

Re: Review Request 32171: Change update list subcommand to accept a hierarchy.

2015-03-19 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32171/#review77101 --- Ship it! Ship It! - Zameer Manji On March 19, 2015, 1:22 p.m.,

Re: Review Request 32171: Change update list subcommand to accept a hierarchy.

2015-03-19 Thread Bill Farner
On March 18, 2015, 12:03 a.m., David McLaughlin wrote: src/test/python/apache/aurora/client/cli/test_supdate.py, line 408 https://reviews.apache.org/r/32171/diff/2/?file=898419#file898419line408 This is more of a general concern, but it would be nice to avoid adding new tests in

Re: Review Request 32171: Change update list subcommand to accept a hierarchy.

2015-03-19 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32171/ --- (Updated March 19, 2015, 9:09 p.m.) Review request for Aurora, David

Re: Review Request 32171: Change update list subcommand to accept a hierarchy.

2015-03-17 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32171/#review76807 --- Master (4b39d26) is red with this patch.

Re: Review Request 32171: Change update list subcommand to accept a hierarchy.

2015-03-17 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32171/#review76844 --- Ship it! Ship It! - Zameer Manji On March 17, 2015, 1:50 p.m.,

Re: Review Request 32171: Change update list subcommand to accept a hierarchy.

2015-03-17 Thread Bill Farner
On March 17, 2015, 11:36 p.m., Aurora ReviewBot wrote: Master (a419863) is red with this patch. ./build-support/jenkins/build.sh Installing

Re: Review Request 32171: Change update list subcommand to accept a hierarchy.

2015-03-17 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32171/ --- (Updated March 17, 2015, 11:30 p.m.) Review request for Aurora, David

Re: Review Request 32171: Change update list subcommand to accept a hierarchy.

2015-03-17 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32171/#review76853 --- src/test/python/apache/aurora/client/cli/test_supdate.py

Re: Review Request 32171: Change update list subcommand to accept a hierarchy.

2015-03-17 Thread Maxim Khutornenko
On March 18, 2015, 12:04 a.m., Kevin Sweeney wrote: src/test/python/apache/aurora/client/cli/test_supdate.py, lines 404-406 https://reviews.apache.org/r/32171/diff/3/?file=898440#file898440line404 How would you feel about including all of the currently available data with a

Re: Review Request 32171: Change update list subcommand to accept a hierarchy.

2015-03-17 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32171/#review76855 --- Master (a419863) is red with this patch.