Re: Review Request 45717: Added checks to verify that `rootDir` and `pluginDir` `isSome.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45717/#review127058 --- Patch looks great! Reviews applied: [45717] Passed command: expo

Re: Review Request 45731: Introduced '--networks' flag to mesos-execute.

> On April 5, 2016, 2:25 p.m., Gilbert Song wrote: > > src/cli/execute.cpp, line 152 > > > > > > Should we mention that an image has to be provided && for mesos > > containerizer only? > > Avinash sridharan wrote:

Re: Review Request 45731: Introduced '--networks' flag to mesos-execute.

> On April 5, 2016, 2:25 p.m., Gilbert Song wrote: > > src/cli/execute.cpp, line 187 > > > > > > Could you put it below `_dockerImage`? Thanks. Can you please elaborate why we should put it below `_dockerImage`? The

Re: Review Request 45731: Introduced '--networks' flag to mesos-execute.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45731/ --- (Updated April 5, 2016, 5:13 p.m.) Review request for mesos, Avinash sridharan

Review Request 45734: Fix 'pivot_root is not available' error on powerpc platform.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45734/ --- Review request for mesos and Vinod Kone. Bugs: MESOS-5121 https://issues.ap

Re: Review Request 37168: Add an example framework using dynamic reservation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37168/ --- (Updated April 5, 2016, 5:28 p.m.) Review request for mesos, Greg Mann, Joerg S

Re: Review Request 45707: Minor cleanups to command executor.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45707/#review127071 --- Ship it! Ship It! - Qian Zhang On April 5, 2016, 5:32 a.m.,

Re: Review Request 42551: Porting mesos on ppc64le.

> On March 24, 2016, 10:14 p.m., Mesos ReviewBot wrote: > > Patch looks great! > > > > Reviews applied: [42551] > > > > Passed command: export OS='ubuntu:14.04' CONFIGURATION='--verbose' > > COMPILER='gcc' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; > > ./support/docker_build.sh This seems not t

Re: Review Request 45731: Introduced '--networks' flag to mesos-execute.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45731/#review127079 --- Patch looks great! Reviews applied: [45731] Passed command: expo

Re: Review Request 45734: Fix 'pivot_root is not available' error on powerpc platform.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45734/#review127086 --- Patch looks great! Reviews applied: [45734] Passed command: expo

Re: Review Request 44424: Updated http_command_executor.cpp to use v1 API.

> On April 5, 2016, 7:37 a.m., Anand Mazumdar wrote: > > src/launcher/http_command_executor.cpp, lines 475-477 > > > > > > Why change this? What's wrong with the previous version spanning 2 > > lines? The reason sh

Re: Review Request 44424: Updated http_command_executor.cpp to use v1 API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44424/ --- (Updated April 5, 2016, 8:36 p.m.) Review request for mesos, Anand Mazumdar and

Review Request 45746: Fixed the indent for continuation statements.

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

Re: Review Request 45039: Updated the comment for launching tasks and accepting offers.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45039/ --- (Updated April 5, 2016, 1:27 p.m.) Review request for mesos, Ben Mahler and Nei

Re: Review Request 37168: Add an example framework using dynamic reservation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37168/#review127088 --- Patch looks great! Reviews applied: [37168] Passed command: expo

Re: Review Request 45039: Updated the comment for launching tasks and accepting offers.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45039/#review127091 --- Ship it! Ship It! - Neil Conway On April 5, 2016, 1:27 p.m.,

Re: Review Request 45039: Updated the comment for launching tasks and accepting offers.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45039/#review127092 --- Ship it! Ship It! - Bernd Mathiske On April 5, 2016, 6:27 a.

Re: Review Request 45746: Fixed the indent for continuation statements.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45746/#review127105 --- Patch looks great! Reviews applied: [45746] Passed command: expo

Re: Review Request 45381: Migrate /monitor/statistics and /monitor/statistics.json to slave.

> On March 31, 2016, 7:14 p.m., Jie Yu wrote: > > src/slave/slave.hpp, line 550 > > > > > > Can you move this field to Http class because this is a limiter for the > > http endpoint. Also, `limiter` sounds too gener

Re: Review Request 45381: Migrate /monitor/statistics and /monitor/statistics.json to slave.

> On March 31, 2016, 7:14 p.m., Jie Yu wrote: > > src/slave/slave.hpp, line 550 > > > > > > Can you move this field to Http class because this is a limiter for the > > http endpoint. Also, `limiter` sounds too gener

Review Request 45749: Adjusted lifetimes of member variables.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45749/ --- Review request for mesos, Joseph Wu and Michael Park. Bugs: MESOS-5000 http

Re: Review Request 45039: Updated the comment for launching tasks and accepting offers.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45039/#review127115 --- Patch looks great! Reviews applied: [45039] Passed command: expo

Re: Review Request 45454: Cleanup orphaned docker containers owned by previous agent instance.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45454/#review127119 --- Ship it! Ship It! - Timothy Chen On March 29, 2016, 10:44 p.

Re: Review Request 45455: Added test for recovering orphaned docker containers.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45455/#review127126 --- Fix it, then Ship it! Ship It! src/tests/containerizer/docker

Re: Review Request 45620: Adjusted the status semantics in CNI isolator.

> On April 2, 2016, 7:25 p.m., Avinash sridharan wrote: > > src/slave/containerizer/mesos/isolators/network/cni/cni.cpp, line 817 > > > > > > We should have a comment here explaining why we are not returning a > > f

Re: Review Request 45620: Adjusted the status semantics in CNI isolator.

> On April 5, 2016, 6:32 a.m., Qian Zhang wrote: > > src/slave/containerizer/mesos/isolators/network/cni/cni.cpp, line 824 > > > > > > Just wondering if there are any cases that > > `containerNetwork.networkInfo` is

Re: Review Request 45620: Adjusted the status semantics in CNI isolator.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45620/ --- (Updated April 5, 2016, 4:45 p.m.) Review request for mesos, Avinash sridharan

Re: Review Request 44948: Add XFS disk resource isolator.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44948/#review126845 --- src/slave/containerizer/mesos/isolators/xfs/disk.hpp (line 33)

Re: Review Request 44948: Add XFS disk resource isolator.

> On March 24, 2016, 9:54 a.m., Jiang Yan Xu wrote: > > src/slave/containerizer/mesos/isolators/disk/xfs.cpp, line 568 > > > > > > It seems that we don't need to modify `totalProjectIds` (which can be a > > const) h

Re: Review Request 45360: Added dvd client for mount and unmount.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45360/#review127121 --- src/slave/containerizer/mesos/isolators/docker/dvd/dvd_client.hpp

Re: Review Request 45689: Make Cluster::Slave more tolerant of start failures.

> On April 4, 2016, 5:35 p.m., haosdent huang wrote: > > src/tests/cluster.cpp, line 437 > > > > > > how about > > > > ``` > > if (!containerizer) { > > return; > > } > > ``` > > Jiang Yan

Re: Review Request 45270: Added spec protobut for external mount.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45270/#review127132 --- src/slave/containerizer/mesos/isolators/docker/dvd/spec.proto (li

Re: Review Request 45360: Added dvd client for mount and unmount.

> On April 5, 2016, 4:55 p.m., haosdent huang wrote: > > By the way, I saw you didn't support `create`, `remove` and `path` methods while it exists in dvdcli. Are they unnecessary here? - haosdent --- This is an automatically generated

Re: Review Request 45690: Ensured the bind mount root is a shared mount in its own peer group.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45690/#review127136 --- src/slave/containerizer/mesos/isolators/network/port_mapping.cpp

Re: Review Request 45734: Fix 'pivot_root is not available' error on powerpc platform.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45734/#review127138 --- Ship it! Ship It! - Vinod Kone On April 5, 2016, 9:25 a.m.,

Re: Review Request 45373: Ignored the DOCKER_VOLUME volume source.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45373/#review127137 --- src/slave/containerizer/mesos/isolators/filesystem/linux.cpp (lin

Re: Review Request 42551: Porting mesos on ppc64le.

> On March 24, 2016, 2:14 p.m., Mesos ReviewBot wrote: > > Patch looks great! > > > > Reviews applied: [42551] > > > > Passed command: export OS='ubuntu:14.04' CONFIGURATION='--verbose' > > COMPILER='gcc' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; > > ./support/docker_build.sh > > Zhiwei Chen w

Re: Review Request 45690: Ensured the bind mount root is a shared mount in its own peer group.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45690/ --- (Updated April 5, 2016, 5:25 p.m.) Review request for mesos, Ian Downes and Con

Re: Review Request 45690: Ensured the bind mount root is a shared mount in its own peer group.

> On April 5, 2016, 5:03 p.m., Cong Wang wrote: > > src/slave/containerizer/mesos/isolators/network/port_mapping.cpp, line 1963 > > > > > > It is still not clear why we need to handle this case, at least I don't >

Re: Review Request 45726: Fixed a bug in the `flags::parse` function and added a test case.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45726/ --- (Updated April 5, 2016, 5:33 p.m.) Review request for mesos and Jie Yu. Chang

Re: Review Request 45717: Added checks to verify that `rootDir` and `pluginDir` `isSome.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45717/#review127149 --- Ship it! Ship It! - Jie Yu On April 5, 2016, 6:19 a.m., Avin

Re: Review Request 45726: Fixed a bug in the `flags::parse` function and added a test case.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45726/#review127151 --- Ship it! Ship It! - Jie Yu On April 5, 2016, 5:33 p.m., Mich

Re: Review Request 45214: Updated protobuf to support external storage.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45214/#review127155 --- include/mesos/mesos.proto (line 1604)

Re: Review Request 45749: Adjusted lifetimes of member variables.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45749/#review127157 --- Patch looks great! Reviews applied: [45749] Passed command: expo

Re: Review Request 45370: Implemented prepare() for dvd isolator.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45370/#review127145 --- src/slave/containerizer/mesos/isolators/docker/dvd/dvd.hpp (line

Re: Review Request 45749: Adjusted lifetimes of member variables.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45749/#review127165 --- Ship it! Ship It! - Joseph Wu On April 5, 2016, 9:01 a.m., B

Re: Review Request 45731: Introduced '--networks' flag to mesos-execute.

> On April 5, 2016, 6:25 a.m., Gilbert Song wrote: > > src/cli/execute.cpp, line 152 > > > > > > Should we mention that an image has to be provided && for mesos > > containerizer only? > > Avinash sridharan wrote:

Re: Review Request 45690: Ensured the bind mount root is a shared mount in its own peer group.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45690/ --- (Updated April 5, 2016, 6:25 p.m.) Review request for mesos, Ian Downes and Con

