Re: Review Request 26688: Fix errors in help rendering:

2014-10-15 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26688/#review56811 --- Ship it! Ship It! - Joshua Cohen On Oct. 15, 2014, 5:57 p.m., Ma

Re: Review Request 26688: Fix errors in help rendering:

2014-10-15 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26688/#review56750 --- Ship it! Ship It! - Zameer Manji On Oct. 15, 2014, 10:57 a.m., M

Re: Review Request 26688: Fix errors in help rendering:

2014-10-15 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26688/ --- (Updated Oct. 15, 2014, 1:57 p.m.) Review request for Aurora, Joshua Cohen and

Re: Review Request 26688: Fix errors in help rendering:

2014-10-15 Thread Zameer Manji
> On Oct. 15, 2014, 10:35 a.m., Mark Chu-Carroll wrote: > > src/main/python/apache/aurora/client/cli/standalone_client.py, line 61 > > > > > > It's the standard python loglevels. The number is basically any > > positiv

Re: Review Request 26688: Fix errors in help rendering:

2014-10-15 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26688/#review56737 --- src/main/python/apache/aurora/client/cli/standalone_client.py

Re: Review Request 26688: Fix errors in help rendering:

2014-10-15 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26688/#review56733 --- src/main/python/apache/aurora/client/cli/standalone_client.py

Re: Review Request 26688: Fix errors in help rendering:

2014-10-15 Thread Mark Chu-Carroll
> On Oct. 14, 2014, 12:46 p.m., Joshua Cohen wrote: > > src/test/python/apache/aurora/client/cli/test_help.py, line 75 > > > > > > Are option names guaranteed to be unique? If not this test could > > potentially pass i

Re: Review Request 26688: Fix errors in help rendering:

2014-10-15 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26688/ --- (Updated Oct. 15, 2014, 12:46 p.m.) Review request for Aurora, Joshua Cohen and

Re: Review Request 26688: Fix errors in help rendering:

2014-10-15 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26688/ --- (Updated Oct. 15, 2014, 12:41 p.m.) Review request for Aurora, Joshua Cohen and

Re: Review Request 26688: Fix errors in help rendering:

2014-10-14 Thread Mark Chu-Carroll
> On Oct. 14, 2014, 1 p.m., Mark Chu-Carroll wrote: > > src/test/python/apache/aurora/client/cli/test_help.py, line 75 > > > > > > Yes, they are guaranteed to be unique. The argparse framework that this > > is built on

Re: Review Request 26688: Fix errors in help rendering:

2014-10-14 Thread Joshua Cohen
> On Oct. 14, 2014, 5 p.m., Mark Chu-Carroll wrote: > > src/test/python/apache/aurora/client/cli/test_help.py, line 75 > > > > > > Yes, they are guaranteed to be unique. The argparse framework that this > > is built on

Re: Review Request 26688: Fix errors in help rendering:

2014-10-14 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26688/#review56545 --- src/test/python/apache/aurora/client/cli/test_help.py

Re: Review Request 26688: Fix errors in help rendering:

2014-10-14 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26688/#review56541 --- This looks good to me, just a couple of questions about the tests. S

Review Request 26688: Fix errors in help rendering:

2014-10-14 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26688/ --- Review request for Aurora, David McLaughlin and Joshua Cohen. Bugs: aurora-831