Re: Review Request 20146: AURORA-265: Add missing tooltips for task states in dictionary.js

2014-04-09 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20146/#review39901 ---

Re: Review Request 20146: AURORA-265: Add missing tooltips for task states in dictionary.js

2014-04-09 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20146/#review39907 --- Thanks for the diff Dan. As part of the new UI roll out

Re: Review Request 19833: Migrated Job page to angular JS

2014-04-09 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19833/#review39908 --- pinging Kevin. - Suman Karumuri On April 4, 2014, 11:52 p.m.,

Re: Review Request 19833: Migrated Job page to angular JS

2014-04-09 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19833/#review39909 --- pinging Kevin. - Suman Karumuri On April 4, 2014, 11:52 p.m.,

Re: Review Request 20146: AURORA-265: Add missing tooltips for task states in dictionary.js

2014-04-09 Thread Suman Karumuri
On April 9, 2014, 5:35 p.m., Suman Karumuri wrote: Thanks for the diff Dan. As part of the new UI roll out https://reviews.apache.org/r/19833/, I have removed this file from the server and moved this logic into the client side. I can add these changes as part of that diff or you can

Re: Review Request 20174: Put executors in default PATH in Vagrant example.

2014-04-09 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20174/#review39928 --- Executors aren't meant to be invoked directly - they're invoked by

Re: Review Request 20174: Put executors in default PATH in Vagrant example.

2014-04-09 Thread Johan Liseborn
On April 9, 2014, 10:23 p.m., Kevin Sweeney wrote: Executors aren't meant to be invoked directly - they're invoked by the mesos-slave with some special environment variables. So they're more akin to libexec files. The observer is standalone though and can be moved. Aha, I see. I will

Re: Review Request 20174: Put executors in default PATH in Vagrant example.

2014-04-09 Thread Johan Liseborn
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20174/ --- (Updated April 9, 2014, 10:41 p.m.) Review request for Aurora. Changes

Re: Review Request 20174: Put executors in default PATH in Vagrant example.

2014-04-09 Thread Kevin Sweeney
On April 9, 2014, 1:23 p.m., Kevin Sweeney wrote: Executors aren't meant to be invoked directly - they're invoked by the mesos-slave with some special environment variables. So they're more akin to libexec files. The observer is standalone though and can be moved. Johan Liseborn

Re: Review Request 20174: Link thermos_observer in default path in Vagrant example.

2014-04-09 Thread Johan Liseborn
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20174/ --- (Updated April 9, 2014, 10:44 p.m.) Review request for Aurora. Changes

Re: Review Request 20174: Link thermos_observer in default path in Vagrant example.

2014-04-09 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20174/#review39931 --- Ship it! Ship It! - Kevin Sweeney On April 9, 2014, 1:44 p.m.,

Re: Review Request 20174: Link thermos_observer in default path in Vagrant example.

2014-04-09 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20174/#review39946 --- This is also fixed in https://reviews.apache.org/r/20067/, which is

Re: Review Request 20067: Add a test to validate that the tutorial instructions work.

2014-04-09 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20067/#review39947 --- Ship it! Ship It! - Kevin Sweeney On April 7, 2014, 8:20 a.m.,

Re: Review Request 20069: Move gradlew back to project root.

2014-04-09 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20069/#review39957 --- Ship it! Ship It! - Bill Farner On April 6, 2014, 9:23 p.m.,

Review Request 20188: Update end-to-end test for single-node cluster

2014-04-09 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20188/ --- Review request for Aurora, Jake Farrell and Bill Farner. Bugs: AURORA-315