Re: Review Request 23912: Fix MESOS-947: Slave should properly handle a killTask() that arrives between runTask() and _runTask()

2014-10-30 Thread Bernd Mathiske
> On Oct. 23, 2014, 12:54 p.m., Ben Mahler wrote: > > src/slave/slave.cpp, lines 1270-1273 > > > > > > Hey Bernd and Vinod, coverity just reported this, looks like it's now > > dead code! > > > > Can you fo

Re: Build failed in Jenkins: Mesos-Trunk-Ubuntu-Build-Out-Of-Src-Disable-Java-Disable-Python-Disable-Webui #2508

2014-10-30 Thread Yan Xu
ca received write > request for position 4 > I1030 05:55:05.082358 24486 hierarchical_allocator_process.hpp:679] > Performed allocation for slave 20141030-055504-3142697795-40429-24459-S0 in > 632705ns > I1030 05:55:05.082669 24483 leveldb.cpp:343] Persisting action (16 bytes) > to leve

Re: Review Request 27175: Added support for handling 'rc' tag to Version.

2014-10-30 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27175/ --- (Updated Oct. 30, 2014, 4:32 p.m.) Review request for mesos and Ben Mahler. C

Re: Review Request 27175: Added support for handling 'rc' tag to Version.

2014-10-30 Thread Kapil Arya
> On Oct. 28, 2014, 11:17 p.m., Cody Maloney wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/version.hpp, line 73 > > > > > > If we go with semantic versioning: > > > > Rule 10: "Precedence for two

Unable to install Mesos on Ubuntu 14.04. Error during 'make'

2014-10-30 Thread Sweta Rani
Tried all the suggestions and tips given in JIRA and different google forums but still the error remains same. Making all in . make[1]: Entering directory `/home/proxima/mesos/build' make[1]: Nothing to be done for `all-am'. make[1]: Leaving directory `/home/proxima/mesos/build' Making all in 3rdp

Re: Review Request 27175: Added support for handling 'rc' tag to Version.

2014-10-30 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27175/#review59259 --- Patch looks great! Reviews applied: [27175] All tests passed. - M

Review Request 27395: Fixed test namespace setter compilation exclusion on OSX.

2014-10-30 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27395/ --- Review request for mesos and Ian Downes. Bugs: MESOS-2015 https://issues.ap

Re: Unable to install Mesos on Ubuntu 14.04. Error during 'make'

2014-10-30 Thread Dominic Hamon
Let's tackle the easy stuff: did you run bootstrap and configure as per the getting started doc? On Oct 30, 2014 2:04 PM, "Sweta Rani" wrote: > Tried all the suggestions and tips given in JIRA and different google > forums but still the error remains same. > > Making all in . > make[1]: Entering

Re: Review Request 27395: Fixed test namespace setter compilation exclusion on OSX.

2014-10-30 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27395/#review59267 --- Patch looks great! Reviews applied: [27395] All tests passed. - M

Re: Review Request 26859: Integrated CRAM-MD5 Authenticator module.

2014-10-30 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26859/ --- (Updated Oct. 30, 2014, 10:18 p.m.) Review request for mesos, Adam B, Benjamin

Re: Review Request 26859: Integrated CRAM-MD5 Authenticator module.

2014-10-30 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26859/ --- (Updated Oct. 30, 2014, 11:58 p.m.) Review request for mesos, Adam B, Benjamin

Re: Review Request 26860: Integrated CRAM-MD5 Authenticator module into tests.

2014-10-30 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26860/ --- (Updated Oct. 31, 2014, 12:02 a.m.) Review request for mesos, Adam B, Benjamin

Re: Review Request 26857: Added CRAM-MD5 Test Authenticator module.

2014-10-30 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26857/#review59285 --- Ship it! Ship It! - Kapil Arya On Oct. 29, 2014, 10:25 p.m., Til

Re: Review Request 27122: Reorganized authentication implementation locations.

2014-10-30 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27122/#review59286 --- Ship it! Ship It! - Kapil Arya On Oct. 29, 2014, 9:13 p.m., Till

Re: Review Request 26859: Integrated CRAM-MD5 Authenticator module.

2014-10-30 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26859/#review59108 --- src/master/master.cpp

Re: Review Request 26859: Integrated CRAM-MD5 Authenticator module.

2014-10-30 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26859/ --- (Updated Oct. 31, 2014, 1:37 a.m.) Review request for mesos, Adam B, Benjamin H

Re: Review Request 26859: Integrated CRAM-MD5 Authenticator module.

2014-10-30 Thread Till Toenshoff
> On Oct. 31, 2014, 12:43 a.m., Vinod Kone wrote: > > src/master/master.cpp, line 124 > > > > > > i wish you did this cleanup as a dependent patch. oh well. That indeed would have been better - sry for the inconvenien

Re: Review Request 26859: Integrated CRAM-MD5 Authenticator module.

2014-10-30 Thread Adam B
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26859/#review59270 --- src/master/flags.hpp

Review Request 27413: Rename the Node class in Endpoint and replace the ip and port pairs from the UPID class and process namespace with this concept.

2014-10-30 Thread Evelina Dumitrescu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27413/ --- Review request for mesos, Dominic Hamon and switched to 'mcypark'. Bugs: MESOS-

Re: Review Request 27413: Rename the Node class in Endpoint and replace the ip and port pairs from the UPID class and process namespace with this concept.

2014-10-30 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27413/#review59301 --- Bad patch! Reviews applied: [27413] Failed command: ./support/appl

Re: Review Request 26857: Added CRAM-MD5 Test Authenticator module.

2014-10-30 Thread Adam B
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26857/#review59305 --- Ship it! Ship It! - Adam B On Oct. 29, 2014, 7:25 p.m., Till Toe

Re: Review Request 26859: Integrated CRAM-MD5 Authenticator module.

2014-10-30 Thread Adam B
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26859/#review59308 --- Ship it! A few minor re-wordings, then we can commit it! src/mast

Re: Review Request 26860: Integrated CRAM-MD5 Authenticator module into tests.

2014-10-30 Thread Adam B
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26860/#review59310 --- Ship it! src/tests/flags.hpp

Review Request 27419: Fix typos in user facing documentation

2014-10-30 Thread Joe Gordon
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27419/ --- Review request for mesos. Repository: mesos-git Description --- Fix a fe