Re: Review Request 45690: Ensured the bind mount root is a shared mount in its own peer group.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45690/#review127173 --- Ship it! Ship It! - Cong Wang On April 5, 2016, 6:25 p.m., J

Re: Review Request 45491: Refactored subprocess options [1/5].

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45491/#review127172 --- 3rdparty/libprocess/include/process/subprocess.hpp (line 80)

Re: Review Request 45492: Used ChildHooks in Mesos [2/5].

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45492/#review127177 --- Fix it, then Ship it! src/slave/containerizer/docker.cpp (line

Re: Review Request 45493: Added namespace option to subprocess [3/5].

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45493/#review127182 --- What is the high-level reasoning for making these namespace specif

Re: Review Request 45495: Removed custom clone functions from Mesos [5/5].

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45495/#review127185 --- src/slave/containerizer/mesos/linux_launcher.cpp (line 279)

Re: Review Request 45529: Do not leak roleSorter and quotaRoleSorter in Mesos allocator.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45529/#review127187 --- Fix it, then Ship it! src/master/allocator/mesos/hierarchical.

Re: Review Request 45594: Introduced a new agent flag docker_config.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45594/ --- (Updated April 5, 2016, 12:20 p.m.) Review request for mesos and Jie Yu. Repo

Re: Review Request 45594: Introduced a new agent flag docker_config.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45594/ --- (Updated April 5, 2016, 12:21 p.m.) Review request for mesos, Guangya Liu, Arte

Re: Review Request 45595: Implemented slave default docker config file support.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45595/ --- (Updated April 5, 2016, 12:22 p.m.) Review request for mesos, Guangya Liu, Arte

Re: Review Request 45594: Introduced a new agent flag docker_config.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45594/#review127191 --- src/slave/flags.cpp (line 136)

Re: Review Request 45595: Implemented slave default docker config file support.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45595/ --- (Updated April 5, 2016, 12:34 p.m.) Review request for mesos, Guangya Liu, Arte

