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

2014-09-30 Thread Apache Jenkins Server
See

Re: Review Request 26208: Added a test for the Master <-> Slave reconciliation race.

2014-09-30 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26208/#review55053 --- Patch looks great! Reviews applied: [26198, 26199, 26200, 26201, 26

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

2014-09-30 Thread Apache Jenkins Server
See Changes: [yujie.jay] Synced container network configurations with host network -- [...truncated 34121 lines...] I1001 03:47:36.658612 9438 replica.cpp:474] Rep

Build failed in Jenkins: mesos-reviewbot #1782

2014-09-30 Thread Apache Jenkins Server
See Changes: [yujie.jay] Synced container network configurations with host network -- [...truncated 5582 lines...] sys 7m59.016s + chmod -R +w 3rdparty CHANGELOG Doxyfile LICENSE Makefile Mak

Release of jesos 1.0 - pure java bindings for Apache Mesos

2014-09-30 Thread Henning Schmiedehausen
Hi, I am happy to announce the very first release of the Jesos project which offers pure Java bindings for Apache Mesos. This allows to write frameworks and executors without having to use the native libmesos library. Source code: https://github.com/groupon/jesos/releases/tag/jesos-1.0 or from M

Re: MESOS-1739 Design Document: Allow slave reconfiguration on restart

2014-09-30 Thread Vinod Kone
All our design docs (likely after review phase in google docs) will live here https://cwiki.apache.org/confluence/display/MESOS. We haven't been great about design docs in the past, but we are getting better. On Tue, Sep 30, 2014 at 6:05 PM, Jay Buffington wrote: > Design documents like this ar

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

2014-09-30 Thread Apache Jenkins Server
See Changes: [yujie.jay] Synced container network configurations with host network -- [...truncated 60932 lines...] I1001 01:28:30.2

Re: MESOS-1739 Design Document: Allow slave reconfiguration on restart

2014-09-30 Thread Jay Buffington
Design documents like this are fantastic. Not only do they make it easier to understand justifications for changes, but they help me understand the current state of affairs as well. Keep them coming! Even after the patch is merged this document has a lot of value. Please make sure that the doc l

Re: Review Request 26199: Eliminated redundant resource accounting in the master.

2014-09-30 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26199/#review55037 --- Ship it! Ship It! - Timothy Chen On Sept. 30, 2014, 11:30 p.m.,

Review Request 26210: routing::diagnositics: Changed State from enum to int so they can be bitwise or'ed together.

2014-09-30 Thread Chi Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26210/ --- Review request for mesos and Jie Yu. Bugs: mesos-1808 https://issues.apache

Re: Review Request 25848: Introducing mesos modules.

2014-09-30 Thread Kapil Arya
> On Sept. 29, 2014, 5:57 p.m., Niklas Nielsen wrote: > > include/mesos/module.hpp.in, lines 46-47 > > > > > > Can we use existing boost helpers for this? Something like > > http://www.boost.org/doc/libs/1_56_0/libs/p

Re: Review Request 26197: Synced container network configurations with host network configurations.

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

Re: Review Request 22169: Added External Containerizer documentation.

2014-09-30 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22169/#review55024 --- Fly by review - it is much better than the current (non-existing) do

Re: Review Request 26197: Synced container network configurations with host network configurations.

2014-09-30 Thread Ian Downes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26197/#review55030 --- Ship it! src/slave/containerizer/isolators/network/port_mapping.cp

Review Request 26202: Introduced a version during slave (re-)registration.

2014-09-30 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26202/ --- Review request for mesos and Vinod Kone. Bugs: MESOS-1696 https://issues.ap

Review Request 26200: Added validation for missing ExecutorInfo::framework_id.

2014-09-30 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26200/ --- Review request for mesos and Vinod Kone. Repository: mesos-git Description --

Review Request 26208: Added a test for the Master <-> Slave reconciliation race.

2014-09-30 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26208/ --- Review request for mesos and Vinod Kone. Bugs: MESOS-1696 https://issues.ap

Review Request 26204: Removed the need for Master::readdSlave.

2014-09-30 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26204/ --- Review request for mesos and Vinod Kone. Repository: mesos-git Description --

Review Request 26198: Fixed a log line in the master.

2014-09-30 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26198/ --- Review request for mesos and Vinod Kone. Repository: mesos-git Description --

Review Request 26205: Cleaned up Master::addFramework.

2014-09-30 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26205/ --- Review request for mesos and Vinod Kone. Repository: mesos-git Description --

Review Request 26206: Introduced Master <-> Slave reconciliation.

2014-09-30 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26206/ --- Review request for mesos and Vinod Kone. Bugs: MESOS-1696 https://issues.ap

Review Request 26207: Split out partition and reconciliation tests from FaultTolerace tests.

2014-09-30 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26207/ --- Review request for mesos and Vinod Kone. Bugs: MESOS-1696 https://issues.ap

Review Request 26201: Properly deprecated ReregisterSlaveMessage::slave_id.

2014-09-30 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26201/ --- Review request for mesos and Vinod Kone. Repository: mesos-git Description --

Review Request 26203: Removed redundant logging in the Master.

2014-09-30 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26203/ --- Review request for mesos and Vinod Kone. Repository: mesos-git Description --

Review Request 26199: Eliminated redundant resource accounting in the master.

2014-09-30 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26199/ --- Review request for mesos and Vinod Kone. Repository: mesos-git Description --

Re: Review Request 26197: Synced container network configurations with host network configurations.

2014-09-30 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26197/ --- (Updated Sept. 30, 2014, 11:05 p.m.) Review request for mesos, Ian Downes and V

Re: Review Request 25848: Introducing mesos modules.

