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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17270/#review32992 --- src/main/python/apache/aurora/client/cli/__init__.py

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17270/ --- (Updated Jan. 28, 2014, 10:23 a.m.) Review request for Aurora, Bill Farner and

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17457/ --- Review request for Aurora, Bill Farner and Brian Wickman. Bugs: aurora-80 h

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17457/#review32997 --- src/test/sh/org/apache/aurora/e2e/flask/BUILD

Re: Review Request 17293: s/thermos_observe/thermos_observer/ in README.md

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17293/#review33014 --- Ship it! Ship It! - Brian Wickman On Jan. 24, 2014, 1:28 a.m., D

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17270/#review33035 --- Ship it! cool, drive-by LGTM! src/main/python/apache/aurora/clien

Re: Review Request 17430: Bug fix: Return job count instead of task count on GetRoleSummary() call.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17430/ --- (Updated Jan. 28, 2014, 8:51 p.m.) Review request for Aurora, Kevin Sweeney and

Re: Review Request 17430: Bug fix: Return job count instead of task count on GetRoleSummary() call.

> On Jan. 28, 2014, 3:37 a.m., Bill Farner wrote: > > Can you also attach the JIRA issue please? Done. - Suman --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17430/#review32947 -

Re: Review Request 17430: Bug fix: Return job count instead of task count on GetRoleSummary() call.

> On Jan. 28, 2014, 3:32 a.m., Bill Farner wrote: > > src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java, > > line 384 > > > > > > Thinking out loud, a helper function reduces some duplicat

Re: Review Request 17430: Bug fix: Return job count instead of task count on GetRoleSummary() call.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17430/#review33041 --- src/main/thrift/org/apache/aurora/gen/api.thrift

Review Request 17464: Upgrade RBTools to 0.5.5.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17464/ --- Review request for Aurora and Bill Farner. Repository: aurora Description ---

Re: Review Request 17464: Upgrade RBTools to 0.5.5.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17464/#review33045 --- Ship it! Ship It! - Bill Farner On Jan. 28, 2014, 9:13 p.m., Kev

Re: Review Request 16873: Refactor StateManagerImpl and TaskStateMachine for less code and better readability.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16873/#review33054 --- Kevin — ping? Presumably you can review while i fix up Maxim's non-

Re: Review Request 16873: Refactor StateManagerImpl and TaskStateMachine for less code and better readability.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16873/#review32551 --- Ship it! src/main/java/org/apache/aurora/scheduler/state/StateMana

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17270/#review33055 --- src/main/python/apache/aurora/client/cli/__init__.py

Re: Review Request 17430: Bug fix: Return job count instead of task count on GetRoleSummary() call.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17430/ --- (Updated Jan. 28, 2014, 10:17 p.m.) Review request for Aurora, Kevin Sweeney an

Re: Review Request 17430: Bug fix: Return job count instead of task count on GetRoleSummary() call.

> On Jan. 28, 2014, 9:02 p.m., Bill Farner wrote: > > src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java, > > line 1087 > > > > > > This task needs to be different from task1 in some w

Re: Review Request 16873: Refactor StateManagerImpl and TaskStateMachine for less code and better readability.

> On Jan. 22, 2014, 8:43 p.m., Maxim Khutornenko wrote: > > src/main/java/org/apache/aurora/scheduler/state/SideEffect.java, line 42 > > > > > > Missing comments for public methods in this class? We typically don't bot

Re: Review Request 16873: Refactor StateManagerImpl and TaskStateMachine for less code and better readability.

> On Jan. 28, 2014, 9:50 p.m., Kevin Sweeney wrote: > > src/main/java/org/apache/aurora/scheduler/state/StateManagerImpl.java, > > lines 543-546 > > > > > > This seems like a big enough gotcha to warrant filing a tick

Re: Review Request 16873: Refactor StateManagerImpl and TaskStateMachine for less code and better readability.

> On Jan. 22, 2014, 8:43 p.m., Maxim Khutornenko wrote: > > src/main/java/org/apache/aurora/scheduler/state/SideEffect.java, line 77 > > > > > > Should these be javadoc comments instead? Good call, done. - Bill ---

Re: Review Request 17063: Update tag-release to allow patch level increments.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17063/ --- (Updated Jan. 28, 2014, 11:11 p.m.) Review request for Aurora and Kevin Sweeney

Re: Review Request 17063: Update tag-release to allow patch level increments.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17063/#review33068 --- build-support/release/tag-release

Re: Review Request 17063: Update tag-release to allow patch level increments.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17063/ --- (Updated Jan. 28, 2014, 11:32 p.m.) Review request for Aurora and Kevin Sweeney

Re: Review Request 16873: Refactor StateManagerImpl and TaskStateMachine for less code and better readability.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16873/ --- (Updated Jan. 29, 2014, 12:09 a.m.) Review request for Aurora, Kevin Sweeney an

Re: Review Request 17430: Bug fix: Return job count instead of task count on GetRoleSummary() call.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17430/#review33074 --- Ship it! Ship It! - Bill Farner On Jan. 28, 2014, 10:17 p.m., Su

Re: Review Request 17430: Bug fix: Return job count instead of task count on GetRoleSummary() call.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17430/#review33076 --- Ship it! Ship It! - Kevin Sweeney On Jan. 28, 2014, 2:17 p.m., S

Re: Review Request 17063: Update tag-release to allow patch level increments.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17063/#review33077 --- Ship it! Ship It! - Kevin Sweeney On Jan. 28, 2014, 3:32 p.m., B

Review Request 17475: Upgrade mesos to 0.15.0

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17475/ --- Review request for Aurora, Vinod Kone, Bill Farner, and Brian Wickman. Bugs: AU

Re: Review Request 17475: Upgrade mesos to 0.15.0

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17475/#review33080 --- Ship it! Ship It! - Vinod Kone On Jan. 29, 2014, 12:40 a.m., Kev

Re: Review Request 17475: Upgrade mesos to 0.15.0

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17475/#review33081 --- Ship it! Ship It! - Jake Farrell On Jan. 29, 2014, 12:40 a.m., K