Jenkins build is back to normal : Mesos-Ubuntu-distcheck #366

2014-09-30 Thread Apache Jenkins Server
See https://builds.apache.org/job/Mesos-Ubuntu-distcheck/366/changes

Jenkins build is back to normal : Mesos-Trunk-Ubuntu-Build-In-Src-Set-JAVA_HOME #2135

2014-09-30 Thread Apache Jenkins Server
See https://builds.apache.org/job/Mesos-Trunk-Ubuntu-Build-In-Src-Set-JAVA_HOME/2135/changes

Re: Review Request 25848: Introducing mesos modules.

2014-09-30 Thread Bernd Mathiske
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25848/#review54955 --- include/mesos/module.hpp.in

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 https://reviews.apache.org/r/26150/diff/1/?file=708531#file708531line139 I'd like to start using all the nice C++11 features, but not sure if we want to start introducing them

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

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 https://builds.apache.org/job/Mesos-Trunk-Ubuntu-Build-Out-Of-Src-Disable-Java-Disable-Python-Disable-Webui/2406/changes

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

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 available

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: 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

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

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

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.

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. -

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 https://reviews.apache.org/r/26185/diff/1/?file=709643#file709643line81 s/ip/IP/ here and everywhere else. also, for master: s/on which listening is

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
--- 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 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

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

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 framework

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. -

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

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!

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 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 think

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. -

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

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

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

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 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

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

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 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

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 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

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

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!

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)

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. -

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 https://reviews.apache.org/r/25848/diff/11/?file=708522#file708522line46 Can we use existing boost helpers for this? Something like

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

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.,

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

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 https://builds.apache.org/job/Mesos-Trunk-Ubuntu-Build-Out-Of-Src-Disable-Java-Disable-Python-Disable-Webui/2408/changes Changes: [yujie.jay] Synced container network configurations with host network -- [...truncated 60932 lines...] I1001

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 m...@jaybuff.com wrote: Design

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

Build failed in Jenkins: mesos-reviewbot #1782

2014-09-30 Thread Apache Jenkins Server
See https://builds.apache.org/job/mesos-reviewbot/1782/changes Changes: [yujie.jay] Synced container network configurations with host network -- [...truncated 5582 lines...] sys 7m59.016s + chmod -R +w 3rdparty CHANGELOG Doxyfile LICENSE Makefile

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

2014-09-30 Thread Apache Jenkins Server
See https://builds.apache.org/job/Mesos-Trunk-Ubuntu-Build-In-Src-Set-JAVA_HOME/2137/changes Changes: [yujie.jay] Synced container network configurations with host network -- [...truncated 34121 lines...] I1001 03:47:36.658612 9438 replica.cpp:474]

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,

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

2014-09-30 Thread Apache Jenkins Server
See https://builds.apache.org/job/mesos-reviewbot/1783/