Re: Review Request 25124: Added 'net-bench' libprocess benchmarking tool.

2014-08-28 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25124/ --- (Updated Aug. 28, 2014, 10:19 p.m.) Review request for mesos, Benjamin Hindman

Re: Review Request 25084: Switched to single variadic 'join' function

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

Re: Review Request 25105: Explore disk io isolation in cgroups

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

Jenkins build is back to normal : mesos-reviewbot #1428

2014-08-28 Thread Apache Jenkins Server
See

Re: Path forward to C++11.

2014-08-28 Thread Dominic Hamon
Tomorrow, I'm planning to land changes that complete the deprecation cycle for the with-cxx11 configure flag. From this point on, the compiler you use will need to support the subset of C++11 language features checked by our configure script. These are: - static_assert - right angle brackets - dec

Re: Review Request 25169: require some c++11 features for mesos

2014-08-28 Thread Dominic Hamon
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25169/ --- (Updated Aug. 28, 2014, 4:41 p.m.) Review request for mesos, Benjamin Hindman a

Re: Review Request 25168: require some c++11 features for libprocess

2014-08-28 Thread Dominic Hamon
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25168/ --- (Updated Aug. 28, 2014, 4:40 p.m.) Review request for mesos, Benjamin Hindman a

Re: Review Request 25169: require some c++11 features for mesos

2014-08-28 Thread Vinod Kone
> On Aug. 28, 2014, 11:30 p.m., Vinod Kone wrote: > > 3rdparty/libprocess/configure.ac, line 518 > > > > > > Can we print the supported versions here. I think that will be helpful > > for users. nm. i think we are ge

Re: Review Request 25168: require some c++11 features for libprocess

2014-08-28 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25168/#review51847 --- Ship it! 3rdparty/libprocess/configure.ac

Re: Review Request 25169: require some c++11 features for mesos

2014-08-28 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25169/#review51846 --- Ship it! 3rdparty/libprocess/configure.ac

Review Request 25169: require some c++11 features for mesos

2014-08-28 Thread Dominic Hamon
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25169/ --- Review request for mesos, Benjamin Hindman and Vinod Kone. Bugs: MESOS-750

Review Request 25168: require some c++11 features for libprocess

2014-08-28 Thread Dominic Hamon
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25168/ --- Review request for mesos, Benjamin Hindman and Vinod Kone. Bugs: MESOS-750

Build failed in Jenkins: mesos-reviewbot #1427

2014-08-28 Thread Apache Jenkins Server
See -- [...truncated 5588 lines...] rm -f slave/containerizer/.deps/.dirstamp rm -f slave/containerizer/.dirstamp rm -f slave/containerizer/isolators/cgroups/.deps/.dirstamp rm -f slave/containerizer/isol

Re: Review Request 25167: Switch to using compiler intrinsics for unreachable, exit, and abort

2014-08-28 Thread Patrick Reilly
> On Aug. 28, 2014, 10:25 p.m., Dominic Hamon wrote: > > a quick high-level note that it's generally prefered that patches are split > > along stout, libprocess, and mesos boundaries. you can set reviewboard > > patches to depend on other ones to ensure that they land together. [~dhamon] Okay

Re: Build failed in Jenkins: Mesos-Trunk-Ubuntu-Build-Out-Of-Src-Set-JAVA_HOME #2371

2014-08-28 Thread Niklas Nielsen
Thanks for the prompt response Vinod! Niklas On 28 August 2014 15:28, Vinod Kone wrote: > libcurl is not installed on some jenkins slaves. i pinged jake farrell on > a separate thread and waiting to hear back. > > > On Thu, Aug 28, 2014 at 3:26 PM, Niklas Nielsen > wrote: > >> Hmm - don't kn

Re: Build failed in Jenkins: Mesos-Trunk-Ubuntu-Build-Out-Of-Src-Set-JAVA_HOME #2371

