Re: Review Request 42685: Added roles validation to master.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42685/#review141534 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 42794: URL query string order is defined.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42794/#review141535 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 49381: WIP: Benchmark for Resources class.

2016-07-09 Thread Klaus Ma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49381/ --- (Updated July 10, 2016, 9:04 a.m.) Review request for mesos, Benjamin Mahler an

Re: Review Request 42591: Added Framework protobufs to registry.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42591/#review141536 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 42594: Introduced framework registry operations.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42594/#review141537 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 43182: Support libprocess build run in aarch64 cpu.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43182/#review141538 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 43091: Renamed a variable and updated a comment for clarity.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43091/#review141539 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 43183: Support routing_tests run in aarch64 cpu.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43183/#review141541 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 43223: Pin intra-package python dependencies to the same internal version

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43223/#review141542 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 42946: Replaced tabs with spaces in configure.ac.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42946/#review141543 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 41108: CMake: Add sasl and dl link flags, add curl link library and add protobuf library directory.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41108/#review141545 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 43358: Added support for all additional Perf Events in PerfEventIsolator.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43358/#review141544 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 43041: Update Master to send revocable resources in separate offers.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43041/#review141546 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 43593: Add test for spliting revocable resources in separate offer.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43593/#review141547 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 36071: Add flow diagram for docker containerizer.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36071/#review141548 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 43269: MasterContender/MasterDetector loadable as modules.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43269/#review141549 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 43814: Enabled volume propagation for docker volume.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43814/#review141550 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 43964: CMAKE: Add leveldb library (3rdparty project).

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43964/#review141551 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 44044: CMake: src CMakeLists-Added source to build master.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44044/#review141554 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 44008: CMake: Top CMakeLists-add master to cmake module path.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44008/#review141553 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 44007: CMake: MesosConfigure-add include of master configure.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44007/#review141552 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 44005: CMake: StoutTestsConfigure-removed FindApr & FindSvn, moved to master.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44005/#review141557 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 43997: CMake: Move FindApr and FindSvn to master.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43997/#review141556 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 43995: CMake: CompilationConfigure-Creating MACROS to get time and date.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43995/#review141555 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 41092: CMake: Added CMake file for agent executable build.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41092/#review141559 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 44003: CMake: Add CMakeLists for master executable build.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44003/#review141558 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 41854: Added module initialization to Master main().

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41854/#review141561 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 44001: CMake: Add MasterConfigure for master executable build.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44001/#review141560 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 41760: Add initialization method to Anonymous class.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41760/#review141563 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 43032: Set Docker labels based on TaskInfo labels.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43032/#review141562 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 42100: Updated the jenkins build script to copy out xml testing reports.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42100/#review141565 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 43932: Added overlayfs provisioning backend.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43932/#review141566 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 44338: Added URI test for HDFS.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44338/#review141567 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 44337: Updated utils for URI constructor streaming.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44337/#review141568 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 43293: Ignored invalid env vars.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43293/#review141570 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 39939: Make docker_socket option support different protocols.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39939/#review141569 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 43807: Replaced with `~` in authentication.md.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43807/#review141571 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 43920: Added a helper function to stout : os/which.hpp.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43920/#review141572 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 44531: Fixed parsing network ip address with docker.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44531/#review141573 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 44911: Fix the issue related to --disable-optimize (MESOS-4621).

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44911/#review141575 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 44315: Do not traverse offer list if there is only one offer.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44315/#review141574 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 45345: Added a python module to mock a CNI plugin and IPAM.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45345/#review141576 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 41410: Export CFS metrics for docker containers

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41410/#review141577 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 45419: Cleaned up ModuleManager.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45419/#review141578 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 45505: WIP : Providing starting implementation for appc runtime isolator.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45505/#review141579 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 45567: Added an interface to insert `iptables` rules in linux.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45567/#review141580 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 42860: Add paths::overlapping to check whether paths are overlapping.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42860/#review141581 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 43471: Add the zookeeper patch for the allow add_auth calls.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43471/#review141582 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 49688: Added cmake build for mesos tests.

2016-07-09 Thread Srinivas Brahmaroutu
> On July 6, 2016, 4:17 p.m., Joseph Wu wrote: > > src/tests/CMakeLists.txt, line 17 > > > > > > It does look like this MesosTestConfigure thing is unused, but I'll > > need to double-check with Alex Clemmer. We use

Re: Review Request 49688: Added cmake build for mesos tests.

2016-07-09 Thread Srinivas Brahmaroutu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49688/ --- (Updated July 10, 2016, 1:49 a.m.) Review request for mesos, Alex Clemmer and J

Re: Review Request 49862: Changed libmesos from static library to a shared library.

2016-07-09 Thread Srinivas Brahmaroutu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49862/ --- (Updated July 10, 2016, 1:49 a.m.) Review request for mesos, Alex Clemmer and J

Re: Review Request 49863: Added Test Modules that are loaded by mesos tests.

2016-07-09 Thread Srinivas Brahmaroutu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49863/ --- (Updated July 10, 2016, 1:50 a.m.) Review request for mesos, Alex Clemmer and J

Re: Review Request 49863: Added Test Modules that are loaded by mesos tests.

2016-07-09 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49863/#review141583 --- Patch looks great! Reviews applied: [49688, 49862, 49863] Passed

Re: Review Request 49688: Added cmake build for mesos tests.

2016-07-09 Thread Srinivas Brahmaroutu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49688/ --- (Updated July 10, 2016, 2:10 a.m.) Review request for mesos, Alex Clemmer and J

Re: Review Request 49862: Changed libmesos from static library to a shared library.

2016-07-09 Thread Srinivas Brahmaroutu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49862/ --- (Updated July 10, 2016, 2:10 a.m.) Review request for mesos, Alex Clemmer and J

Re: Review Request 49863: Added Test Modules that are loaded by mesos tests.

2016-07-09 Thread Srinivas Brahmaroutu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49863/ --- (Updated July 10, 2016, 2:10 a.m.) Review request for mesos, Alex Clemmer and J

Re: Review Request 49817: Implemented `CgroupsIsolatorProcess::recover`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49817/#review141584 --- src/slave/containerizer/mesos/isolators/cgroups/cgroups.cpp (line

Re: Review Request 49818: Implemented `CgroupsIsolatorProcess::createCgroup`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49818/#review141585 --- src/slave/containerizer/mesos/isolators/cgroups/cgroups.cpp (line

Re: Review Request 49819: Implemented `CgroupsIsolatorProcess::prepare`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49819/#review141586 --- src/slave/containerizer/mesos/isolators/cgroups/cgroups.cpp (line

Re: Review Request 49820: Implemented `CgroupsIsolatorProcess::isolate`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49820/#review141587 --- src/slave/containerizer/mesos/isolators/cgroups/cgroups.cpp (line

Re: Review Request 49823: Implemented `CgroupsIsolatorProcess::update`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49823/#review141588 --- src/slave/containerizer/mesos/isolators/cgroups/cgroups.cpp (line

Re: Review Request 49824: Implemented `CgroupsIsolatorProcess::usage`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49824/#review141589 --- src/slave/containerizer/mesos/isolators/cgroups/cgroups.cpp (line

Re: Review Request 49825: Implemented `CgroupsIsolatorProcess::status`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49825/#review141590 --- src/slave/containerizer/mesos/isolators/cgroups/cgroups.cpp (line

Re: Review Request 49381: WIP: Benchmark for Resources class.

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

Re: Review Request 49827: Implemented `CgroupsIsolatorProcess::cleanup`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49827/#review141592 --- src/slave/containerizer/mesos/isolators/cgroups/cgroups.cpp (line

Re: Review Request 49851: Implemented `MemorySubsystem`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49851/#review141593 --- src/slave/containerizer/mesos/isolators/cgroups/subsystem.cpp (li

Re: Review Request 49851: Implemented `MemorySubsystem`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49851/#review141594 --- src/slave/containerizer/mesos/isolators/cgroups/subsystem.cpp (li

Re: Review Request 49813: Added stubs for the unified cgroups isolator.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49813/#review141595 --- src/slave/containerizer/mesos/isolators/cgroups/cgroups.cpp (line

Re: Review Request 45573: Add `PerfEventHandleManager`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45573/ --- (Updated July 10, 2016, 4:05 a.m.) Review request for mesos, Gilbert Song, Guan

Review Request 49864: Fixed ExecutorPIDTest.

2016-07-09 Thread Haris Choudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49864/ --- Review request for mesos and Jie Yu. Repository: mesos Description --- F

Re: Review Request 49864: Fixed ExecutorPIDTest.

2016-07-09 Thread Haris Choudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49864/ --- (Updated July 10, 2016, 4:19 a.m.) Review request for mesos and Jie Yu. Repos

Re: Review Request 49863: Added Test Modules that are loaded by mesos tests.

2016-07-09 Thread Srinivas Brahmaroutu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49863/ --- (Updated July 10, 2016, 5:17 a.m.) Review request for mesos, Alex Clemmer and J

Re: Review Request 49863: Added Test Modules that are loaded by mesos tests.

2016-07-09 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49863/#review141597 --- Patch looks great! Reviews applied: [49688, 49862, 49863] Passed

Re: Review Request 49864: Fixed ExecutorPIDTest.

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

Re: Review Request 49864: Fixed ExecutorPIDTest.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49864/#review141599 --- Fix it, then Ship it! src/tests/containerizer/mesos_containeri

Re: Review Request 49864: Fixed ExecutorPIDTest.

2016-07-09 Thread Haris Choudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49864/ --- (Updated July 10, 2016, 6:28 a.m.) Review request for mesos and Jie Yu. Repos

Re: Review Request 49685: Added filtering for orphaned executors in `GET_EXECUTORS` operator API.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49685/#review141601 --- src/master/http.cpp (line 1557)

Re: Review Request 49685: Added filtering for orphaned executors in `GET_EXECUTORS` operator API.

2016-07-09 Thread haosdent huang
> On July 6, 2016, 6:37 p.m., Joerg Schad wrote: > > src/master/http.cpp, line 1561 > > > > > > s/info/executorInfo ? Thank you very much. Post a follow up fix at https://reviews.apache.org/r/49865/ - haosdent

Review Request 49865: Renamed `info` to `executorInfo` in `Master::Http::_getExecutors`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49865/ --- Review request for mesos, Joerg Schad and Vinod Kone. Bugs: MESOS-5750 http

Re: Review Request 49846: The SorterTest.RevocableResources should add total resources for slave.

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

Re: Review Request 49223: Fixed Value parsing code to only accept the canonical formats.

2016-07-09 Thread Klaus Ma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49223/ --- (Updated July 9, 2016, 3:54 p.m.) Review request for mesos and Benjamin Mahler.

Re: Review Request 49223: Fixed Value parsing code to only accept the canonical formats.

2016-07-09 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49223/#review141396 --- Your patch cannot be applied with description exceed 72 characters

Re: Review Request 49847: Added document about reopening old reviews.

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

Re: Review Request 49844: Fixed a subscriber FD leak when running tests.

2016-07-09 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49844/#review141398 --- Patch looks great! Reviews applied: [49837, 49844] Passed comman

Re: Review Request 49813: Added stubs for the unified cgroups isolator.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49813/ --- (Updated July 9, 2016, 10:16 a.m.) Review request for mesos, Gilbert Song, Jie

Re: Review Request 49815: Implemented `CgroupsIsolatorProcess::prepareHierarchy`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49815/ --- (Updated July 9, 2016, 10:16 a.m.) Review request for mesos, Gilbert Song, Jie

Re: Review Request 49812: Added missing header `version.hpp` in `src/linux/perf.hpp`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49812/ --- (Updated July 9, 2016, 10:16 a.m.) Review request for mesos, Gilbert Song, Jie

Re: Review Request 49814: Implemented `CgroupsIsolatorProcess::create`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49814/ --- (Updated July 9, 2016, 10:16 a.m.) Review request for mesos, Gilbert Song, Jie

Re: Review Request 49816: Implemented `CgroupsIsolatorProcess::initialize`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49816/ --- (Updated July 9, 2016, 10:16 a.m.) Review request for mesos, Gilbert Song, Jie

Re: Review Request 49817: Implemented `CgroupsIsolatorProcess::recover`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49817/ --- (Updated July 9, 2016, 10:16 a.m.) Review request for mesos, Gilbert Song, Jie

Re: Review Request 49821: Implemented `CgroupsIsolatorProcess::watch`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49821/ --- (Updated July 9, 2016, 10:16 a.m.) Review request for mesos, Gilbert Song, Jie

Re: Review Request 49824: Implemented `CgroupsIsolatorProcess::usage`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49824/ --- (Updated July 9, 2016, 10:16 a.m.) Review request for mesos, Gilbert Song, Jie

Re: Review Request 49819: Implemented `CgroupsIsolatorProcess::prepare`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49819/ --- (Updated July 9, 2016, 10:16 a.m.) Review request for mesos, Gilbert Song, Jie

Re: Review Request 49823: Implemented `CgroupsIsolatorProcess::update`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49823/ --- (Updated July 9, 2016, 10:16 a.m.) Review request for mesos, Gilbert Song, Jie

Re: Review Request 49818: Implemented `CgroupsIsolatorProcess::createCgroup`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49818/ --- (Updated July 9, 2016, 10:16 a.m.) Review request for mesos, Gilbert Song, Jie

Re: Review Request 49820: Implemented `CgroupsIsolatorProcess::isolate`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49820/ --- (Updated July 9, 2016, 10:16 a.m.) Review request for mesos, Gilbert Song, Jie

Re: Review Request 49822: Implemented `CgroupsIsolatorProcess::notify`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49822/ --- (Updated July 9, 2016, 10:16 a.m.) Review request for mesos, Gilbert Song, Jie

Re: Review Request 49827: Implemented `CgroupsIsolatorProcess::cleanup`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49827/ --- (Updated July 9, 2016, 10:17 a.m.) Review request for mesos, Gilbert Song, Jie

Re: Review Request 49825: Implemented `CgroupsIsolatorProcess::status`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49825/ --- (Updated July 9, 2016, 10:17 a.m.) Review request for mesos, Gilbert Song, Jie

Review Request 49849: Implemented `CpuSubsystem`.

2016-07-09 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49849/ --- Review request for mesos, Gilbert Song, Jie Yu, and Qian Zhang. Repository: mes

  1   2   3   >