Re: Review Request 15653: Adds loadavg() convenience method to stout

2013-12-09 Thread Niklas Nielsen
On Dec. 9, 2013, 9:42 p.m., Ben Mahler wrote: 3rdparty/libprocess/3rdparty/stout/include/stout/os.hpp, line 849 https://reviews.apache.org/r/15653/diff/4/?file=395223#file395223line849 Should this be ErrnoError? getloadavg is most likely just a wrapper for sysctl (which sets

Re: Review Request 16111: Fixed the zookeeper client wrappers to use the negotiated session timeout value as their local reconnect timeout.

2013-12-09 Thread Benjamin Hindman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16111/#review30064 --- src/zookeeper/group.hpp

Re: Review Request 16136: Fixed the python tests in the presence of muliple eggs.

2013-12-09 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16136/ --- (Updated Dec. 10, 2013, 1:04 a.m.) Review request for mesos, Benjamin Hindman

Re: Review Request 16136: Fixed the python tests in the presence of muliple eggs.

2013-12-09 Thread Ben Mahler
On Dec. 9, 2013, 10:22 p.m., Benjamin Hindman wrote: src/examples/python/test-executor.in, line 16 https://reviews.apache.org/r/16136/diff/1/?file=395813#file395813line16 Can we set PYTHON_VERSION in our automake variables and then use that instead of EGG_PYTHON_VERSION? Turns

Re: Review Request 15653: Adds loadavg() convenience method to stout

