Re: Review Request 39854: Retry scheduler connection after unknown failures

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

Review Request 39854: Retry scheduler connection after unknown failures

2015-11-02 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39854/ --- Review request for Aurora, Maxim Khutornenko and Bill Farner. Bugs:

Re: Review Request 39784: Upgrade Aurora to pants 0.0.55.

2015-11-02 Thread John Sirois
> On Oct. 29, 2015, 4:15 p.m., Zameer Manji wrote: > > John, would you mind filing a ticket to track the followup work to use the > > built in pants checkstyle? > > Joe Smith wrote: > https://issues.apache.org/jira/browse/AURORA-1532 Thanks Joe. - John

Re: Review Request 39822: upgrade psutil to 3.2.2

2015-11-02 Thread Dmitriy Shirchenko
> On Nov. 1, 2015, 4:59 p.m., Bill Farner wrote: > > Ship It! sweet, if there are no other objections, can someone land this for me, please? - Dmitriy --- This is an automatically generated e-mail. To reply, visit:

Re: Review Request 39854: Retry scheduler connection after unknown failures

2015-11-02 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39854/#review104754 --- Ship it! Ship It! - Maxim Khutornenko On Nov. 2, 2015, 1:09

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 39784: Upgrade Aurora to pants 0.0.55.

2015-11-02 Thread John Sirois
> On Oct. 29, 2015, 4:57 p.m., Aurora ReviewBot wrote: > > Master (bcb4774) is red with this patch. > > ./build-support/jenkins/build.sh > > > > 22:56:59 00:00 [unpack-jars] > > 22:56:59 00:00 [unpack-jars] > > 22:56:59 00:00 [deferred-sources] > > 22:56:59 00:00

Re: Review Request 39784: Upgrade Aurora to pants 0.0.55.

2015-11-02 Thread John Sirois
> On Oct. 29, 2015, 4:24 p.m., Zameer Manji wrote: > > Would you also mind updating the commit message to point to the pants > > changelog? Will do. > On Oct. 29, 2015, 4:24 p.m., Zameer Manji wrote: > > BUILD.tools, line 1 > >

Re: Review Request 39822: upgrade psutil to 3.2.2

2015-11-02 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39822/#review104787 --- Ship it! Ship It! - Zameer Manji On Oct. 30, 2015, 1:14 p.m.,

Re: Review Request 39822: upgrade psutil to 3.2.2

2015-11-02 Thread Zameer Manji
> On Nov. 1, 2015, 8:59 a.m., Bill Farner wrote: > > Ship It! > > Dmitriy Shirchenko wrote: > sweet, if there are no other objections, can someone land this for me, > please? Maxim is the last reviewer, so once he ships this he will land it. - Zameer

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