Re: Review Request 45596: Updated docker containerizer private registry doc.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45596/ --- (Updated April 5, 2016, 12:34 p.m.) Review request for mesos and Jie Yu. Repo

Review Request 45767: Fixed commit message hook to iterate over the lines rather than words.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45767/ --- Review request for mesos, Joerg Schad and Vinod Kone. Bugs: MESOS-5125 http

Review Request 45768: Fixed commit message hook to skip over the commented lines.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45768/ --- Review request for mesos, Joerg Schad and Vinod Kone. Bugs: MESOS-5126 http

Re: Review Request 45689: Make Cluster::Slave more tolerant of start failures.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45689/#review127195 --- src/tests/cluster.cpp (lines 364 - 366)

Re: Review Request 45689: Make Cluster::Slave more tolerant of start failures.

> On April 4, 2016, 10:35 a.m., haosdent huang wrote: > > src/tests/cluster.cpp, line 437 > > > > > > how about > > > > ``` > > if (!containerizer) { > > return; > > } > > ``` > > Jiang Ya

Re: Review Request 45768: Fixed commit message hook to skip over the commented lines.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45768/#review127198 --- support/hooks/commit-msg (line 20)

Re: Review Request 45768: Fixed commit message hook to skip over the commented lines.

> On April 5, 2016, 8:01 p.m., Kevin Klues wrote: > > support/hooks/commit-msg, line 20 > > > > > > You should probably quote the $LINE variable here. You should also use > > a single "=", not "==". The double equals

Review Request 45773: Updated the CHANGELOG for 0.28.1.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45773/ --- Review request for mesos, Ben Mahler, Joris Van Remoortere, Michael Park, and Vi

Re: Review Request 45773: Updated the CHANGELOG for 0.28.1.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45773/#review127202 --- Ship it! Ship It! - Ben Mahler On April 5, 2016, 8:13 p.m.,

Re: Review Request 44080: Windows:[1/2] Implemented `os::gmtime_r`.

> On March 18, 2016, 5:36 p.m., Michael Park wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/windows/os.hpp, line 177 > > > > > > Can we remove the `struct` in `struct tm`? Submitted with this fix. - M

Re: Review Request 45594: Introduced a new agent flag docker_config.

> On April 5, 2016, 12:28 p.m., Timothy Chen wrote: > > src/slave/flags.cpp, line 136 > > > > > > Can you probably elaborate a bit more why a docker config file > > configuration is needed? > > > > I know

Re: Review Request 45596: Updated docker containerizer private registry doc.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45596/#review127207 --- Patch looks great! Reviews applied: [45594, 45595, 45596] Passed

Re: Review Request 45595: Implemented slave default docker config file support.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45595/#review127217 --- Ship it! - Jie Yu On April 5, 2016, 7:34 p.m., Gilbert Song w

Re: Review Request 45529: Do not leak roleSorter and quotaRoleSorter in Mesos allocator.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45529/ --- (Updated April 5, 2016, 11:15 p.m.) Review request for mesos and Joris Van Remo

Re: Review Request 45596: Updated docker containerizer private registry doc.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45596/#review127223 --- docs/docker-containerizer.md (lines 64 - 65)

Re: Review Request 45768: Fixed commit message hook to skip over the commented lines.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45768/#review127228 --- Patch looks great! Reviews applied: [45767, 45768] Passed comman

Re: Review Request 45455: Added test for recovering orphaned docker containers.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45455/ --- (Updated April 5, 2016, 10:05 p.m.) Review request for mesos, Jie Yu and Timoth

Re: Review Request 45622: Fixed Nvidia GPU isolator build for gcc.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45622/#review127231 --- Ship it! Ship It! - Ben Mahler On April 2, 2016, 9:35 p.m.,

Re: Review Request 44946: Add utility functions to manipulate XFS project quotas.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44946/#review127166 --- Fix it, then Ship it! Looking good. Just a few minor comments,

Re: Review Request 45067: Updated the long-lived-framework example.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45067/ --- (Updated April 5, 2016, 3:33 p.m.) Review request for mesos, Greg Mann, Artem H

Re: Review Request 45440: Added some metrics to the long-lived-framework example.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45440/ --- (Updated April 5, 2016, 3:33 p.m.) Review request for mesos, Greg Mann, Artem H

