Re: Review Request 25523: Add Docker pull to docker abstraction

2014-09-11 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25523/ --- (Updated Sept. 11, 2014, 6:44 a.m.) Review request for mesos and Benjamin

Re: Review Request 25403: Override entrypoint when shell enabled in Docker

2014-09-11 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25403/#review53014 --- Bad patch! Reviews applied: [25403] Failed command: ./bootstrap

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

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

Build failed in Jenkins: mesos-reviewbot #1519

2014-09-11 Thread Apache Jenkins Server
See https://builds.apache.org/job/mesos-reviewbot/1519/ -- [...truncated 5391 lines...] rm -f scheduler/*.o rm -f slave/containerizer/.deps/.dirstamp rm -rf docker/.libs docker/_libs rm -f scheduler/*.lo rm -f slave/containerizer/.dirstamp rm -rf exec/.libs

Re: Review Request 25035: Fix for MESOS-1688

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

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

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

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

2014-09-11 Thread Apache Jenkins Server
See https://builds.apache.org/job/Mesos-Trunk-Ubuntu-Build-Out-Of-Src-Disable-Java-Disable-Python-Disable-Webui/2371/changes

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

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

Re: Review Request 25523: Add Docker pull to docker abstraction

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

ANN: Mesos bindings for Haskell

2014-09-11 Thread Ian Duncan
I’ve written pretty comprehensive Haskell bindings for Mesos that wrap the C++ API: https://github.com/iand675/hs-mesos http://hackage.haskell.org/package/hs-mesos In the process of putting the package together, I ended up with some questions: * It seems like there are portions of

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

2014-09-11 Thread Timothy St. Clair
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25434/#review53038 --- src/exec/exec.cpp https://reviews.apache.org/r/25434/#comment92392

Build failed in Jenkins: mesos-reviewbot #1522

2014-09-11 Thread Apache Jenkins Server
See https://builds.apache.org/job/mesos-reviewbot/1522/ -- [...truncated 5545 lines...] rm -f slave/containerizer/.deps/.dirstamp rm -f slave/containerizer/.dirstamp rm -f slave/containerizer/isolators/cgroups/.deps/.dirstamp rm -f slave/*.lo rm -f

Introduction

2014-09-11 Thread Sanjeet B
Hi, I am Sanjeet Bhatti from India. I have done my Post-Graduation in MCA (Masters of Computer Applications). I have some intermediate experience in web technologies namely HTML, DHTML, XML, CSS, Javascript and PHP along with Joomla and WordPress CMSs. And I have command over programming

Re: Completed tasks remains in TASK_RUNNING when framework is disconnected

2014-09-11 Thread Vinod Kone
The semantics of these changes would have an impact on the upcoming task reconciliation. @BenM: Can you chime in here on how this fits into the task reconciliation work that you've been leading? On Wed, Sep 10, 2014 at 6:12 PM, Adam Bordelon a...@mesosphere.io wrote: I agree with Niklas that

Re: Review Request 25270: Enable bridge network in Mesos

2014-09-11 Thread Timothy Chen
On Sept. 9, 2014, 6:48 p.m., Benjamin Hindman wrote: src/tests/docker_containerizer_tests.cpp, line 243 https://reviews.apache.org/r/25270/diff/4/?file=680521#file680521line243 Ultimately it would be great to get this under the 'mesos' registry so other folks can easily make

Re: Review Request 25270: Enable bridge network in Mesos

2014-09-11 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25270/ --- (Updated Sept. 11, 2014, 5:48 p.m.) Review request for mesos, Benjamin

Re: Mesos webcast

2014-09-11 Thread Kamil Domański
Hello Vinod, thank you for doing this. How sweet is the irony of the fact, that this webinar cannot be attended from a Linux machine. If anyone could later post a link to a recording of this webcast, then the magnitude of my gratitude could only be eclipsed by my disappointment with these

Re: Review Request 24178: Basic filesystem isolator for Linux.

2014-09-11 Thread Ian Downes
On Aug. 19, 2014, 5:21 p.m., Jie Yu wrote: src/slave/containerizer/isolators/filesystem.cpp, lines 44-46 https://reviews.apache.org/r/24178/diff/1/?file=647967#file647967line44 Check for kernel version? Check for root permission? For example, when bind mount is introduced? bind

Review Request 25549: Basic filesystem isolator for Linux.

2014-09-11 Thread Ian Downes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25549/ --- Review request for mesos, Ben Mahler, Jie Yu, and Vinod Kone. Bugs: MESOS-1586

Re: mesos authorization

2014-09-11 Thread Kevin Sweeney
On Tue, Sep 9, 2014 at 5:04 PM, Jay Buffington m...@jaybuff.com wrote: Hi All, I've had a few conversations offline with mesos contributors regarding authorization and authentication. I'd like to solicit the larger community for comments. I want to create groups of people and allow those

Re: Completed tasks remains in TASK_RUNNING when framework is disconnected

2014-09-11 Thread Adam Bordelon
Definitely relevant. If the master could be trusted to persist all the task status updates, then they could be queued up at the master instead of the slave once the master has acknowledged its receipt. Then the master could have the most up-to-date task state and can recover the resources as soon

