Re: Review Request 39797: Adding help message in case kerberos auth fails.

2015-11-03 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39797/#review104943 --- Ship it! Master (f41938d) is green with this patch. ./build-supp

Re: Review Request 39797: Adding help message in case kerberos auth fails.

2015-11-03 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39797/ --- (Updated Nov. 3, 2015, 5:53 p.m.) Review request for Aurora, Joshua Cohen and B

Re: Review Request 39797: Adding help message in case kerberos auth fails.

2015-11-03 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39797/ --- (Updated Nov. 3, 2015, 5:53 p.m.) Review request for Aurora, Joshua Cohen and B

Re: Review Request 39797: Adding help message in case kerberos auth fails.

2015-11-03 Thread Maxim Khutornenko
> On Nov. 3, 2015, 1:20 a.m., Joshua Cohen wrote: > > src/main/python/apache/aurora/client/cli/client.py, line 49 > > > > > > Do you think it's worth adding a test case for this functionality? The `logging.getLogger(

Re: Review Request 39797: Adding help message in case kerberos auth fails.

2015-11-02 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39797/#review104866 --- Ship it! Ship It! - Bill Farner On Nov. 2, 2015, 5:13 p.m., Max

Re: Review Request 39797: Adding help message in case kerberos auth fails.

2015-11-02 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39797/#review104844 --- Ship it! Master (f41938d) is green with this patch. ./build-supp

Re: Review Request 39797: Adding help message in case kerberos auth fails.

2015-11-02 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39797/#review104841 --- Ship it! LGTM! src/main/python/apache/aurora/client/cli/client.p

Re: Review Request 39797: Adding help message in case kerberos auth fails.

2015-11-02 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39797/ --- (Updated Nov. 3, 2015, 1:13 a.m.) Review request for Aurora, Joshua Cohen and B

Re: Review Request 39797: Adding help message in case kerberos auth fails.

2015-11-02 Thread Maxim Khutornenko
> On Oct. 30, 2015, 2:31 p.m., Joshua Cohen wrote: > > src/main/python/apache/aurora/client/api/scheduler_client.py, lines 305-306 > > > > > > It seems like this friendly error message is going to get buried at the

Re: Review Request 39797: Adding help message in case kerberos auth fails.

2015-11-02 Thread Joshua Cohen
> On Oct. 30, 2015, 2:31 p.m., Joshua Cohen wrote: > > src/main/python/apache/aurora/client/api/scheduler_client.py, lines 305-306 > > > > > > It seems like this friendly error message is going to get buried at the

Re: Review Request 39797: Adding help message in case kerberos auth fails.

2015-11-02 Thread Bill Farner
> On Oct. 30, 2015, 7:31 a.m., Joshua Cohen wrote: > > src/main/python/apache/aurora/client/api/scheduler_client.py, lines 305-306 > > > > > > It seems like this friendly error message is going to get buried at the

Re: Review Request 39797: Adding help message in case kerberos auth fails.

2015-11-02 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39797/#review104755 --- Ping Bill, Joshua. - Maxim Khutornenko On Oct. 30, 2015, 2:24 a.

Re: Review Request 39797: Adding help message in case kerberos auth fails.

2015-10-30 Thread Maxim Khutornenko
> On Oct. 30, 2015, 2:31 p.m., Joshua Cohen wrote: > > src/main/python/apache/aurora/client/api/scheduler_client.py, lines 305-306 > > > > > > It seems like this friendly error message is going to get buried at the

Re: Review Request 39797: Adding help message in case kerberos auth fails.

2015-10-30 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39797/#review104566 --- src/main/python/apache/aurora/client/api/scheduler_client.py (line

Re: Review Request 39797: Adding help message in case kerberos auth fails.

2015-10-29 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39797/#review104517 --- Ship it! Master (1b1c70c) is green with this patch. ./build-supp

Review Request 39797: Adding help message in case kerberos auth fails.

2015-10-29 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39797/ --- Review request for Aurora, Joshua Cohen and Bill Farner. Bugs: AURORA-1364