Re: Review Request 45067: Updated the long-lived-framework example.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45067/ --- (Updated April 5, 2016, 3:40 p.m.) Review request for mesos, Greg Mann, Artem H

Re: Review Request 45689: Make Cluster::Slave more tolerant of start failures.

> On April 5, 2016, 12:52 p.m., Joseph Wu wrote: > > src/tests/cluster.cpp, lines 364-366 > > > > > > The destructor will only dereference a null `containerizer` if this > > error case is hit (or if you pass in a `n

Re: Review Request 44947: Add tests for XFS project quota utilities.

> On April 1, 2016, 12:27 a.m., Jiang Yan Xu wrote: > > src/tests/containerizer/xfs_quota_tests.cpp, line 71 > > > > > > We have > > > > `makeQuotaInfo` vs. `mkfile` & `mkloop`. Can we make the use of the

Re: Review Request 44948: Add XFS disk resource isolator.

> On April 5, 2016, 4:52 p.m., Jiang Yan Xu wrote: > > src/slave/containerizer/mesos/isolators/xfs/disk.hpp, line 33 > > > > > > Doesn't look like the utils header is used in this header? > > > > Otherwise a

Re: Review Request 44949: Add XFS disk isolator tests.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44949/ --- (Updated April 5, 2016, 10:59 p.m.) Review request for mesos, Jie Yu and Jiang

Re: Review Request 44950: Add XFS disk isolator documentation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44950/ --- (Updated April 5, 2016, 10:59 p.m.) Review request for mesos, Jie Yu and Jiang

Re: Review Request 44948: Add XFS disk resource isolator.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44948/ --- (Updated April 5, 2016, 10:59 p.m.) Review request for mesos, Jie Yu and Jiang

Re: Review Request 44946: Add utility functions to manipulate XFS project quotas.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44946/ --- (Updated April 5, 2016, 11 p.m.) Review request for mesos, Jie Yu and Jiang Yan

Re: Review Request 44947: Add tests for XFS project quota utilities.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44947/ --- (Updated April 5, 2016, 11 p.m.) Review request for mesos, Jie Yu and Jiang Yan

Re: Review Request 44945: Add autoconf tests for XFS project quotas.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44945/ --- (Updated April 5, 2016, 11 p.m.) Review request for mesos, Jie Yu and Jiang Yan

Re: Review Request 44945: Add autoconf tests for XFS project quotas.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44945/ --- (Updated April 5, 2016, 11:15 p.m.) Review request for mesos, Jie Yu and Jiang

Re: Review Request 44950: Add XFS disk isolator documentation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44950/ --- (Updated April 5, 2016, 11:15 p.m.) Review request for mesos, Jie Yu and Jiang

Re: Review Request 44947: Add tests for XFS project quota utilities.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44947/ --- (Updated April 5, 2016, 11:15 p.m.) Review request for mesos, Jie Yu and Jiang

Re: Review Request 44948: Add XFS disk resource isolator.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44948/ --- (Updated April 5, 2016, 11:15 p.m.) Review request for mesos, Jie Yu and Jiang

Re: Review Request 44949: Add XFS disk isolator tests.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44949/ --- (Updated April 5, 2016, 11:15 p.m.) Review request for mesos, Jie Yu and Jiang

Re: Review Request 44946: Add utility functions to manipulate XFS project quotas.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44946/ --- (Updated April 5, 2016, 11:15 p.m.) Review request for mesos, Jie Yu and Jiang

Re: Review Request 45455: Added test for recovering orphaned docker containers.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45455/#review127248 --- Patch looks great! Reviews applied: [45453, 45454, 45455] Passed

Re: Review Request 44947: Add tests for XFS project quota utilities.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44947/ --- (Updated April 5, 2016, 11:40 p.m.) Review request for mesos, Jie Yu and Jiang

Re: Review Request 44945: Add autoconf tests for XFS project quotas.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44945/ --- (Updated April 5, 2016, 11:40 p.m.) Review request for mesos, Jie Yu and Jiang

Re: Review Request 44946: Add utility functions to manipulate XFS project quotas.

> On April 5, 2016, 10:25 p.m., Jiang Yan Xu wrote: > > src/slave/containerizer/mesos/isolators/xfs/utils.cpp, line 286 > > > > > > `s/directory/path`. As discussed, we only accept directories now. > On April 5,

  1   2   >