Build failed in Jenkins: mesos-reviewbot #1523

2014-09-11 Thread Apache Jenkins Server
See https://builds.apache.org/job/mesos-reviewbot/1523/ -- [...truncated 5545 lines...] rm -f sched/*.o rm -f slave/containerizer/isolators/network/.dirstamp rm -f slave/containerizer/mesos/.deps/.dirstamp rm -f sched/*.lo rm -f

Re: Review Request 25549: Basic filesystem isolator for Linux.

2014-09-11 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25549/#review53090 --- src/slave/containerizer/isolators/filesystem/shared.cpp

Re: [jira] [Resolved] (MESOS-1740) Bad error message when docker containerizer isn't enabled

2014-09-11 Thread Benjamin Mahler
Can you include the commit log for this change in the ticket? Makes it easier to do cherry picking, especially since this one is going in 0.20.1. On Thu, Sep 11, 2014 at 12:01 PM, Timothy Chen (JIRA) j...@apache.org wrote: [

Review Request 25551: Add standard versioning to shared libmesos.so

2014-09-11 Thread Timothy St. Clair
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25551/ --- Review request for mesos, Jie Yu and Vinod Kone. Bugs: MESOS-1764

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

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

Re: Review Request 25551: Add standard versioning to shared libmesos.so

2014-09-11 Thread Timothy St. Clair
On Sept. 11, 2014, 8:15 p.m., Vinod Kone wrote: src/Makefile.am, line 565 https://reviews.apache.org/r/25551/diff/1/?file=686371#file686371line565 What is the benefit of doing this now? I think we need to have a discussion around how to version the library before we go down this

Re: Review Request 25270: Enable bridge network in Mesos

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

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

2014-09-11 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25525/#review53100 --- OK. I went through parts of this review but I have a bigger

Re: Review Request 25549: Basic filesystem isolator for Linux.

2014-09-11 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25549/#review53125 --- Bad patch! Reviews applied: [24177, 25549] Failed command: git

Re: Review Request 25549: Basic filesystem isolator for Linux.

2014-09-11 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25549/#review53126 --- src/common/parse.hpp

Re: Review Request 25549: Basic filesystem isolator for Linux.

2014-09-11 Thread Vinod Kone
On Sept. 11, 2014, 7:13 p.m., Timothy Chen wrote: src/slave/containerizer/isolators/filesystem/shared.cpp, line 125 https://reviews.apache.org/r/25549/diff/1/?file=686147#file686147line125 how about when the host path doesn't start with /? +1 - Vinod

Re: Review Request 25261: Check for variadic template and default/deleted function support

2014-09-11 Thread Cody Maloney
On Sept. 2, 2014, 7:50 p.m., Michael Park wrote: Just something to note here, there's a bug in earlier GCC versions where the access control of `= default`ed functions aren't enforced correctly. e.g. ``` class Foo { private: Foo() = default; }; class Bar

0.20.1 Release Manager

2014-09-11 Thread Benjamin Mahler
Do we have a volunteer to be the 0.20.1 release manager? Looks like a number of tickets are being added at this point for 0.20.1: https://issues.apache.org/jira/issues/?jql=project%20%3D%20MESOS%20AND%20%22Target%20Version%2Fs%22%20%3D%200.20.1

Build failed in Jenkins: mesos-reviewbot #1527

2014-09-11 Thread Apache Jenkins Server
See https://builds.apache.org/job/mesos-reviewbot/1527/ -- [...truncated 5390 lines...] rm -f tests/common/.dirstamp rm -f logging/*.o rm -f usage/.deps/.dirstamp rm -f logging/*.lo rm -f usage/.dirstamp rm -f master/*.o rm -f zookeeper/.deps/.dirstamp rm

Review Request 25565: Added a removeExecutor helper in the Master.

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

Review Request 25568: Added tests for terminal unacknowledged tasks in the Master.

2014-09-11 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25568/ --- Review request for mesos, Niklas Nielsen and Vinod Kone. Bugs: MESOS-1410

Review Request 25566: Minor cleanups to the Master code.

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

Review Request 25567: Hold on to unacknowledged terminal tasks in the Master.

2014-09-11 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25567/ --- Review request for mesos, Niklas Nielsen and Vinod Kone. Bugs: MESOS-1410

Review Request 25569: Only perform docker validation once for tests

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

Build failed in Jenkins: mesos-reviewbot #1528

2014-09-11 Thread Apache Jenkins Server
See https://builds.apache.org/job/mesos-reviewbot/1528/ -- [...truncated 5698 lines...] rm -f config.status config.cache config.log configure.lineno config.status.lineno rm -f Makefile make[1]: Leaving directory

Re: 0.20.1 Release Manager

2014-09-11 Thread Jie Yu
I am a little swamped recently, anyone volunteer for the release manager? - Jie On Thu, Sep 11, 2014 at 6:04 PM, Benjamin Mahler benjamin.mah...@gmail.com wrote: Do we have a volunteer to be the 0.20.1 release manager? Looks like a number of tickets are being added at this point for 0.20.1: