Re: Review Request 27760: Revised authenticator interface to allow for two fold implementations.

2015-03-22 Thread Till Toenshoff
On March 11, 2015, 9:42 p.m., Vinod Kone wrote: src/master/master.cpp, lines 3858-3861 https://reviews.apache.org/r/27760/diff/23/?file=890575#file890575line3858 Send a FrameworkError message (instead of AuthenticationError) here to avoid retries? Till Toenshoff wrote:

Re: [VOTE] Release Apache Mesos 0.22.0 (rc4)

2015-03-22 Thread Niklas Nielsen
Did you file a JIRA for the broken test? Niklas On 21 March 2015 at 13:29, Benjamin Mahler benjamin.mah...@gmail.com wrote: On CentOS w/ gcc 4.8, make check passes but sudo make check fails for me: [ RUN ] PerfTest.ROOT_SampleInit ../../src/tests/perf_tests.cpp:147: Failure Expected:

Re: Review Request 30774: Fetcher Cache

2015-03-22 Thread Bernd Mathiske
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30774/ --- (Updated March 21, 2015, 11:54 p.m.) Review request for mesos, Adam B,

Re: Review Request 31444: Support chrooting in MesosContainerizer launch helper.

2015-03-22 Thread Jay Buffington
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31444/#review77361 --- src/slave/containerizer/mesos/launch.cpp

Re: Review Request 30952: Adding scheduler validations to master

2015-03-22 Thread Isabel Jimenez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30952/ --- (Updated March 23, 2015, 2:34 a.m.) Review request for mesos, Jie Yu, Joris

Re: Review Request 30952: Adding scheduler validations to master

2015-03-22 Thread Isabel Jimenez
On March 7, 2015, 12:49 a.m., Vinod Kone wrote: src/master/master.cpp, lines 2542-2545 https://reviews.apache.org/r/30952/diff/4/?file=882660#file882660line2542 Is there a test for this? If not, can you write one? Also add a TODO in the old driver to remove this code.

Re: Review Request 32384: Adding perf check to configure

2015-03-22 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32384/#review77366 --- Bad patch! Reviews applied: [32384] Failed command: ./configure

Review Request 32384: Adding perf check to configure

2015-03-22 Thread Isabel Jimenez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32384/ --- Review request for mesos and Cody Maloney. Bugs: MESOS-2166

Re: Review Request 30952: Adding scheduler validations to master

2015-03-22 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30952/#review77365 --- Patch looks great! Reviews applied: [30952] All tests passed. -