Re: Review Request 32219: Change default htb leaf qdisc to fq_codel

> On March 21, 2015, 6:26 a.m., Jie Yu wrote: > > Also, when fq_codel starts to be available? Do we need to add a check during startup, or you think it's not necessary? - Jie --- This is an automatically generated e-mail. To reply, vis

Re: Review Request 32219: Change default htb leaf qdisc to fq_codel

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32219/#review77335 --- Ship it! src/slave/containerizer/isolators/network/port_mapping.cp

Re: GPGPU SLA support

How do I subscribe to the 'modules' mailing list? On Fri, Mar 20, 2015 at 10:16 PM, Anshuman Goswami < goswami.anshu...@gmail.com> wrote: > Thanks Niklas for the pointer...will get back as I dig in more > > On Fri, Mar 20, 2015 at 6:20 PM, Niklas Nielsen > wrote: > >> Sounds like you are on

Re: GPGPU SLA support

Thanks Niklas for the pointer...will get back as I dig in more On Fri, Mar 20, 2015 at 6:20 PM, Niklas Nielsen wrote: > Sounds like you are on the right track; let us know if you have any > problems developing/running modules. > We created a modules mailing list too where we can go into more

Re: Review Request 31444: Support chrooting in MesosContainerizer launch helper.

> On March 21, 2015, 12:21 a.m., Chi Zhang wrote: > > src/slave/containerizer/mesos/launch.cpp, line 296 > > > > > > From my testing, you only need to make-slave on the newRoot for the > > pivot_root to work. > >

Re: Review Request 31444: Support chrooting in MesosContainerizer launch helper.

> On March 21, 2015, 12:21 a.m., Chi Zhang wrote: > > src/slave/containerizer/mesos/launch.cpp, line 296 > > > > > > From my testing, you only need to make-slave on the newRoot for the > > pivot_root to work. > >

Re: Review Request 31444: Support chrooting in MesosContainerizer launch helper.

> On March 18, 2015, 7:09 p.m., Jie Yu wrote: > > src/slave/containerizer/mesos/launch.cpp, lines 332-340 > > > > > > Do you still need this? Are you worried about the case that newRoot is readonly? If that's the case

Re: Review Request 31444: Support chrooting in MesosContainerizer launch helper.

> On March 18, 2015, 7:09 p.m., Jie Yu wrote: > > src/slave/containerizer/mesos/launch.cpp, lines 268-270 > > > > > > In fact, the chroot target needs to be 'recursively shared' (i.e., > > mount --make-rshared). Do we

Re: Review Request 31444: Support chrooting in MesosContainerizer launch helper.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31444/#review77307 --- src/slave/containerizer/mesos/launch.cpp

Re: Review Request 31444: Support chrooting in MesosContainerizer launch helper.

> On March 18, 2015, 7:09 p.m., Jie Yu wrote: > > src/slave/containerizer/mesos/launch.cpp, lines 268-270 > > > > > > In fact, the chroot target needs to be 'recursively shared' (i.e., > > mount --make-rshared). Do we

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

See Changes: [yujie.jay] Fixed right angle brackets in containerizer code. -- [...truncated 75619 lines...] [ RUN ] FetcherEnvi

Re: Review Request 29329: Add executor for docker containerizer

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29329/#review77312 --- Hey Tim. Made some comments on the doc. Mainly, I'm interested in un

Review Request 32339: Moved http::path::parse from header to .cpp file.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32339/ --- Review request for mesos and Jie Yu. Repository: mesos Description --- M

Review Request 32346: Added failure semantics for http::Pipe::Writer.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32346/ --- Review request for mesos, Benjamin Hindman, Jie Yu, and Joris Van Remoortere. B

Review Request 32336: Moved http::Request::accepts from header into cpp file.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32336/ --- Review request for mesos and Jie Yu. Repository: mesos Description --- M

Review Request 32338: Moved http encode/decode from header to .cpp file.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32338/ --- Review request for mesos and Jie Yu. Repository: mesos Description --- M

Review Request 32343: Re-ordered decoder callbacks to be in execution order.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32343/ --- Review request for mesos and Jie Yu. Repository: mesos Description --- T

Review Request 32341: Fixed a copy/paste naming mistake in decoder_tests.cpp.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32341/ --- Review request for mesos and Jie Yu. Repository: mesos Description --- T

Review Request 32350: Added a wrapper for HttpProcess to prevent segfaults during test failures.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32350/ --- Review request for mesos and Jie Yu. Repository: mesos Description --- I

Review Request 32348: Re-ordered and updated documentation for http::get/post/put.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32348/ --- Review request for mesos and Jie Yu. Repository: mesos Description --- N

