Re: Review Request 17507: Fix indentation in tutorial

2014-01-30 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17507/#review33240 --- Pushing this trivial change to master TBR. - Kevin Sweeney On

Re: Review Request 17457: Initial version of an end-to-end test using vagrant

2014-01-30 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17457/ --- (Updated Jan. 30, 2014, 3:49 p.m.) Review request for Aurora, Bill Farner and

Re: Review Request 17270: Bridge framework for client v2.

2014-01-30 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17270/ --- (Updated Jan. 30, 2014, 4:03 p.m.) Review request for Aurora, Bill Farner and

Re: Review Request 17457: Initial version of an end-to-end test using vagrant

2014-01-30 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17457/ --- (Updated Jan. 30, 2014, 4:09 p.m.) Review request for Aurora, Bill Farner and

Re: Review Request 17559: requirement should be mesos==0.15.0-rc4

2014-01-30 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17559/#review33254 --- This is fixed in https://reviews.apache.org/r/17475/ - any reason

Re: Review Request 17559: requirement should be mesos==0.15.0-rc4

2014-01-30 Thread Brian Wickman
On Jan. 30, 2014, 9:31 p.m., Kevin Sweeney wrote: This is fixed in https://reviews.apache.org/r/17475/ - any reason not to go forward with that one? oh, I hadn't seen it because it's not on master. I'll discard this and ship that. - Brian

Re: Review Request 17475: Upgrade mesos to 0.15.0

2014-01-30 Thread Brian Wickman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17475/#review33257 --- Ship it! Ship It! - Brian Wickman On Jan. 29, 2014, 12:40 a.m.,

Review Request 17562: Added cluster name to page title.

2014-01-30 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17562/ --- Review request for Aurora, Kevin Sweeney and Bill Farner. Bugs: AURORA-120

Re: Review Request 17562: Added cluster name to page title.

2014-01-30 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17562/#review33266 ---

Re: Review Request 17562: Added cluster name to page title.

2014-01-30 Thread Suman Karumuri
On Jan. 31, 2014, 12:56 a.m., Kevin Sweeney wrote: Spoke to Bill before making these changes. We decided to use getInfo API and deprecate getAPIVersion. The getInfo will return with the clusterName and APIVersion for now. In future it may include other attributes and hence the flexible

Re: Review Request 17578: When trying to schedule a task, only query once for active tasks in the job.

2014-01-30 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17578/#review33281 --- Ship it!

Re: Review Request 17578: When trying to schedule a task, only query once for active tasks in the job.

2014-01-30 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17578/#review33285 --- src/main/java/org/apache/aurora/scheduler/async/TaskScheduler.java

Re: Review Request 17578: When trying to schedule a task, only query once for active tasks in the job.

2014-01-30 Thread Bill Farner
On Jan. 31, 2014, 2:55 a.m., Kevin Sweeney wrote: src/main/java/org/apache/aurora/scheduler/async/TaskScheduler.java, lines 185-187 https://reviews.apache.org/r/17578/diff/2/?file=457399#file457399line185 I think this is actually somewhat a regression - in the past it was

Re: Review Request 17579: AURORA-136: SchedulerLifecycle ACTIVE state is unused

2014-01-30 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17579/#review33303 --- Ship it! Thanks for the contribution! This is now on master as