2014-08-28 Thread Vinod Kone
libcurl is not installed on some jenkins slaves. i pinged jake farrell on a separate thread and waiting to hear back. On Thu, Aug 28, 2014 at 3:26 PM, Niklas Nielsen wrote: > Hmm - don't know what happened here. We didn't touch libcurl - do you > guys have any insight here? > > Niklas > > > O

Re: Build failed in Jenkins: Mesos-Trunk-Ubuntu-Build-Out-Of-Src-Set-JAVA_HOME #2371

2014-08-28 Thread Niklas Nielsen
Hmm - don't know what happened here. We didn't touch libcurl - do you guys have any insight here? Niklas On 28 August 2014 15:19, Apache Jenkins Server wrote: > See < > https://builds.apache.org/job/Mesos-Trunk-Ubuntu-Build-Out-Of-Src-Set-JAVA_HOME/2371/changes > > > > Changes: > > [niklas]

Re: Review Request 25167: Switch to using compiler intrinsics for unreachable, exit, and abort

2014-08-28 Thread Dominic Hamon
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25167/#review51840 --- a quick high-level note that it's generally prefered that patches ar

Build failed in Jenkins: Mesos-Trunk-Ubuntu-Build-In-Src-Set-JAVA_HOME #2075

2014-08-28 Thread Apache Jenkins Server
See Changes: [niklas] Modifed execute cli to run docker images. -- Started by an SCM change Building remotely on ubuntu3 (Ubuntu ubuntu) in workspace

Review Request 25167: Switch to using compiler intrinsics for unreachable, exit, and abort

2014-08-28 Thread Patrick Reilly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25167/ --- Review request for mesos, Adam B and Benjamin Hindman. Bugs: MESOS-1744 htt

Build failed in Jenkins: Mesos-Ubuntu-distcheck #306

2014-08-28 Thread Apache Jenkins Server
See Changes: [niklas] Modifed execute cli to run docker images. -- Started by an SCM change Building remotely on ubuntu3 (Ubuntu ubuntu) in workspace

Build failed in Jenkins: Mesos-Trunk-Ubuntu-Build-Out-Of-Src-Set-JAVA_HOME #2371

2014-08-28 Thread Apache Jenkins Server
See Changes: [niklas] Modifed execute cli to run docker images. -- [...truncated 260 lines...] 3rdparty/Makefile.am:145: warning: source file '$(STOUT)/tests/s

Re: Review Request 25111: Added the concept of dynamically configurable slave attributes

2014-08-28 Thread Patrick Reilly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25111/ --- (Updated Aug. 28, 2014, 10:10 p.m.) Review request for mesos, Adam B and Benjam

Build failed in Jenkins: Mesos-Trunk-Ubuntu-Build-In-Src-Set-JAVA_HOME #2074

2014-08-28 Thread Apache Jenkins Server
See Changes: [bmahler] Refactored the JSON output streaming. [bmahler] Adjustments for new JSON streaming. [vinodkone] Added logging when master/slaves/frameworks bind to loopback address. [vinodkone]

Build failed in Jenkins: mesos-reviewbot #1426