Review Request 32342: Updated the http-parser upgrade TODO.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32342/ --- Review request for mesos and Jie Yu. Repository: mesos Description --- h

Review Request 32351: Added http::streaming::get/post for client-side streaming responses.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32351/ --- Review request for mesos, Benjamin Hindman, Jie Yu, and Joris Van Remoortere. B

Review Request 32345: Removed use of 'assert' in decoder.hpp.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32345/ --- Review request for mesos and Jie Yu. Repository: mesos Description --- U

Review Request 32337: Moved http::initialize from header to .cpp file.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32337/ --- Review request for mesos and Jie Yu. Repository: mesos Description --- M

Review Request 32347: Added a StreamingResponseDecoder.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32347/ --- Review request for mesos, Benjamin Hindman and Jie Yu. Bugs: MESOS-2438 htt

Review Request 32340: Moved http::URL output operator from header to .cpp file.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32340/ --- Review request for mesos and Jie Yu. Repository: mesos Description --- M

Review Request 32349: Removed http::put and added a TODO.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32349/ --- Review request for mesos and Jie Yu. Repository: mesos Description --- T

Re: Review Request 32163: Added a function which checks if a json object is contained within another.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32163/#review77287 --- Thanks Alex! This will be an awesome extension of our JSON library :

Re: Review Request 32140: Enable 'Resources' to handle 'Resource::ReservationInfo'.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32140/ --- (Updated March 20, 2015, 10:36 p.m.) Review request for mesos, Alexander Ruklet

Re: GPGPU SLA support

Sounds like you are on the right track; let us know if you have any problems developing/running modules. We created a modules mailing list too where we can go into more details - modu...@mesos.apache.org Cheers, Niklas On 20 March 2015 at 12:47, Anshuman Goswami wrote: > Hi mesos developers, >

Re: Review Request 27760: Revised authenticator interface to allow for two fold implementations.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27760/ --- (Updated March 20, 2015, 10:21 p.m.) Review request for mesos, Adam B, Kapil Ar

Re: Review Request 27760: Revised authenticator interface to allow for two fold implementations.

> On March 16, 2015, 9:41 p.m., Vinod Kone wrote: > > src/authentication/cram_md5/authenticator.cpp, lines 552-554 > > > > > > Why is this an initialization error? The previous semantics are that if > > credentials w

GPGPU SLA support

Hi mesos developers, I am just getting started with understanding the internals. I am working on a project to add some SLA support to GPGPU resources and want to have that work under mesos. After reading this Q&A ( http://stackoverflow.com/questions/27872558/does-apache-mesos-recognize-gpu-cores)

Re: Review Request 32233: Replaced raw pointer by Owned pointer

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32233/#review77259 --- Ship it! Ship It! - Ben Mahler On March 20, 2015, 8:11 a.m., Aka

Re: Review Request 28485: Provide a default bind() in Socket: any port localhost.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28485/#review77255 --- Ship it! Ship It! - Jie Yu On March 20, 2015, 6:24 p.m., Joris V

Re: Review Request 28485: Provide a default bind() in Socket: any port localhost.

> On March 19, 2015, 10:27 p.m., Jie Yu wrote: > > 3rdparty/libprocess/include/process/address.hpp, lines 34-38 > > > > > > Just curious, what's the IPv6 story here? Do you need a > > LOCALHOST_ANY_V6() in the future?

Re: Review Request 31985: Mesos container ID available to the executor through an environment variable.

> On March 14, 2015, 12:49 a.m., Timothy Chen wrote: > > The change looks good, but I'm not sure how exposing the container id is > > the right thing to do overall yet. Container id as I know of is meant to be > > a internal id that is used only in mesos, and I believe the whole > > motivation

Re: Review Request 28485: Provide a default bind() in Socket: any port localhost.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28485/ --- (Updated March 20, 2015, 6:24 p.m.) Review request for mesos, Benjamin Hindman

Re: Review Request 31985: Mesos container ID available to the executor through an environment variable.

> On March 14, 2015, 1:49 a.m., Timothy Chen wrote: > > The change looks good, but I'm not sure how exposing the container id is > > the right thing to do overall yet. Container id as I know of is meant to be > > a internal id that is used only in mesos, and I believe the whole > > motivation

Re: Review Request 28485: Provide a default bind() in Socket: any port localhost.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28485/ --- (Updated March 20, 2015, 5:42 p.m.) Review request for mesos, Benjamin Hindman

Re: Review Request 32306: Suggestion to update launchTasks scheduler Python API docstring

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32306/#review77240 --- Ship it! Thanks! Could you please also update the comments for the

Re: Review Request 32233: Replaced raw pointer by Owned pointer

> On March 20, 2015, 11:01 a.m., Bernd Mathiske wrote: > > LGTM. Thanks! > > Akanksha Agrawal wrote: > Thank you! Could you please merge this? > > Bernd Mathiske wrote: > I'd love to, but I am not a committer. I see you have dedicated this > review to Ben Mahler. He is a committer. I j

Re: Review Request 32306: Suggestion to update launchTasks scheduler Python API docstring

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32306/#review77238 --- Ship it! Ship It! - Niklas Nielsen On March 20, 2015, 7:51 a.m.,

Re: [jira] [Created] (MESOS-2524) Mesos-containerizer not linked from main documentation page.

No reason. Feel free to fix. On Fri, Mar 20, 2015 at 6:02 AM, Joerg Schad (JIRA) wrote: > Joerg Schad created MESOS-2524: > -- > > Summary: Mesos-containerizer not linked from main > documentation page. > Key: MESOS-2524 >

Re: Review Request 31985: Mesos container ID available to the executor through an environment variable.

> On March 14, 2015, 12:49 a.m., Timothy Chen wrote: > > The change looks good, but I'm not sure how exposing the container id is > > the right thing to do overall yet. Container id as I know of is meant to be > > a internal id that is used only in mesos, and I believe the whole > > motivation

Re: Review Request 32306: Suggestion to update launchTasks scheduler Python API docstring

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32306/#review77234 --- LGTM! - Michael Park On March 20, 2015, 2:51 p.m., Itamar Ostrich

FYI: Newly created Austin Mesos User Group

Hey, we will be hosting the first Austin Mesos User Group during ApacheCon NA: http://www.meetup.com/Austin-Mesos-Users-Group/events/22470/ If you're in town, please swing by and special thanks to UA for hosting! -- Cheers,

Re: Review Request 32306: Suggestion to update launchTasks scheduler Python API docstring

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32306/#review77223 --- Patch looks great! Reviews applied: [32306] All tests passed. - M

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

See

Review Request 32306: Suggestion to update launchTasks scheduler Python API docstring

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32306/ --- Review request for mesos. Bugs: MESOS-2525 https://issues.apache.org/jira/b

Build failed in Jenkins: mesos-reviewbot #4720

See -- [...truncated 5596 lines...] make[2]: Leaving directory ` rm -f config.status config.cache config.log configure.lineno c

Re: Review Request 32163: Added a function which checks if a json object is contained within another.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32163/ --- (Updated March 20, 2015, 2:23 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 32198: Added a not equal operator for json objects.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32198/ --- (Updated March 20, 2015, 2:22 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 32163: Added a function which checks if a json object is contained within another.

> On March 18, 2015, 6:28 p.m., Bernd Mathiske wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/json.hpp, line 355 > > > > > > 1. I'd rather forward-declare the class, not the inline function. > > Matter of

Re: Review Request 32233: Replaced raw pointer by Owned pointer

> On March 20, 2015, 4:01 a.m., Bernd Mathiske wrote: > > LGTM. Thanks! > > Akanksha Agrawal wrote: > Thank you! Could you please merge this? I'd love to, but I am not a committer. I see you have dedicated this review to Ben Mahler. He is a committer. I just wanted to help reviewing :-)

Re: Review Request 32233: Replaced raw pointer by Owned pointer

> On March 20, 2015, 11:01 a.m., Bernd Mathiske wrote: > > LGTM. Thanks! Thank you! Could you please merge this? - Akanksha --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32233/#review77197

Re: Review Request 32233: Replaced raw pointer by Owned pointer

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32233/#review77197 --- LGTM. Thanks! - Bernd Mathiske On March 20, 2015, 1:11 a.m., Akan

Re: Review Request 31985: Mesos container ID available to the executor through an environment variable.

> On March 14, 2015, 1:49 a.m., Timothy Chen wrote: > > The change looks good, but I'm not sure how exposing the container id is > > the right thing to do overall yet. Container id as I know of is meant to be > > a internal id that is used only in mesos, and I believe the whole > > motivation

Re: Review Request 31985: Mesos container ID available to the executor through an environment variable.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31985/ --- (Updated March 20, 2015, 10:27 a.m.) Review request for mesos, Bernd Mathiske,

Re: Review Request 32233: Replaced raw pointer by Owned pointer

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32233/#review77192 --- Patch looks great! Reviews applied: [32233] All tests passed. - M

Re: Review Request 32233: Replaced raw pointer by Owned pointer

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32233/ --- (Updated March 20, 2015, 8:11 a.m.) Review request for mesos and Ben Mahler.

Re: Review Request 32233: Replaced raw pointer by Owned pointer

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32233/#review77179 --- src/slave/containerizer/external_containerizer.cpp

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

See