2014-09-30 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25848/ --- (Updated Sept. 30, 2014, 7:01 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 26197: Synced container network configurations with host network configurations.

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

Re: Review Request 25785: Add guide to becoming a committer

2014-09-30 Thread Niklas Nielsen
> On Sept. 19, 2014, 5:24 p.m., Adam B wrote: > > I think it's valuable to outline some guidelines for anybody interested in > > becoming a committer, but I don't think we should set too many strict rules > > and regulations around it. Other thoughts below. > > Dominic Hamon wrote: > I thi

Re: Review Request 26197: Synced container network configurations with host network configurations.

2014-09-30 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26197/#review55021 --- src/slave/containerizer/isolators/network/port_mapping.hpp

Re: Review Request 26197: Synced container network configurations with host network configurations.

2014-09-30 Thread Ian Downes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26197/#review55019 --- Ship it! src/slave/containerizer/isolators/network/port_mapping.cp

Review Request 26197: Synced container network configurations with host network configurations.

2014-09-30 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26197/ --- Review request for mesos, Ian Downes and Vinod Kone. Repository: mesos-git De

Re: Review Request 26185: Update documentation/flags regarding new default hostname semantics

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

Re: Review Request 26125: Add webui_url to FrameworkInfo and web UI

2014-09-30 Thread Vinod Kone
> On Sept. 30, 2014, 9:16 p.m., Niklas Nielsen wrote: > > Vinod & Tobi: I can help out landing this if you are interested. I can't > > claim to be an angular expert though. > > > > One high-level comment: how about we write a test so we make sure it works > > in the future too? A simple framew

Re: Review Request 26125: Add webui_url to FrameworkInfo and web UI

2014-09-30 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26125/#review55010 --- Vinod & Tobi: I can help out landing this if you are interested. I c

Re: Review Request 26125: Add webui_url to FrameworkInfo and web UI

2014-09-30 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26125/#review55009 --- Thanks for the patch Tobi. In the future, I recommend you find a she

Re: Review Request 26185: Update documentation/flags regarding new default hostname semantics

2014-09-30 Thread Evelina Dumitrescu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26185/ --- (Updated Sept. 30, 2014, 8:17 p.m.) Review request for mesos and Vinod Kone.

Re: Review Request 26185: Update documentation/flags regarding new default hostname semantics

2014-09-30 Thread Evelina Dumitrescu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26185/ --- (Updated Sept. 30, 2014, 8:15 p.m.) Review request for mesos and Vinod Kone.

Re: Review Request 26185: Update documentation/flags regarding new default hostname semantics

2014-09-30 Thread Evelina Dumitrescu
> On Sept. 30, 2014, 7:27 p.m., Vinod Kone wrote: > > docs/configuration.md, line 81 > > > > > > s/ip/IP/ > > > > here and everywhere else. > > > > also, > > > > for master: > > > > s

Re: Review Request 25525: MESOS-1739: Allow slave reconfiguration on restart

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

Re: Review Request 26185: Update documentation/flags regarding new default hostname semantics

2014-09-30 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26185/#review55000 --- looking good. minor fixes and i'll get this committed. docs/config

Review Request 26185: Update documentation/flags regarding new default hostname semantics

2014-09-30 Thread Evelina Dumitrescu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26185/ --- Review request for mesos and Vinod Kone. Bugs: MESOS-1506 https://issues.ap

Re: Review Request 25525: MESOS-1739: Allow slave reconfiguration on restart

2014-09-30 Thread Cody Maloney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25525/ --- (Updated Sept. 30, 2014, 6:05 p.m.) Review request for mesos, Adam B, Benjamin

Re: scheduler.killExecutor()

2014-09-30 Thread Benjamin Mahler
Why can't the executor just commit suicide if all running tasks are killed? If you're simultaneously launching two tasks for each executor, you'll only see this race if you kill very quickly after launching. Your scheduler is informed when both tasks are running as well, so that could gate the f

Re: Review Request 24535: Added functionality to create SVN based diffs of arbitrary strings.

2014-09-30 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24535/#review54857 --- 3rdparty/libprocess/3rdparty/Makefile.am

Re: Review Request 19180: Fix mesos command parsing help

2014-09-30 Thread Niklas Nielsen
> On May 15, 2014, 4:32 p.m., Niklas Nielsen wrote: > > Hey Chengwei - sorry for the tardy turnaround time on this review request. > > > > To me, it still seems like we are treating the symptoms of the real issue: > > PATH is appended multiple times and the subsequent globbing adds the > > ava

Re: Review Request 25434: Propagate slave shutdown grace period to Executor and CommandExecutor.

2014-09-30 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25434/ --- (Updated Sept. 30, 2014, 2:16 p.m.) Review request for mesos, Benjamin Hindman,

Jenkins build is back to normal : Mesos-Trunk-Ubuntu-Build-Out-Of-Src-Disable-Java-Disable-Python-Disable-Webui #2406

2014-09-30 Thread Apache Jenkins Server
See

Re: scheduler.killExecutor()

2014-09-30 Thread Tom Arnfeld
Thanks Vinod. I missed that issue when searching! I did consider sending a shutdown task, though my worry was that there may be cases where the task might not launch. Perhaps due to resource starvation and/or no offers being received. Presumably it would not be correct to store the original Of

Re: Review Request 26150: Libprocess Benchmark

2014-09-30 Thread Michael Park
> On Sept. 29, 2014, 9:41 p.m., Timothy Chen wrote: > > 3rdparty/libprocess/src/tests/benchmarks.cpp, line 139 > > > > > > I'd like to start using all the nice C++11 features, but not sure if we > > want to start intr