2014-08-28 Thread Apache Jenkins Server
See -- [...truncated 5588 lines...] rm -f scheduler/*.o rm -f slave/containerizer/isolators/cgroups/.dirstamp rm -f scheduler/*.lo rm -f slave/containerizer/isolators/network/.deps/.dirstamp rm -f slave/*

Re: Review Request 25124: Added 'net-bench' libprocess benchmarking tool.

2014-08-28 Thread Niklas Nielsen
> On Aug. 27, 2014, 4:01 p.m., Dominic Hamon wrote: > > src/net-bench/main.cpp, line 166 > > > > > > it might be worth (for a future version) allowing this to be a random > > string of user-defined length and also tra

Re: Review Request 25124: Added 'net-bench' libprocess benchmarking tool.

2014-08-28 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25124/ --- (Updated Aug. 28, 2014, 11:51 a.m.) Review request for mesos, Benjamin Hindman

Re: Review Request 25111: Added the concept of dynamically configurable slave attributes

2014-08-28 Thread Patrick Reilly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25111/ --- (Updated Aug. 28, 2014, 6:11 p.m.) Review request for mesos, Adam B and Benjami

Re: Review Request 25111: Added the concept of dynamically configurable slave attributes

2014-08-28 Thread Cody Maloney
> On Aug. 28, 2014, 8:37 a.m., Adam B wrote: > > src/common/attributes.cpp, lines 161-164 > > > > > > In your code, this.isSuperset(that) returns false if any of this's > > attributes are not present in that. It there

Re: Review Request 25111: Added the concept of dynamically configurable slave attributes

2014-08-28 Thread Patrick Reilly
> On Aug. 28, 2014, 5:15 p.m., Tobias Weingartner wrote: > > src/slave/slave.cpp, lines 2736-2738 > > > > > > I appreciate the sentiment here. However, both resources and > > attributes (although more attributes) co

Re: Review Request 25111: Added the concept of dynamically configurable slave attributes

2014-08-28 Thread Tobias Weingartner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25111/#review51791 --- src/slave/slave.cpp

Re: Review Request 25111: Added the concept of dynamically configurable slave attributes

2014-08-28 Thread Patrick Reilly
> On Aug. 28, 2014, 8:37 a.m., Adam B wrote: > > Great start! I've got several style nits, and some thoughts about extending > > beyond supersets. > > - How do you plan to test this? Have you tried just restarting a > > checkpointed slave process with new resources/attributes? I think you would

Re: Review Request 25111: Added the concept of dynamically configurable slave attributes

2014-08-28 Thread Patrick Reilly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25111/ --- (Updated Aug. 28, 2014, 4:42 p.m.) Review request for mesos, Adam B and Benjami

Re: Review Request 25111: Added the concept of dynamically configurable slave attributes

2014-08-28 Thread Patrick Reilly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25111/ --- (Updated Aug. 28, 2014, 4:36 p.m.) Review request for mesos, Adam B and Benjami

Re: Review Request 25079: Replaced macro expansion with variadic template

2014-08-28 Thread Patrick Reilly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25079/ --- (Updated Aug. 28, 2014, 4:24 p.m.) Review request for mesos, Adam B and Benjami

Re: Review Request 25111: Added the concept of dynamically configurable slave attributes

2014-08-28 Thread Patrick Reilly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25111/ --- (Updated Aug. 28, 2014, 4:15 p.m.) Review request for mesos, Adam B and Benjami

Build failed in Jenkins: mesos-reviewbot #1425

2014-08-28 Thread Apache Jenkins Server
See -- [...truncated 5589 lines...] rm -f sasl/*.lo rm -f slave/containerizer/mesos/.dirstamp rm -f sched/*.o rm -f sched/*.lo rm -f state/.deps/.dirstamp rm -f scheduler/*.o rm -f state/.dirstamp rm -f s

F21 & Grand unified .spec - CentOS7

2014-08-28 Thread Tim St Clair
Greetings! Because folks have been asking, I've been updating my spec here: http://pkgs.fedoraproject.org/cgit/mesos.git/tree/mesos.spec clone url: git://pkgs.fedoraproject.org/mesos My integration stream is: https://github.com/timothysc/mesos/tree/0.20-integ I intend to push most patches bac

Re: Crashed task is not reaped

2014-08-28 Thread Alex Rukletsov
Hi Brian, thanks for the answer. This sounds reasonable, it would be nice to somehow enforce this "crash-if-fail" behaviour in client executors, but it seems barely possible. Alex On Thu, Aug 28, 2014 at 12:56 AM, Brian Wickman wrote: > A "crashed" thread does not terminate the Python interpr

Re: Review Request 25111: Added the concept of dynamically configurable slave attributes

2014-08-28 Thread Adam B
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25111/#review51753 --- Great start! I've got several style nits, and some thoughts about ex

Re: Review Request 25111: Added the concept of dynamically configurable slave attributes

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