2013-12-09 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15653/ --- (Updated Dec. 10, 2013, 1:27 a.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 14669: launchTasks on list of offers

2013-12-09 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14669/#review30069 --- Hey Nik, a few points below: 1. My only significant comment in

Re: Review Request 16147: Containerizer

2013-12-09 Thread Ian Downes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16147/ --- (Updated Dec. 10, 2013, 2:21 a.m.) Review request for mesos, Benjamin Hindman,

Review Request 16149: Containerizer - launchers

2013-12-09 Thread Ian Downes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16149/ --- Review request for mesos, Benjamin Hindman, Ben Mahler, Chi Zhang, Niklas

Review Request 16150: Containerizer - isolators

2013-12-09 Thread Ian Downes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16150/ --- Review request for mesos, Benjamin Hindman, Ben Mahler, Chi Zhang, Niklas

[jira] [Commented] (MESOS-831) script-without-shebang

2013-12-09 Thread Timothy St. Clair (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-831?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13843877#comment-13843877 ] Timothy St. Clair commented on MESOS-831: - https://reviews.apache.org/r/15764/

[jira] [Commented] (MESOS-750) Require compilers that support c++11

2013-12-09 Thread Till Toenshoff (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-750?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13843913#comment-13843913 ] Till Toenshoff commented on MESOS-750: -- During phase 2-4, should we add something that

Re: Review Request 16136: Fixed the python tests in the presence of muliple eggs.

2013-12-09 Thread Benjamin Hindman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16136/#review30078 --- Ship it! src/examples/python/test-executor.in

[jira] [Commented] (MESOS-750) Require compilers that support c++11

2013-12-09 Thread Benjamin Hindman (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-750?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13843989#comment-13843989 ] Benjamin Hindman commented on MESOS-750: SGTM. Require compilers that support

[jira] [Updated] (MESOS-749) Add support for multiple offers in launchTasks

2013-12-09 Thread Niklas Quarfot Nielsen (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-749?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Niklas Quarfot Nielsen updated MESOS-749: - Fix Version/s: 0.17.0 Add support for multiple offers in launchTasks

Re: Review Request 15802: Catch-up Replicated Log 3: Added log recovery support.

2013-12-09 Thread Benjamin Hindman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15802/#review29966 --- src/log/log.hpp https://reviews.apache.org/r/15802/#comment57464

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

2013-12-09 Thread Apache Jenkins Server
See https://builds.apache.org/job/Mesos-Trunk-Ubuntu-Build-Out-Of-Src-Set-JAVA_HOME/1792/ -- [...truncated 262 lines...] Started by an SCM change Started by an SCM change Started by an SCM change Started by an SCM change Started by an SCM change Started by

Re: Review Request 16111: Fixed the zookeeper client wrappers to use the negotiated session timeout value as their local reconnect timeout.

2013-12-09 Thread Jiang Yan Xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16111/ --- (Updated Dec. 9, 2013, 6:51 p.m.) Review request for mesos, Benjamin Hindman,

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

2013-12-09 Thread Benjamin Mahler
Looks like a bug during tear down: python_framework_test.sh terminated with signal 'Segmentation fault' Will see if I can reproduce locally. On Fri, Dec 6, 2013 at 11:11 PM, Apache Jenkins Server jenk...@builds.apache.org wrote: See

[jira] [Commented] (MESOS-672) Web UI redirection does not work for hosts whose ip addresses are not publicly accessible

2013-12-09 Thread Benjamin Hindman (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13843468#comment-13843468 ] Benjamin Hindman commented on MESOS-672: Hey [~xujyan], given the code today doing

mesos and ec2

2013-12-09 Thread Benjamin Hindman
If you don't use Mesos on EC2 then you can likely stop reading this email. When using Mesos on EC2 you might notice that the web UI seems broken. The two things we know about are: (1) You can't use '/master/redirect' on a master because it attempts to send you to the private IP hostname of the

[jira] [Updated] (MESOS-672) Web UI redirection does not work for hosts whose ip addresses are not publicly accessible

2013-12-09 Thread Jie Yu (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-672?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jie Yu updated MESOS-672: - Assignee: Yan Xu (was: Jie Yu) Web UI redirection does not work for hosts whose ip addresses are not publicly

Re: mesos and ec2

2013-12-09 Thread Tim St Clair
inline - Original Message - From: Benjamin Hindman benjamin.hind...@gmail.com To: dev dev@mesos.apache.org, u...@mesos.apache.org Sent: Monday, December 9, 2013 2:06:36 PM Subject: mesos and ec2 If you don't use Mesos on EC2 then you can likely stop reading this email. When

Re: mesos and ec2

2013-12-09 Thread Tim St Clair
full ref: http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/AESDG-chapter-instancedata.html - Original Message - From: Tim St Clair tstcl...@redhat.com To: u...@mesos.apache.org Cc: dev dev@mesos.apache.org Sent: Monday, December 9, 2013 2:46:46 PM Subject: Re: mesos and ec2

Re: Review Request 15764: Minor fix for script-without-shebang MESOS-831

2013-12-09 Thread Ben Mahler
On Nov. 22, 2013, 9:51 p.m., Ben Mahler wrote: Interesting, these files are not meant to be scripts (they are python libraries), which tool reported the script-without-shebang issue? (Can you kill the trailing whitespace in the Apache header?) Timothy St. Clair wrote: rpmlint

Re: mesos and ec2

2013-12-09 Thread Brenden Matthews
My preferred solution is to: 1. Remove the requirement that the forward and reverse DNS match ( https://github.com/airbnb/mesos/commit/6953571758d287d159a289e62c1fd0d685b5e8e3), and 2. Write the leading master's hostname to ZooKeeper for discovery (

Review Request 16136: Fixed the python tests in the presence of multiple eggs.

2013-12-09 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16136/ --- Review request for mesos, Benjamin Hindman and Vinod Kone. Repository:

Re: Review Request 15764: Minor fix for script-without-shebang MESOS-831

2013-12-09 Thread Timothy St. Clair
On Nov. 22, 2013, 9:51 p.m., Ben Mahler wrote: Interesting, these files are not meant to be scripts (they are python libraries), which tool reported the script-without-shebang issue? (Can you kill the trailing whitespace in the Apache header?) Timothy St. Clair wrote: rpmlint

Re: Review Request 16136: Fixed the python tests in the presence of multiple eggs.

2013-12-09 Thread Benjamin Hindman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16136/#review30053 --- src/examples/python/test-executor.in

Re: Review Request 15764: Minor fix for script-without-shebang MESOS-831

2013-12-09 Thread Benjamin Mahler
Sorry for the trouble! I could submit this and clean up the whitespace for you, but I think this will be a good exercise in using reviewboard. :) Updating will, as far as I know, result in a new revision in the diff page, but there is only one diff revision:

Re: Review Request 15764: Minor fix for script-without-shebang MESOS-831

2013-12-09 Thread Timothy St. Clair
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15764/ --- (Updated Dec. 9, 2013, 10:40 p.m.) Review request for mesos. Changes ---

Re: Review Request 16111: Fixed the zookeeper client wrappers to use the negotiated session timeout value as their local reconnect timeout.

2013-12-09 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16111/#review30048 --- src/state/zookeeper.cpp

Re: Review Request 15764: Minor fix for script-without-shebang MESOS-831

2013-12-09 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15764/#review30055 --- Ship it! Ship It! - Ben Mahler On Dec. 9, 2013, 10:40 p.m.,

Re: Review Request 15764: Minor fix for script-without-shebang MESOS-831

2013-12-09 Thread Ben Mahler
On Dec. 9, 2013, 10:47 p.m., Ben Mahler wrote: Ship It! Thanks Tim, this is now committed, can you mark this review as submitted and update the JIRA ticket? - Ben --- This is an automatically generated e-mail. To reply, visit: