Re: Review Request 43324: Updated endpoints/index.md.

2016-02-07 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43324/#review118208 --- Ship it! Ship It! - Joerg Schad On Feb. 8, 2016, 7:07 a.m.,

Re: Review Request 43293: Ignored invalid env vars.

2016-02-07 Thread Guangya Liu
> On 二月 7, 2016, 9:13 a.m., haosdent huang wrote: > > src/docker/docker.cpp, line 398 > > > > > > Seems add quote back would be better. I was adding '' instead of quota, hope it is OK. > On 二月 7, 2016, 9:13 a.m.,

Re: Review Request 43293: Ignored invalid env vars.

2016-02-07 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43293/ --- (Updated 二月 7, 2016, 9:40 a.m.) Review request for mesos, haosdent huang, Jie

Re: Review Request 43284: Wait for perf statistics processes exit.

2016-02-07 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43284/ --- (Updated Feb. 7, 2016, 8:12 a.m.) Review request for mesos, Jan Schlicht and

Re: Review Request 43284: Wait for perf statistics processes exit.

2016-02-07 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43284/ --- (Updated Feb. 7, 2016, 8:14 a.m.) Review request for mesos, Jan Schlicht and

Re: Review Request 43283: Fix PerfEventIsolatorTest.ROOT_CGROUPS_Sample failed on CentOS 7.1.

2016-02-07 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43283/ --- (Updated Feb. 7, 2016, 8:14 a.m.) Review request for mesos, Jan Schlicht and

Re: Review Request 43293: Ignored invalid env vars.

2016-02-07 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43293/#review118174 --- Fix it, then Ship it! Ship It! src/docker/docker.cpp (line

Re: Review Request 43293: Ignored invalid env vars.

2016-02-07 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43293/#review118176 --- Patch looks great! Reviews applied: [43293] Passed command:

Re: Review Request 43309: Added systemd life-extension option to posix launcher.

2016-02-07 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43309/#review118178 --- Patch looks great! Reviews applied: [43302, 43303, 43304, 43305,

Review Request 43313: Fixed Typos in support/hooks

2016-02-07 Thread Disha Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43313/ --- Review request for mesos and switched to 'mcypark'. Repository: mesos

Review Request 43307: Added systemd life-extension option to docker process containerizer.

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

Review Request 43305: Moved systemd executor slice initialization logic.

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

Review Request 43309: Added systemd life-extension option to posix launcher.

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

Review Request 43304: Mesos: Disambiguated call to 'subprocess' in linux launcher.

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

Review Request 43306: Migrated linux launcher systemd executor logic into subprocess hook.

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

Review Request 43302: Libprocess: Introduced Hooks and Options for Subprocess.

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

Review Request 43308: Added systemd life-extension option to container logrotate module.

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

Review Request 43303: Libprocess: Added 'options' as an argument to 'subprocess'.

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

Re: Review Request 43122: Updated doxygen target for site/Rakefile.

2016-02-07 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43122/#review118185 --- Ship it! Ship It! - Ben Mahler On Feb. 6, 2016, 11:30 p.m.,

Re: Review Request 43263: Updated site building instructions in site/README.md.

2016-02-07 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43263/#review118186 --- Fix it, then Ship it! site/README.md (line 11)

Re: Review Request 43267: Returned "ServiceUnavailable" for slave's /state during recovery.

2016-02-07 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43267/#review118193 --- Looks good. Just a couple of minor issues.

Re: Review Request 43316: Decrease countCacheEntries in FetcherCacheTest.SimpleEviction .

2016-02-07 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43316/#review118184 --- Patch looks great! Reviews applied: [42696, 43316] Passed

Re: Review Request 43312: Updated docs/presentations.md

2016-02-07 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43312/#review118179 --- Bad patch! Reviews applied: [43312] Failed command:

Re: Review Request 43313: Fixed Typos in support/hooks

2016-02-07 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43313/#review118180 --- Bad patch! Reviews applied: [43313] Failed command:

Review Request 43316: Decrease countCacheEntries in FetcherCacheTest.SimpleEviction .

2016-02-07 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43316/ --- Review request for mesos. Repository: mesos Description --- Decrease

Re: Review Request 43316: Decrease countCacheEntries in FetcherCacheTest.SimpleEviction .

2016-02-07 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43316/ --- (Updated Feb. 7, 2016, 6:38 p.m.) Review request for mesos and Bernd Mathiske.

Review Request 43319: Remove md files from doxygen.

2016-02-07 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43319/ --- Review request for mesos and Ben Mahler. Bugs: MESOS-4619

Review Request 43318: Updated release guide to include doxygen site generation.

2016-02-07 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43318/ --- Review request for mesos and Ben Mahler. Repository: mesos Description

Re: Review Request 43319: Remove md files from doxygen.

2016-02-07 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43319/#review118199 --- Patch looks great! Reviews applied: [43120, 43121, 43122, 43263,

Re: Review Request 43263: Updated site building instructions in site/README.md.

2016-02-07 Thread Kapil Arya
> On Feb. 7, 2016, 3:29 p.m., Ben Mahler wrote: > > site/README.md, line 11 > > > > > > Conflicting version numbers of what? Seems a bit vague? Elaborated a bit. > On Feb. 7, 2016, 3:29 p.m., Ben Mahler wrote: > >

Re: Review Request 43263: Updated site building instructions in site/README.md.

2016-02-07 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43263/ --- (Updated Feb. 7, 2016, 7:33 p.m.) Review request for mesos and Ben Mahler.

Review Request 43321: Speeded up SchedulerTest.Decline by advancing the clock.

2016-02-07 Thread Shuai Lin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43321/ --- Review request for mesos and Alexander Rukletsov. Bugs: MESOS-4175

Re: Review Request 43321: Speeded up SchedulerTest.Decline by advancing the clock.

2016-02-07 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43321/#review118203 --- Ship it! Ship It! - haosdent huang On Feb. 8, 2016, 4:20

Review Request 43324: Updated endpoints/index.md.

2016-02-07 Thread Kevin Klues
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43324/ --- Review request for mesos and Ben Mahler. Bugs: MESOS-3831

Re: Review Request 43220: Added script to generate docs from endpoint help strings.

2016-02-07 Thread Kevin Klues
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43220/ --- (Updated Feb. 8, 2016, 7:06 a.m.) Review request for mesos, Ben Mahler, Greg

Re: Review Request 43313: Fixed Typos in support/hooks.

2016-02-07 Thread Disha Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43313/ --- (Updated Feb. 8, 2016, 5:22 a.m.) Review request for mesos and switched to

Re: Review Request 43312: Updated docs/presentations.md.

2016-02-07 Thread Disha Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43312/ --- (Updated Feb. 8, 2016, 5:23 a.m.) Review request for mesos and switched to

Re: Review Request 43321: Speeded up SchedulerTest.Decline by advancing the clock.

2016-02-07 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43321/#review118202 --- Patch looks great! Reviews applied: [43321] Passed command:

Re: Review Request 42782: Added a unit-test to test net_cls major handles set from command line.

2016-02-07 Thread Avinash sridharan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42782/ --- (Updated Feb. 8, 2016, 6:43 a.m.) Review request for mesos and Jie Yu.

Re: Review Request 42947: Implemented the `status` method in `CgroupNetClsIsolatorProcess`.

2016-02-07 Thread Avinash sridharan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42947/ --- (Updated Feb. 8, 2016, 6:44 a.m.) Review request for mesos and Jie Yu.

Review Request 43322: Exposed `ContainerStatus` in `state.json`.

2016-02-07 Thread Avinash sridharan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43322/ --- Review request for mesos. Repository: mesos Description --- Exposed

Re: Review Request 42983: Implemented the `status` method in `MesosContainerizer`.

2016-02-07 Thread Avinash sridharan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42983/ --- (Updated Feb. 8, 2016, 6:43 a.m.) Review request for mesos and Jie Yu. Bugs:

Re: Review Request 43319: Remove md files from doxygen.

2016-02-07 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43319/#review118205 --- Ship it! Ship It! - Joerg Schad On Feb. 8, 2016, 12:40