reviews@mesos.apache.org, 2017-08

You are viewing a plain text version of this content. The canonical link for it is here.

- Re: Review Request 60600: Set container DNS with `--default_container_dns` in CNI isolator. - posted by Avinash sridharan <av...@mesosphere.io> on 2017/08/01 00:15:18 UTC, 2 replies.
- Re: Review Request 60761: Added a test `DockerContainerizerTest.ROOT_DOCKER_DefaultDNS`. - posted by Avinash sridharan <av...@mesosphere.io> on 2017/08/01 00:16:52 UTC, 0 replies.
- Re: Review Request 61124: Push metric history entries with their timestamp. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/01 00:35:34 UTC, 1 replies.
- Re: Review Request 61058: Added a lock-free event queue. - posted by Dario Rexin <da...@mesosphere.io> on 2017/08/01 03:59:35 UTC, 0 replies.
- Re: Review Request 60793: Added a test `CniIsolatorTest.ROOT_VerifyDefaultDNS`. - posted by Avinash sridharan <av...@mesosphere.io> on 2017/08/01 05:24:35 UTC, 0 replies.
- Re: Review Request 61171: Enabled filtering of the 'GET_AGENTS' v1 API call. - posted by Greg Mann <gr...@mesosphere.io> on 2017/08/01 05:58:39 UTC, 12 replies.
- Re: Review Request 61257: Added full authz for non summarized fields of `/slaves` endpoint. - posted by Greg Mann <gr...@mesosphere.io> on 2017/08/01 06:04:12 UTC, 2 replies.
- Re: Review Request 61219: Added a test `CniIsolatorTest.ROOT_VerifyDefaultDNSWildcardMatch`. - posted by Avinash sridharan <av...@mesosphere.io> on 2017/08/01 06:44:50 UTC, 1 replies.
- Re: Review Request 61245: Added a test `SlaveTest.ValidateDefaultContainerDNSFlag`. - posted by Avinash sridharan <av...@mesosphere.io> on 2017/08/01 06:51:29 UTC, 4 replies.
- Re: Review Request 61179: Added 'devolve' overload for 'ResourceProviderID'. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/08/01 07:48:26 UTC, 1 replies.
- Re: Review Request 61181: Stored subscribed resources in resource provider manager. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/08/01 07:52:15 UTC, 1 replies.
- Re: Review Request 61134: Added name flag to disk full framework. - posted by Armand Grillet <ag...@mesosphere.io> on 2017/08/01 08:36:49 UTC, 0 replies.
- Re: Review Request 60558: Set container DNS with `--default_container_dns` in Docker executor. - posted by Qian Zhang <zh...@gmail.com> on 2017/08/01 08:41:33 UTC, 4 replies.
- Re: Review Request 61110: Added name flag to balloon and disk full frameworks. - posted by Armand Grillet <ag...@mesosphere.io> on 2017/08/01 10:20:55 UTC, 2 replies.
- Re: Review Request 61112: Added executor_uris flag to long lived and balloon frameworks. - posted by Armand Grillet <ag...@mesosphere.io> on 2017/08/01 10:20:56 UTC, 1 replies.
- Re: Review Request 61137: Cleaned up style in example frameworks. - posted by Armand Grillet <ag...@mesosphere.io> on 2017/08/01 10:21:17 UTC, 4 replies.
- Review Request 61275: Added a UPID parameter to the resource provider driver. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/08/01 14:04:18 UTC, 1 replies.
- Re: Review Request 61272: Added a MockResourceProvider. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/08/01 14:06:16 UTC, 16 replies.
- Re: Review Request 55334: Checkpoint and recover `ContainerConfig` in Mesos containerizer. - posted by Zhitao Li <zh...@gmail.com> on 2017/08/01 17:10:33 UTC, 0 replies.
- Re: Review Request 55335: Backfilled required fields in TaskInfo in MesosContainerizer* tests. - posted by Zhitao Li <zh...@gmail.com> on 2017/08/01 17:10:53 UTC, 0 replies.
- Re: Review Request 61180: Added a resource providers total resources to the subscribe call. - posted by Benjamin Bannier <be...@mesosphere.io> on 2017/08/01 17:42:07 UTC, 1 replies.
- Re: Review Request 61182: Sent a resource provider message when providers subscribe. - posted by Benjamin Bannier <be...@mesosphere.io> on 2017/08/01 17:42:11 UTC, 2 replies.
- Re: Review Request 61183: Triggered 'UpdateSlaveMessage' when 'ResourceProviderManager' updates. - posted by Benjamin Bannier <be...@mesosphere.io> on 2017/08/01 17:42:12 UTC, 0 replies.
- Review Request 61282: Added a test verifying that DefaultExecutor tasks can use nvidia GPUs. - posted by Gastón Kleiman <ga...@mesosphere.io> on 2017/08/01 21:48:40 UTC, 3 replies.
- Review Request 61284: Made `libprocess` use default linking. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:52:50 UTC, 1 replies.
- Review Request 61285: Added library linkage and suffix variables. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:53:23 UTC, 1 replies.
- Review Request 61287: Removed `GroupSource` from `stout`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:53:31 UTC, 3 replies.
- Review Request 61288: Linked `libprocess` to `stout` interface library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:53:38 UTC, 2 replies.
- Review Request 61289: Removed unnecessary includes of `StoutConfigure`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:53:46 UTC, 1 replies.
- Review Request 61290: Linked `mesos` target to `stout` interface library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:53:52 UTC, 1 replies.
- Review Request 61286: Added `stout` interface library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:54:07 UTC, 1 replies.
- Review Request 61291: Imported `glog` library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:54:27 UTC, 3 replies.
- Review Request 61292: Linked `stout` interface library to `glog` imported library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:54:35 UTC, 0 replies.
- Review Request 61293: Removed `glog` configuration from `libprocess`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:54:51 UTC, 0 replies.
- Review Request 61294: Imported `protobuf` library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:54:55 UTC, 1 replies.
- Review Request 61295: Linked `stout` interface library to `protobuf` imported library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:54:56 UTC, 0 replies.
- Review Request 61296: Removed `protobuf` configuration from `libprocess`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:54:57 UTC, 0 replies.
- Review Request 61297: Linked `mesos-protobufs` to `protobuf` imported library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:54:58 UTC, 0 replies.
- Review Request 61298: Replaced `include_directories()` for `mesos-protobufs`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:55:19 UTC, 0 replies.
- Review Request 61299: Removed unnecessary dependency on `mesos-protobufs`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:55:20 UTC, 0 replies.
- Review Request 61300: Imported `protoc` executable. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:55:20 UTC, 1 replies.
- Review Request 61301: Replaced `PROTOC` variable with `protoc` imported executable. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:55:21 UTC, 0 replies.
- Review Request 61302: Added CMake function `make_include()` to work around bug. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:55:22 UTC, 1 replies.
- Review Request 61303: Imported `curl` library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:55:22 UTC, 2 replies.
- Review Request 61304: Linked `stout` to `curl` imported library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:55:23 UTC, 0 replies.
- Review Request 61305: Removed `curl` configuration from `libprocess`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:55:24 UTC, 0 replies.
- Review Request 61306: Enabled building Java artifacts with CMake. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:55:25 UTC, 2 replies.
- Review Request 61307: Refactored `MesosProtobuf` code. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:55:26 UTC, 1 replies.
- Review Request 61308: Updated the ZooKeeper patch. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:55:50 UTC, 2 replies.
- Review Request 61309: Imported ZooKeeper library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:55:51 UTC, 2 replies.
- Review Request 61310: Linked `master` and `agent` to imported ZooKeeper library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:55:51 UTC, 0 replies.
- Review Request 61311: Ported the ZooKeeper tests to CMake and Windows. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:55:52 UTC, 1 replies.
- Review Request 61312: Switched Linux build of `protobuf` to use CMake. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:55:52 UTC, 2 replies.
- Review Request 61313: Bumped minimum required CMake version to 3.7.0. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:55:53 UTC, 3 replies.
- Review Request 61314: Added `boost` interface library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:57:36 UTC, 1 replies.
- Review Request 61315: Linked `stout` to `boost` interface library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:57:38 UTC, 0 replies.
- Review Request 61316: Removed `boost` configuration from `libprocess`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:57:38 UTC, 0 replies.
- Review Request 61318: Added `concurrentqueue` interface library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:57:39 UTC, 1 replies.
- Review Request 61319: Linked `libprocess` to `concurrentqueue` interface library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:57:40 UTC, 0 replies.
- Review Request 61320: Added `elfio` interface library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:57:40 UTC, 1 replies.
- Review Request 61321: Removed `elfio` configuration from `libprocess`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:57:41 UTC, 0 replies.
- Review Request 61323: Added `picojson` interface library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:57:43 UTC, 0 replies.
- Review Request 61324: Linked `stout` to `picojson` interface library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:57:44 UTC, 0 replies.
- Review Request 61325: Removed `picojson` configuration from `libprocess`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:57:45 UTC, 0 replies.
- Review Request 61326: Removed `picojson` configuration from global configuration. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:57:46 UTC, 0 replies.
- Review Request 61327: Imported `zlib` library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:57:47 UTC, 1 replies.
- Review Request 61328: Linked `stout` to `zlib` imported library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:57:49 UTC, 0 replies.
- Review Request 61329: Removed `zlib` configuration from `libprocess`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:57:50 UTC, 0 replies.
- Review Request 61330: Added `nvml` interface library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:57:51 UTC, 0 replies.
- Review Request 61331: Removed `nvml` configuration from `stout`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:57:52 UTC, 0 replies.
- Review Request 61332: Removed `nvml` configuration from `libprocess`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:57:53 UTC, 0 replies.
- Review Request 61333: Linked `libmesos` to `nvml` interface library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:57:54 UTC, 0 replies.
- Review Request 61334: Imported `http_parser` library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:57:55 UTC, 1 replies.
- Review Request 61322: Linked `stout` to `elfio` interface library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:58:05 UTC, 0 replies.
- Review Request 61335: Linked `libprocess` to `http_parser` imported library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:58:49 UTC, 0 replies.
- Review Request 61336: Prioritized provided paths in `FindPackageHelper`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:58:53 UTC, 0 replies.
- Review Request 61337: Imported `apr` library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:58:54 UTC, 0 replies.
- Review Request 61338: Removed `apr` configuration from `libprocess`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:58:55 UTC, 0 replies.
- Review Request 61339: Linked `stout` to `apr` imported library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:58:56 UTC, 0 replies.
- Review Request 61340: Imported `leveldb` library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:58:57 UTC, 0 replies.
- Review Request 61341: Removed `leveldb` configuration from `Mesos3rdpartyConfigure`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:58:59 UTC, 0 replies.
- Review Request 61342: Made master and agent depend on `leveldb` imported library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:59:00 UTC, 0 replies.
- Review Request 61343: Supported finding the correct subversion libraries on macOS. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:59:02 UTC, 0 replies.
- Review Request 61344: Imported `svn` library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:59:03 UTC, 0 replies.
- Review Request 61345: Linked `stout` to `svn` interface library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:59:04 UTC, 0 replies.
- Review Request 61346: Imported `gtest` and `gmock` libraries with a `googletest` interface. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:59:05 UTC, 0 replies.
- Review Request 61347: Linked `stout-tests` to `googletest`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:59:06 UTC, 1 replies.
- Review Request 61348: Linked `libprocess` to `googletest`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:59:08 UTC, 0 replies.
- Review Request 61349: Linked `mesos-tests` to `googletest`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:59:10 UTC, 0 replies.
- Review Request 61350: Removed `GroupSource` usage from `libprocess`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:59:11 UTC, 0 replies.
- Review Request 61351: Removed `GroupSource.cmake`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:59:14 UTC, 0 replies.
- Review Request 61367: Downgraded the `task.executor.resources` in `TASK_GROUP`. - posted by Michael Park <mp...@apache.org> on 2017/08/02 00:59:14 UTC, 1 replies.
- Review Request 61352: Replaced `StoutConfigure.cmake` with `target_link_libraries()`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:59:16 UTC, 0 replies.
- Review Request 61368: Minor clean up: Pulled out the reference to `TaskInfo`. - posted by Michael Park <mp...@apache.org> on 2017/08/02 00:59:17 UTC, 2 replies.
- Review Request 61353: Removed `librt` and `libpthread` from `libprocess`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:59:17 UTC, 0 replies.
- Review Request 61354: Updated ZooKeeper patch to work on OS X. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:59:18 UTC, 0 replies.
- Review Request 61355: Removed unused partial `gperftools` setup. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:59:19 UTC, 0 replies.
- Review Request 61356: Removed `zlib` configuration from `CompilationConfigure.cmake`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:59:21 UTC, 0 replies.
- Review Request 61357: Imported `libev` and `libevent` libraries. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:59:22 UTC, 1 replies.
- Review Request 61358: Linked `libprocess` to `libev` and `libevent`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:59:23 UTC, 0 replies.
- Review Request 61359: Removed `libev` and `libevent` configuration from master and agent. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:59:24 UTC, 0 replies.
- Review Request 61360: Cleaned up `3rdparty/CMakelists.txt`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:59:26 UTC, 0 replies.
- Review Request 61361: Replaced `MESOS_3RDPARTY_*` with `CMAKE_CURRENT_*_DIR`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:59:27 UTC, 2 replies.
- Review Request 61363: Rewrote `libprocess` CMake build. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:59:29 UTC, 1 replies.
- Review Request 61364: Linked `master` and `agent` to `libprocess` properly. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:59:31 UTC, 0 replies.
- Review Request 61365: Added `get_byproducts()` function to fix Ninja generator. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 00:59:37 UTC, 2 replies.
- Re: Review Request 60003: Reduced copying in defer, dispatch and Future. - posted by Michael Park <mp...@apache.org> on 2017/08/02 02:28:17 UTC, 2 replies.
- Re: Review Request 60793: Added a test `DefaultContainerDNSCniTest.ROOT_VerifyDefaultDNS`. - posted by Qian Zhang <zh...@gmail.com> on 2017/08/02 08:13:05 UTC, 2 replies.
- Re: Review Request 61245: Added a test `DefaultContainerDNSFlagTest.ValidateFlag`. - posted by Qian Zhang <zh...@gmail.com> on 2017/08/02 08:15:13 UTC, 3 replies.
- Re: Review Request 60439: Added scripts to automate website publishing. - posted by Benjamin Bannier <be...@mesosphere.io> on 2017/08/02 08:52:54 UTC, 8 replies.
- Re: Review Request 60440: Updated local development workflow of mesos website. - posted by Benjamin Bannier <be...@mesosphere.io> on 2017/08/02 09:11:42 UTC, 2 replies.
- Re: Review Request 61098: Added unit tests for gRPC support in libprocess. - posted by Benjamin Bannier <be...@mesosphere.io> on 2017/08/02 09:55:04 UTC, 8 replies.
- Re: Review Request 61275: Added a URL parameter to the resource provider driver. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/08/02 11:37:01 UTC, 3 replies.
- Re: Review Request 61271: Implemented HTTP connection handling for the resource provider driver. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/08/02 12:38:33 UTC, 9 replies.
- Re: Review Request 61032: Fixed slaves_unreachable metric. - posted by Vinod Kone <vi...@gmail.com> on 2017/08/02 16:27:29 UTC, 0 replies.
- Re: Review Request 61270: Added container PID namespace control protobuf field in LinuxInfo. - posted by Gilbert Song <so...@gmail.com> on 2017/08/02 17:51:59 UTC, 5 replies.
- Re: Review Request 60508: Fixed Attributes comparison. - posted by Anand Mazumdar <an...@apache.org> on 2017/08/02 17:56:41 UTC, 2 replies.
- Re: Review Request 61237: Updated docker executor to return IPv6 address of a container. - posted by Avinash sridharan <av...@mesosphere.io> on 2017/08/02 18:57:46 UTC, 3 replies.
- Re: Review Request 61097: Added gRPC support in libprocess. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/08/02 19:19:14 UTC, 0 replies.
- Review Request 61382: Added MESOS-6375 (hierarchical role support) to the 1.4.0 CHANGELOG. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/02 19:27:49 UTC, 1 replies.
- Review Request 61385: Fixed linking to OpenSSL libraries. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/02 20:12:10 UTC, 1 replies.
- Review Request 61387: Made the rlimits isolator work with nested containers. - posted by Gastón Kleiman <ga...@mesosphere.io> on 2017/08/02 22:05:48 UTC, 6 replies.
- Re: Review Request 58939: Filesystem isolation check for Mesos image provisioner. - posted by Gilbert Song <so...@gmail.com> on 2017/08/02 23:39:01 UTC, 0 replies.
- Re: Review Request 61222: Added V1 teardown call. - posted by Anand Mazumdar <an...@apache.org> on 2017/08/02 23:43:29 UTC, 0 replies.
- Review Request 61393: Added evolve function for CapabilityInfo. - posted by Gastón Kleiman <ga...@mesosphere.io> on 2017/08/03 00:50:01 UTC, 0 replies.
- Review Request 61394: Made the capabilities isolator work with nested containers. - posted by Gastón Kleiman <ga...@mesosphere.io> on 2017/08/03 00:50:58 UTC, 4 replies.
- Re: Review Request 60500: Introduced `--default_container_dns` agent flag. - posted by Qian Zhang <zh...@gmail.com> on 2017/08/03 02:57:24 UTC, 1 replies.
- Review Request 61406: Introduced `--share_pid_namespace` agent flag. - posted by Qian Zhang <zh...@gmail.com> on 2017/08/03 15:20:26 UTC, 3 replies.
- Re: Review Request 61222: Added V1 teardown Call. - posted by Quinn Leng <qu...@gmail.com> on 2017/08/03 17:56:20 UTC, 3 replies.
- Review Request 61408: Added test cases for V1 teardown Call. - posted by Quinn Leng <qu...@gmail.com> on 2017/08/03 17:57:49 UTC, 11 replies.
- Re: Review Request 55318: Used loop in PollSocketImpl. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/03 18:20:23 UTC, 0 replies.
- Re: Review Request 61151: Used Future::recover in Mesos. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/03 18:20:50 UTC, 0 replies.
- Re: Review Request 61157: Refactored ProcessManager::handle for future use with http::Server. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/03 18:24:22 UTC, 0 replies.
- Review Request 61410: Remove support for omitting 202 responses to old libprocess clients. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/03 18:30:18 UTC, 2 replies.
- Re: Review Request 61172: Added mesos.http and mesos.exceptions for CLI. - posted by Eric Chung <ci...@gmail.com> on 2017/08/03 20:45:37 UTC, 13 replies.
- Re: Review Request 61125: Track the total number of time series samples. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/03 22:08:36 UTC, 0 replies.
- Re: Review Request 60957: Added a time window for Timer metrics. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/03 22:10:42 UTC, 0 replies.
- Re: Review Request 61096: Building gRPC with CMake. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/08/04 01:17:51 UTC, 2 replies.
- Re: Review Request 61118: Building gRPC support in libprocess with CMake. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/08/04 01:20:31 UTC, 1 replies.
- Re: Review Request 61095: Updated LICENSE information for grpc 1.4.2. - posted by Jie Yu <yu...@gmail.com> on 2017/08/04 15:43:10 UTC, 0 replies.
- Re: Review Request 61406: Introduced `--disallow_top_level_pid_ns_sharing` agent flag. - posted by Qian Zhang <zh...@gmail.com> on 2017/08/04 16:00:26 UTC, 5 replies.
- Review Request 61428: Added pid ns sharing based on agent flag and protobuf message field. - posted by Qian Zhang <zh...@gmail.com> on 2017/08/04 16:02:55 UTC, 11 replies.
- Review Request 61434: Updated health-checks.md to include description of generalized checks. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/08/04 18:08:40 UTC, 19 replies.
- Review Request 61435: Added logging in docker executor on `docker stop` failure. - posted by Andrei Budnik <ab...@mesosphere.com> on 2017/08/04 18:34:47 UTC, 2 replies.
- Review Request 61433: Checked openssl and zlib as required libraries. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/08/04 18:55:51 UTC, 1 replies.
- Re: Review Request 58898: Send task kill for non-Partition Aware frameworks. - posted by Megha Sharma <ms...@apple.com> on 2017/08/04 23:24:49 UTC, 0 replies.
- Review Request 61438: Improved `NvidiaGpuTest.ROOT_CGROUPS_NVIDIA_GPU_VerifyDeviceAccess`. - posted by Gastón Kleiman <ga...@mesosphere.io> on 2017/08/05 00:06:48 UTC, 1 replies.
- Review Request 61440: Added a TODO to correctly represent framework lifecycle in agent. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/05 02:55:00 UTC, 0 replies.
- Review Request 61441: Updated Framework::removePendingTask to take only a TaskID. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/05 02:55:01 UTC, 0 replies.
- Review Request 61442: Introduced a Framework::idle function in the agent. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/05 02:55:02 UTC, 0 replies.
- Review Request 61443: Marked Framework::hasTask in agent as const. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/05 02:55:03 UTC, 0 replies.
- Review Request 61444: Renamed Framework::pending to Framework::pendingTasks in agent. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/05 02:55:04 UTC, 0 replies.
- Review Request 61445: Fixed a bug in the agent where a kill task is dropped. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/05 02:55:06 UTC, 0 replies.
- Review Request 61446: Added a test to verify a fix for MESOS-7863. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/05 02:55:07 UTC, 1 replies.
- Re: Review Request 52284: Implemented more quota validation tests and validate duplicate name. - posted by Zhitao Li <zh...@gmail.com> on 2017/08/05 04:38:47 UTC, 0 replies.
- Re: Review Request 53691: Implemented some quota functionality tests. - posted by Zhitao Li <zh...@gmail.com> on 2017/08/05 04:40:23 UTC, 0 replies.
- Re: Review Request 52103: Implemented quota update in `/quota` endpoint. - posted by Zhitao Li <zh...@gmail.com> on 2017/08/05 04:41:10 UTC, 0 replies.
- Re: Review Request 58898: Do not kill non partition aware tasks. - posted by Megha Sharma <ms...@apple.com> on 2017/08/06 04:35:02 UTC, 4 replies.
- Re: Review Request 61406: Introduced `--disallow_sharing_agent_pid_namespace` agent flag. - posted by Qian Zhang <zh...@gmail.com> on 2017/08/07 02:52:37 UTC, 1 replies.
- Re: Review Request 60646: Modified handling of parallel test configure flag for documentation. - posted by Benjamin Bannier <be...@mesosphere.io> on 2017/08/07 10:30:45 UTC, 1 replies.
- Review Request 61455: Tweaked checker logs to avoid repeated information. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/08/07 10:47:26 UTC, 1 replies.
- Review Request 61458: Added documentation of parallel test execution config flag. - posted by Benjamin Bannier <be...@mesosphere.io> on 2017/08/07 11:52:25 UTC, 2 replies.
- Re: Review Request 61111: Extracted strings into constants in example frameworks. - posted by Armand Grillet <ag...@mesosphere.io> on 2017/08/07 12:50:28 UTC, 3 replies.
- Review Request 61463: Fixed a bug in the test `NamespacesIsolatorTest.ROOT_PidNamespace`. - posted by Qian Zhang <zh...@gmail.com> on 2017/08/07 15:12:05 UTC, 3 replies.
- Review Request 61464: Added a test `NamespacesIsolatorTest.ROOT_SharePidNamespace`. - posted by Qian Zhang <zh...@gmail.com> on 2017/08/07 15:13:00 UTC, 3 replies.
- Review Request 61465: Added test `NamespacesIsolatorTest.ROOT_SharePidNamespaceWhenDisallow`. - posted by Qian Zhang <zh...@gmail.com> on 2017/08/07 15:13:59 UTC, 2 replies.
- Review Request 61466: Added the test `ROOT_CGROUPS_LaunchNestedSharePidNamespace`. - posted by Qian Zhang <zh...@gmail.com> on 2017/08/07 15:14:54 UTC, 3 replies.
- Review Request 61470: Agent log available with the `/agent/path`. - posted by Alexander Rojas <al...@mesosphere.io> on 2017/08/07 15:55:37 UTC, 1 replies.
- Re: Review Request 53993: Updated quota doc to support quota update. - posted by Zhitao Li <zh...@gmail.com> on 2017/08/07 16:35:34 UTC, 2 replies.
- Re: Review Request 60931: Added test cases for framework streaming events. - posted by Mesos Reviewbot Windows <re...@mesos.apache.org> on 2017/08/07 17:24:59 UTC, 0 replies.
- Review Request 61473: Do not kill non partition aware tasks. - posted by Megha Sharma <ms...@apple.com> on 2017/08/07 18:22:50 UTC, 15 replies.
- Review Request 61474: Removed an unused function in src/linux/ns.hpp. - posted by Jie Yu <yu...@gmail.com> on 2017/08/07 19:05:25 UTC, 1 replies.
- Review Request 61476: Used clone instead of fork in ns::clone. - posted by Jie Yu <yu...@gmail.com> on 2017/08/07 19:07:05 UTC, 4 replies.
- Review Request 61477: Used async signal safe assert in ns::clone. - posted by Jie Yu <yu...@gmail.com> on 2017/08/07 19:07:29 UTC, 1 replies.
- Re: Review Request 56721: Implemented pruneImages with a mark and sweep in docker store. - posted by Zhitao Li <zh...@gmail.com> on 2017/08/07 21:48:17 UTC, 0 replies.
- Re: Review Request 60511: Added MockMetadataManager and test concurrent prune and pull. - posted by Zhitao Li <zh...@gmail.com> on 2017/08/07 21:49:18 UTC, 0 replies.
- Review Request 61483: Added a test using CMD health checks + DefaultExecutor w/ Docker image. - posted by Gastón Kleiman <ga...@mesosphere.io> on 2017/08/07 22:47:46 UTC, 3 replies.
- Review Request 61484: Added a benchmark for ns::clone. - posted by Jie Yu <yu...@gmail.com> on 2017/08/07 23:01:10 UTC, 0 replies.
- Re: Review Request 61433: Checked openssl and zlib as required libraries for Mesos. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/08/08 00:44:29 UTC, 0 replies.
- Re: Review Request 58021: Added storage-related offer operations. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/08/08 08:21:03 UTC, 5 replies.
- Re: Review Request 61189: Added authorization for V1 events. - posted by Alexander Rojas <al...@mesosphere.io> on 2017/08/08 09:36:08 UTC, 21 replies.
- Re: Review Request 61463: Fixed a bug in the test `NamespacesIsolatorTest`. - posted by Qian Zhang <zh...@gmail.com> on 2017/08/08 09:43:04 UTC, 1 replies.
- Review Request 61493: Added a test `DefaultExecutorTest.MultiTaskgroupSharePidNamespace`. - posted by Qian Zhang <zh...@gmail.com> on 2017/08/08 09:44:29 UTC, 12 replies.
- Review Request 61511: Improved the readability of some assertions/expectations. - posted by Gastón Kleiman <ga...@mesosphere.io> on 2017/08/09 00:14:54 UTC, 8 replies.
- Review Request 61512: Fixed up 3rdparty CMake modules. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/09 01:00:14 UTC, 0 replies.
- Review Request 61513: Removed `Mesos3rdpartyConfigure` from `MesosConfigure`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/09 01:01:19 UTC, 0 replies.
- Review Request 61514: Removed moved/deleted CMake modules from libprocess. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/09 01:01:40 UTC, 0 replies.
- Review Request 61515: Updated `CompilationConfigure.cmake` for imported libraries. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/09 01:02:07 UTC, 1 replies.
- Review Request 61516: Fixed `check` and `tests` targets. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/09 01:02:44 UTC, 0 replies.
- Re: Review Request 61433: Refactored OpenSSL library checks in Mesos. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/08/09 03:00:32 UTC, 6 replies.
- Review Request 61517: Refactored OpenSSL library checks in libprocess. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/08/09 03:00:37 UTC, 5 replies.
- Review Request 61524: Moved task validation from `getExecutorInfo` to `runTask` on agent. - posted by Andrei Budnik <ab...@mesosphere.com> on 2017/08/09 13:14:36 UTC, 4 replies.
- Re: Review Request 61495: Removed table from markdown and added cross-links. - posted by Benno Evers <be...@yandex-team.ru> on 2017/08/09 15:41:27 UTC, 2 replies.
- Review Request 61530: Enabled retries for `killTasks` in docker executor. - posted by Andrei Budnik <ab...@mesosphere.com> on 2017/08/09 16:55:33 UTC, 4 replies.
- Review Request 61531: [WIP] Fixed the device number proto 'major' and 'minor' to avoid MACROs. - posted by Gilbert Song <so...@gmail.com> on 2017/08/09 17:38:22 UTC, 2 replies.
- Review Request 61534: Added test to verify the fix for a failed agent assertion. - posted by Greg Mann <gr...@mesosphere.io> on 2017/08/09 18:28:14 UTC, 3 replies.
- Review Request 61535: Updated comments for the agent's 'getExecutorInfo()' helper function. - posted by Greg Mann <gr...@mesosphere.io> on 2017/08/09 18:33:56 UTC, 4 replies.
- Re: Review Request 61531: Fixed the device number proto 'major' and 'minor' to avoid MACROs. - posted by Gilbert Song <so...@gmail.com> on 2017/08/09 19:05:27 UTC, 5 replies.
- Review Request 61538: Used common port range interval code in the port_mapping isolator. - posted by James Peach <jp...@apache.org> on 2017/08/09 20:18:43 UTC, 4 replies.
- Re: Review Request 60495: Added network ports isolator listen socket utilities. - posted by James Peach <jp...@apache.org> on 2017/08/09 20:19:26 UTC, 2 replies.
- Review Request 61544: WIP: Rewrote Mesos CMake build. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/10 04:20:55 UTC, 1 replies.
- Review Request 61546: Created staging dir only when needed. - posted by Qian Zhang <zh...@gmail.com> on 2017/08/10 09:28:58 UTC, 2 replies.
- Re: Review Request 61495: Add documentation for possible task reasons. - posted by Benno Evers <be...@yandex-team.ru> on 2017/08/10 10:43:30 UTC, 4 replies.
- Review Request 61558: Move duplicate comment closer to implementation. - posted by Benno Evers <be...@yandex-team.ru> on 2017/08/10 14:01:34 UTC, 3 replies.
- Re: Review Request 60235: Linted support/test-upgrade.py. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/08/10 16:43:03 UTC, 0 replies.
- Review Request 61204: CLI: Added 'mesos task list' command. - posted by Armand Grillet <ag...@mesosphere.io> on 2017/08/10 17:10:30 UTC, 0 replies.
- Re: Review Request 60088: CLI: Added 'master' key as an acceptable key in config.toml. - posted by Kevin Klues <kl...@gmail.com> on 2017/08/10 17:29:35 UTC, 0 replies.
- Review Request 61564: Libprocess: Improved the readability of some assertions/expectations. - posted by Gastón Kleiman <ga...@mesosphere.io> on 2017/08/10 17:51:35 UTC, 1 replies.
- Review Request 61565: Stout: Improved the readability of some assertions/expectations. - posted by Gastón Kleiman <ga...@mesosphere.io> on 2017/08/10 17:52:17 UTC, 3 replies.
- Review Request 61569: Added the field `signal` to the `KillNestedContainer` call. - posted by Anand Mazumdar <an...@apache.org> on 2017/08/10 19:32:41 UTC, 2 replies.
- Review Request 61575: Added a test for verifying signal escalation on the default executor. - posted by Anand Mazumdar <an...@apache.org> on 2017/08/10 19:32:48 UTC, 11 replies.
- Review Request 61574: Added `kill()` support to the test containerizer interface. - posted by Anand Mazumdar <an...@apache.org> on 2017/08/10 19:32:52 UTC, 4 replies.
- Review Request 61573: Made the default executor support signal escalation. - posted by Anand Mazumdar <an...@apache.org> on 2017/08/10 19:32:56 UTC, 12 replies.
- Review Request 61572: Made `killNestedContainer()` use `kill()` on the containerizer. - posted by Anand Mazumdar <an...@apache.org> on 2017/08/10 19:33:02 UTC, 3 replies.
- Review Request 61571: Added `kill()` call to the composing containerizer. - posted by Anand Mazumdar <an...@apache.org> on 2017/08/10 19:33:19 UTC, 2 replies.
- Review Request 61570: Added the `kill()` function to the containerizer interface. - posted by Anand Mazumdar <an...@apache.org> on 2017/08/10 19:33:24 UTC, 7 replies.
- Review Request 61580: Extracted JNI code into a protected function for clarity. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/08/10 20:41:18 UTC, 6 replies.
- Review Request 61579: Ensured JAVA HTTP adapter propagates a subscription error. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/08/10 20:41:20 UTC, 3 replies.
- Review Request 61578: Updated the CPP linter to ignore patch files for gRPC. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/08/10 20:51:14 UTC, 2 replies.
- Review Request 61576: Added a patch for building/installing the bundled gRPC library. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/08/10 20:52:12 UTC, 1 replies.
- Re: Review Request 61262: Added 'heartbeat' event for the operator API. - posted by Quinn Leng <qu...@gmail.com> on 2017/08/10 21:09:41 UTC, 15 replies.
- Review Request 61584: Fixed a typo in a test. - posted by Gastón Kleiman <ga...@mesosphere.io> on 2017/08/10 21:52:19 UTC, 1 replies.
- Re: Review Request 61260: Added agent garbage collection metrics. - posted by James Peach <jp...@apache.org> on 2017/08/10 22:00:51 UTC, 5 replies.
- Re: Review Request 60088: CLI: Added 'masters' key as an acceptable key in config.toml. - posted by Armand Grillet <ag...@mesosphere.io> on 2017/08/10 23:19:29 UTC, 0 replies.
- Review Request 61588: Added a `--skip-style-check` option when applying reviews. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/08/10 23:41:16 UTC, 4 replies.
- Re: Review Request 59146: Added a 'UNKNOWN' field to the Update.State enumeration. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/08/10 23:43:25 UTC, 0 replies.
- Review Request 61583: Bundled gRPC v1.4.2 into 3rdparty libraries. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/08/11 00:41:25 UTC, 0 replies.
- Re: Review Request 60494: Exposed LinuxLauncher cgroups helper. - posted by Qian Zhang <zh...@gmail.com> on 2017/08/11 13:47:16 UTC, 2 replies.
- Re: Review Request 61118: WIP: Building gRPC support in libprocess with CMake. - posted by Mesos Reviewbot Windows <re...@mesos.apache.org> on 2017/08/11 15:44:38 UTC, 0 replies.
- Re: Review Request 61261: Documented agent garbage collection metrics. - posted by Mesos Reviewbot Windows <re...@mesos.apache.org> on 2017/08/11 17:47:17 UTC, 6 replies.
- Review Request 61597: Windows: Fixed linking to `IPHlpAPI` library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/11 18:31:46 UTC, 5 replies.
- Re: Review Request 60721: Stout: Made `Duration::parse()` handle durations out of range. - posted by Greg Mann <gr...@mesosphere.io> on 2017/08/11 18:32:41 UTC, 3 replies.
- Re: Review Request 61588: Added a `[-s|--skip-hooks]` option when applying reviews. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/08/11 18:45:32 UTC, 7 replies.
- Re: Review Request 60525: Fixed the default filter used by the allocator. - posted by Greg Mann <gr...@mesosphere.io> on 2017/08/11 19:47:37 UTC, 4 replies.
- Re: Review Request 61597: Fixed linking to `IPHlpAPI` library. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/11 23:27:12 UTC, 60 replies.
- Review Request 61602: Fixed mesos containerizer to support docker image WORKDIR missing. - posted by Gilbert Song <so...@gmail.com> on 2017/08/12 01:04:48 UTC, 4 replies.
- Review Request 61601: Stout: make boundary checking in Duration consistent. - posted by Gastón Kleiman <ga...@mesosphere.io> on 2017/08/12 01:06:03 UTC, 1 replies.
- Review Request 61609: Refactored UPID to be immutable and updated libprocess as necessary. - posted by Benjamin Hindman <be...@berkeley.edu> on 2017/08/13 18:46:49 UTC, 0 replies.
- Review Request 61610: Changes in Mesos necessary for UPID immutability. - posted by Benjamin Hindman <be...@berkeley.edu> on 2017/08/13 18:46:52 UTC, 1 replies.
- Review Request 61611: Added hashset::contains for iterable. - posted by Benjamin Hindman <be...@berkeley.edu> on 2017/08/13 19:03:57 UTC, 0 replies.
- Re: Review Request 61220: Added MESOS-5116 to the CHANGELOG. - posted by James Peach <jp...@apache.org> on 2017/08/14 15:45:02 UTC, 4 replies.
- Re: Review Request 60815: Prefer leading instead of trailing spaces for allocator log messages. - posted by Greg Mann <gr...@mesosphere.io> on 2017/08/14 16:36:29 UTC, 0 replies.
- Review Request 61620: Tracked successful task fetches rather than total. - posted by James Peach <jp...@apache.org> on 2017/08/14 17:01:57 UTC, 5 replies.
- Review Request 61621: Added MESOS-7660 to the changelog. - posted by Gastón Kleiman <ga...@mesosphere.io> on 2017/08/14 17:44:35 UTC, 1 replies.
- Re: Review Request 60524: Stout: Made the `Duration` operators handle int overflows explicitly. - posted by Mesos Reviewbot Windows <re...@mesos.apache.org> on 2017/08/14 18:57:35 UTC, 0 replies.
- Review Request 61639: Fixed an bug where the agent kills and still launches a task. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/15 07:00:33 UTC, 1 replies.
- Review Request 61640: Improved the reason and message for killed queued tasks updates. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/15 07:00:34 UTC, 1 replies.
- Review Request 61641: Updated Framework::removePendingTask to take only a TaskID. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/15 07:00:35 UTC, 1 replies.
- Review Request 61642: Introduced a Framework::idle function in the agent. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/15 07:00:36 UTC, 4 replies.
- Review Request 61643: Renamed Framework::pending to Framework::pendingTasks in agent. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/15 07:00:38 UTC, 1 replies.
- Review Request 61644: Introduced a CHECK_NOTNONE. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/15 07:00:39 UTC, 1 replies.
- Review Request 61645: Fixed a bug where TASK_KILLED updates can be dropped by the agent. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/15 07:00:41 UTC, 6 replies.
- Review Request 61646: Added a test to verify a fix for MESOS-7863. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/15 07:00:42 UTC, 1 replies.
- Review Request 61647: Added a TODO to correctly represent framework lifecycle in agent. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/15 07:00:44 UTC, 1 replies.
- Review Request 61648: Marked Framework::hasTask in agent as const. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/15 07:00:45 UTC, 1 replies.
- Review Request 61649: Unschedule directories for GC in parallel in the agent. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/15 07:00:48 UTC, 1 replies.
- Review Request 61638: Implemented LinkedHashMap copy construction and assignment. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/15 07:00:49 UTC, 4 replies.
- Review Request 61650: Controlled queued task handling with helpers in the agent. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/15 07:00:51 UTC, 5 replies.
- Review Request 61654: Added a stream ID to resource provider manager connections. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/08/15 12:01:22 UTC, 3 replies.
- Review Request 61661: Added 'latest' symlink removal. - posted by Ilya Pronin <ip...@twopensource.com> on 2017/08/15 14:48:13 UTC, 1 replies.
- Review Request 61662: Added SlaveRecoveryTest.RebootWithSlaveInfoMismatchAndRestart test. - posted by Ilya Pronin <ip...@twopensource.com> on 2017/08/15 14:48:19 UTC, 2 replies.
- Review Request 61664: Libprocess: Added a timeout for send socket operation. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/08/15 15:45:50 UTC, 4 replies.
- Review Request 61668: Added support for kill policies to the default executor. - posted by Anand Mazumdar <an...@apache.org> on 2017/08/15 17:40:35 UTC, 6 replies.
- Re: Review Request 60620: Modify os::write to write binary files on Windows. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/15 17:47:53 UTC, 3 replies.
- Review Request 61666: Added a test to verify filtering of resource reservations on agent. - posted by Andrei Budnik <ab...@mesosphere.com> on 2017/08/15 17:59:43 UTC, 2 replies.
- Re: Review Request 60622: Add new stout function: path::uri (convert filename to valid URI). - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/15 18:04:42 UTC, 2 replies.
- Re: Review Request 60623: Convert "file://" URI handling to use new path::uri() function. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/15 18:05:55 UTC, 0 replies.
- Re: Review Request 60624: Enable HDFS compilation and associated tests. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/15 18:11:40 UTC, 0 replies.
- Re: Review Request 60625: Normalize file separation characters on Windows when building path. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/15 18:14:11 UTC, 0 replies.
- Re: Review Request 60626: Eliminate os::shell calls from HDFS for Windows compatibility. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/15 18:16:55 UTC, 0 replies.
- Re: Review Request 60628: Enable fetcher_tests.cpp unit test module on Windows platform. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/15 18:51:43 UTC, 0 replies.
- Review Request 61675: Updated doc for the changes of pid namespace sharing. - posted by Gilbert Song <so...@gmail.com> on 2017/08/15 22:50:11 UTC, 4 replies.
- Review Request 61682: Fixed usage of 'ATOMIC_FLAG_INIT'. - posted by Benjamin Bannier <be...@mesosphere.io> on 2017/08/16 08:35:46 UTC, 2 replies.
- Review Request 61689: Removed duplicate using directive. - posted by Ilya Pronin <ip...@twopensource.com> on 2017/08/16 13:58:18 UTC, 1 replies.
- Review Request 61690: Added `--zk_session_timeout` flag for agent. - posted by Ilya Pronin <ip...@twopensource.com> on 2017/08/16 13:58:21 UTC, 3 replies.
- Review Request 61693: Fixed a bug around the `kill()` not removing active containers. - posted by Anand Mazumdar <an...@apache.org> on 2017/08/16 17:27:12 UTC, 6 replies.
- Review Request 61697: Included nested command checks output in the executor logs. - posted by Gastón Kleiman <ga...@mesosphere.io> on 2017/08/16 20:11:45 UTC, 18 replies.
- Review Request 61703: Bundled CSI v0.1.0 into 3rdparty libraries. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/08/16 23:06:44 UTC, 1 replies.
- Review Request 61704: Added CSI client classes to talk to CSI plugins. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/08/16 23:08:29 UTC, 5 replies.
- Review Request 61705: Added a mock CSI plugin and a unit test for CSI client classes. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/08/16 23:13:44 UTC, 4 replies.
- Re: Review Request 61536: Added network ports isolator socket utilities tests. - posted by Qian Zhang <zh...@gmail.com> on 2017/08/17 09:10:29 UTC, 1 replies.
- Re: Review Request 60496: Added socket checking to the network ports isolator. - posted by Qian Zhang <zh...@gmail.com> on 2017/08/17 14:45:34 UTC, 15 replies.
- Re: Review Request 59029: Added a script to publish JARs to maven snapshot repository. - posted by Kapil Arya <ka...@mesosphere.io> on 2017/08/17 21:01:05 UTC, 2 replies.
- Review Request 61723: Bumped version to 1.5.0. - posted by Kapil Arya <ka...@mesosphere.io> on 2017/08/17 23:41:13 UTC, 1 replies.
- Review Request 61725: Added secrets docs. - posted by Kapil Arya <ka...@mesosphere.io> on 2017/08/18 01:30:54 UTC, 10 replies.
- Re: Review Request 60592: Configure the `network/ports` isolator watch interval. - posted by Qian Zhang <zh...@gmail.com> on 2017/08/18 03:03:33 UTC, 3 replies.
- Review Request 61731: Updated configuration.md to reflect new flags. - posted by Kapil Arya <ka...@mesosphere.io> on 2017/08/18 13:32:42 UTC, 1 replies.
- Review Request 61732: Updated endpoints help. - posted by Kapil Arya <ka...@mesosphere.io> on 2017/08/18 13:32:43 UTC, 5 replies.
- Review Request 61733: Synchronized comment in v1 and unversioned master.proto. - posted by Kapil Arya <ka...@mesosphere.io> on 2017/08/18 13:32:45 UTC, 4 replies.
- Review Request 61735: Removed diacritics from docs/health-checks.md. - posted by Till Toenshoff <to...@me.com> on 2017/08/18 14:41:06 UTC, 2 replies.
- Re: Review Request 61149: Added Future::condition. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/08/18 15:17:03 UTC, 0 replies.
- Re: Review Request 61528: Implemented a registrar for resource provider manager state. - posted by Benjamin Bannier <be...@mesosphere.io> on 2017/08/18 15:46:57 UTC, 0 replies.
- Review Request 61740: Added a no-op master registry operation. - posted by Benjamin Bannier <be...@mesosphere.io> on 2017/08/18 15:46:57 UTC, 0 replies.
- Review Request 61742: Added a master-registry back resource provider manager registry. - posted by Benjamin Bannier <be...@mesosphere.io> on 2017/08/18 15:46:59 UTC, 1 replies.
- Review Request 61745: Fixed capitalization and typos in endpoint helps. - posted by Kapil Arya <ka...@mesosphere.io> on 2017/08/18 17:25:04 UTC, 1 replies.
- Re: Review Request 61600: Rewrited rules for generating Protobuf and gRPC code. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/08/18 18:58:57 UTC, 5 replies.
- Review Request 61748: Style cleanup for grpc tests. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/08/18 19:00:45 UTC, 2 replies.
- Review Request 61750: Updated ZooKeeper patch to link with /MTd in Debug configurations. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/18 21:04:14 UTC, 0 replies.
- Review Request 61751: Updated Cotire.cmake module to version 1.7.10. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/18 21:04:28 UTC, 0 replies.
- Review Request 61752: Audited linkage of 3rdparty dependencies. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/18 21:04:45 UTC, 1 replies.
- Review Request 61753: Refactored CMake build system. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/18 21:10:45 UTC, 2 replies.
- Review Request 61759: Updated the docs for V1 HEARTBEAT Event. - posted by Quinn Leng <qu...@gmail.com> on 2017/08/18 22:04:56 UTC, 3 replies.
- Review Request 61763: Added heartbeat interval for V1 Operator API. - posted by Quinn Leng <qu...@gmail.com> on 2017/08/18 23:11:06 UTC, 5 replies.
- Re: Review Request 61600: WIP: Rewrited rules for generating Protobuf and gRPC code. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/08/18 23:58:27 UTC, 0 replies.
- Review Request 61766: Made the output handling of TCP and HTTP checks consistent. - posted by Gastón Kleiman <ga...@mesosphere.io> on 2017/08/19 00:27:14 UTC, 7 replies.
- Re: Review Request 61706: Added Autotools rules to build the CSI client module in Mesos. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/08/19 00:38:34 UTC, 2 replies.
- Re: Review Request 60591: Optionally isolate only the agent network ports. - posted by Qian Zhang <zh...@gmail.com> on 2017/08/21 07:15:38 UTC, 20 replies.
- Re: Review Request 60766: Ignored containers that join CNI networks. - posted by Qian Zhang <zh...@gmail.com> on 2017/08/21 08:32:48 UTC, 9 replies.
- Re: Review Request 60903: Added the `network/ports` isolator to the Mesos containerizer. - posted by Qian Zhang <zh...@gmail.com> on 2017/08/21 08:49:22 UTC, 0 replies.
- Review Request 61791: Raised the logging level of some (health) check messages. - posted by Gastón Kleiman <ga...@mesosphere.io> on 2017/08/21 19:46:52 UTC, 2 replies.
- Review Request 61797: Added async-signal safe implementation of `write` function in stout. - posted by Andrei Budnik <ab...@mesosphere.com> on 2017/08/21 20:54:13 UTC, 9 replies.
- Review Request 61798: Added _EXIT as alternative to ABORT. - posted by Andrei Budnik <ab...@mesosphere.com> on 2017/08/21 20:54:20 UTC, 7 replies.
- Review Request 61799: Replaced ABORT with _EXIT in `childMain` to handle `os::execvpe` error. - posted by Andrei Budnik <ab...@mesosphere.com> on 2017/08/21 20:54:28 UTC, 10 replies.
- Review Request 61800: Enhanced async-signal safety of `signalSafeWriteStatus` function. - posted by Andrei Budnik <ab...@mesosphere.com> on 2017/08/21 20:54:34 UTC, 0 replies.
- Review Request 61801: Used _EXIT macro in `CgroupsAnyHierarchyTest.ROOT_CGROUPS_Write` test. - posted by Andrei Budnik <ab...@mesosphere.com> on 2017/08/21 20:54:41 UTC, 7 replies.
- Review Request 61805: Renamed CONTAINERS_DIR to EXECUTOR_RUNS_DIR. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/08/21 22:21:20 UTC, 1 replies.
- Review Request 61806: Updated depiction of runtime directory file structure. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/08/21 22:21:21 UTC, 2 replies.
- Re: Review Request 60888: WIP: Added recovery logic for standalone containers. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/08/21 22:21:33 UTC, 1 replies.
- Review Request 61808: Add task env overwriting executor env glog print - posted by Andy Pang <pa...@huawei.com> on 2017/08/22 01:56:36 UTC, 2 replies.
- Re: Review Request 60491: Captured the inode when scanning for sockets. - posted by Jie Yu <yu...@gmail.com> on 2017/08/22 03:03:59 UTC, 0 replies.
- Re: Review Request 60493: Removed diagnostic socket IPv4 assumptions. - posted by Jie Yu <yu...@gmail.com> on 2017/08/22 03:05:07 UTC, 0 replies.
- Re: Review Request 60764: Refactored isolator dependency checking. - posted by Jie Yu <yu...@gmail.com> on 2017/08/22 03:32:14 UTC, 2 replies.
- Re: Review Request 60901: Use a consistent preprocessor check for ENABLE_PORT_MAPPING_ISOLATOR. - posted by Jie Yu <yu...@gmail.com> on 2017/08/22 03:32:42 UTC, 0 replies.
- Re: Review Request 60836: Added IntervalSet to Ranges conversion helper declarations. - posted by Jie Yu <yu...@gmail.com> on 2017/08/22 03:43:40 UTC, 0 replies.
- Re: Review Request 60902: Moved the libnl3 configure checks into a macro. - posted by Jie Yu <yu...@gmail.com> on 2017/08/22 03:57:51 UTC, 0 replies.
- Re: Review Request 60492: Added a `network/ports` isolator skeleton. - posted by Jie Yu <yu...@gmail.com> on 2017/08/22 04:04:22 UTC, 0 replies.
- Review Request 61810: Added a function to apply offer operations. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/08/22 10:06:31 UTC, 1 replies.
- Review Request 61811: Flattened continuation chains in containerizer for readability. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/08/22 12:42:29 UTC, 2 replies.
- Re: Review Request 59746: Stopped accounting aborted container launches as failures. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/08/22 13:47:59 UTC, 4 replies.
- Review Request 61818: Adjusted a comment and a log message around container termination. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/08/22 17:18:09 UTC, 2 replies.
- Re: Review Request 60836: Added IntervalSet to Ranges conversion helpers. - posted by James Peach <jp...@apache.org> on 2017/08/22 18:56:45 UTC, 0 replies.
- Re: Review Request 61639: Fixed a bug where the agent kills and still launches a task. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/22 19:11:42 UTC, 0 replies.
- Re: Review Request 61640: Improved the reason and message for tasks killed before delivery. - posted by Benjamin Mahler <bm...@apache.org> on 2017/08/22 19:45:33 UTC, 2 replies.
- Re: Review Request 60765: Added basic `network/ports` isolator tests. - posted by Qian Zhang <zh...@gmail.com> on 2017/08/23 07:58:51 UTC, 2 replies.
- Review Request 61849: Improved consistency of cout/cerr and glog usage in main functions. - posted by Armand Grillet <ag...@mesosphere.io> on 2017/08/23 14:21:38 UTC, 15 replies.
- Re: Review Request 60593: Test the `network/ports` isolator recovery. - posted by Qian Zhang <zh...@gmail.com> on 2017/08/23 14:53:41 UTC, 1 replies.
- Re: Review Request 60594: Added a`network/ports` isolator nested container test. - posted by Qian Zhang <zh...@gmail.com> on 2017/08/23 14:53:46 UTC, 1 replies.
- Re: Review Request 61666: Added test to verify filtering of resource reservations & allocations. - posted by Andrei Budnik <ab...@mesosphere.com> on 2017/08/23 15:05:00 UTC, 5 replies.
- Re: Review Request 60621: Add new stout capability: os::copyfile. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/23 18:10:48 UTC, 2 replies.
- Review Request 61855: Removed a stray trailing parenthesis from a validation error. - posted by Gastón Kleiman <ga...@mesosphere.io> on 2017/08/23 18:46:14 UTC, 0 replies.
- Review Request 61866: Adds port mapping information in NetworkInfo as part of `state` output. - posted by Deepak Goel <de...@gmail.com> on 2017/08/23 22:05:21 UTC, 1 replies.
- Review Request 61869: Adjusted the ContianerLaunchInfo.command merge logic. - posted by Jie Yu <yu...@gmail.com> on 2017/08/23 23:25:23 UTC, 1 replies.
- Review Request 61874: Added a test for IPv6 containers on docker user networks. - posted by Avinash sridharan <av...@mesosphere.io> on 2017/08/24 00:59:22 UTC, 4 replies.
- Review Request 61873: Added test filter for docker user network tests. - posted by Avinash sridharan <av...@mesosphere.io> on 2017/08/24 00:59:52 UTC, 2 replies.
- Review Request 61880: Fixed agent downgrades for reservation refinement. - posted by Michael Park <mp...@apache.org> on 2017/08/24 08:07:29 UTC, 1 replies.
- Review Request 61892: Propagate the error from os::rmdir. - posted by James Peach <jp...@apache.org> on 2017/08/24 19:34:06 UTC, 1 replies.
- Review Request 61893: Fix garbage collection metrics test failures. - posted by James Peach <jp...@apache.org> on 2017/08/24 19:34:46 UTC, 2 replies.
- Review Request 61899: Added HOST_PATH volume source protobuf. - posted by Jie Yu <yu...@gmail.com> on 2017/08/24 23:57:44 UTC, 3 replies.
- Review Request 61900: Added equal operator for Error. - posted by Jie Yu <yu...@gmail.com> on 2017/08/24 23:58:06 UTC, 1 replies.
- Review Request 61901: Added common validation for Volume. - posted by Jie Yu <yu...@gmail.com> on 2017/08/24 23:58:32 UTC, 4 replies.
- Review Request 61902: Added master and agent validation for ContainerInfo. - posted by Jie Yu <yu...@gmail.com> on 2017/08/24 23:58:57 UTC, 1 replies.
- Review Request 61903: Fixed a indentation issue in linux filesystem isolator. - posted by Jie Yu <yu...@gmail.com> on 2017/08/24 23:59:17 UTC, 1 replies.
- Review Request 61904: Added volume/host_path isolator. - posted by Jie Yu <yu...@gmail.com> on 2017/08/24 23:59:46 UTC, 2 replies.
- Review Request 61905: Integrated 'volume/host_path' into MesosContainerizer. - posted by Jie Yu <yu...@gmail.com> on 2017/08/25 00:00:08 UTC, 1 replies.
- Review Request 61906: Added a dependency check 'filesystem/linux' isolator. - posted by Jie Yu <yu...@gmail.com> on 2017/08/25 00:00:28 UTC, 1 replies.
- Review Request 61907: Moved host volume related tests to a dedicated file. - posted by Jie Yu <yu...@gmail.com> on 2017/08/25 00:00:48 UTC, 1 replies.
- Review Request 61908: Used path::absolute to replace some startsWith checks. - posted by Jie Yu <yu...@gmail.com> on 2017/08/25 00:01:11 UTC, 1 replies.
- Re: Review Request 60647: Fixed the mention of default ports in 'configuration.md'. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/08/25 11:42:19 UTC, 0 replies.
- Review Request 61915: Enabled `DockerContainerizerProcess::usage` for all platforms. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/08/25 18:10:18 UTC, 5 replies.
- Review Request 61919: Adjusted the test helpers for creating host and sandbox path volumes. - posted by Jie Yu <yu...@gmail.com> on 2017/08/25 22:29:39 UTC, 1 replies.
- Review Request 61920: Added a test that uses environment secrets and the DefaultExecutor. - posted by Gastón Kleiman <ga...@mesosphere.io> on 2017/08/25 23:45:04 UTC, 5 replies.
- Review Request 61921: Added tests to ensure that tasks can access their parent's volumes. - posted by Gastón Kleiman <ga...@mesosphere.io> on 2017/08/25 23:45:38 UTC, 9 replies.
- Review Request 61927: MESOS-7912 In strict mode code, functions can only be declared at top level - posted by Alastair Montgomery <am...@williamhill.co.uk> on 2017/08/26 13:14:33 UTC, 3 replies.
- Review Request 61922: Added a move constructor for `Owned`. - posted by Greg Mann <gr...@mesosphere.io> on 2017/08/26 14:34:38 UTC, 1 replies.
- Review Request 61923: Updated `AuthorizationAcceptor::create()` to use move semantics. - posted by Greg Mann <gr...@mesosphere.io> on 2017/08/26 14:40:05 UTC, 0 replies.
- Review Request 61924: Moved the 'RejectingObjectApprover' to a public header. - posted by Greg Mann <gr...@mesosphere.io> on 2017/08/26 14:40:35 UTC, 2 replies.
- Review Request 61929: Parameterized the volume host path tests on executor type. - posted by Jie Yu <yu...@gmail.com> on 2017/08/26 20:21:18 UTC, 3 replies.
- Re: Review Request 60915: Enabled filtering of resource allocations and reservations in agent. - posted by Alexander Rojas <al...@mesosphere.io> on 2017/08/28 12:33:42 UTC, 0 replies.
- Review Request 61946: Added validation of resource provider operations. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/08/28 15:28:53 UTC, 2 replies.
- Review Request 61947: Implemented handling of resource provider offer operations. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/08/28 15:30:05 UTC, 2 replies.
- Review Request 61952: Fixed the communication between old masters and new agents. - posted by Michael Park <mp...@apache.org> on 2017/08/28 18:20:56 UTC, 6 replies.
- Re: Review Request 58048: Added id to Resource.DiskInfo. - posted by Jie Yu <yu...@gmail.com> on 2017/08/28 18:24:43 UTC, 0 replies.
- Re: Review Request 60890: WIP: Defined API for launching standalone containers. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/08/29 00:26:39 UTC, 2 replies.
- Review Request 61965: Moved `--zk_session_timeout` flag docs to the common flags section. - posted by Ilya Pronin <ip...@twopensource.com> on 2017/08/29 12:51:44 UTC, 4 replies.
- Review Request 61982: Cleaned up DefaultExecutor tests. - posted by Gastón Kleiman <ga...@mesosphere.io> on 2017/08/30 01:05:02 UTC, 1 replies.
- Review Request 61987: Added an undiscardable() helper that blocks discards from propagating. - posted by Benjamin Hindman <be...@berkeley.edu> on 2017/08/30 06:30:47 UTC, 11 replies.
- Re: Review Request 61981: Added unit test for killing the default executor process. - posted by Gilbert Song <so...@gmail.com> on 2017/08/30 08:29:53 UTC, 5 replies.
- Review Request 61991: Added several logs to the C++ part of the v1-v0 adapter. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/08/30 14:14:44 UTC, 6 replies.
- Review Request 62003: Added `network/ports` isolator nested container tests. - posted by James Peach <jp...@apache.org> on 2017/08/30 23:16:14 UTC, 2 replies.
- Re: Review Request 60593: Added `network/ports` isolator recovery tests. - posted by James Peach <jp...@apache.org> on 2017/08/31 00:08:52 UTC, 0 replies.
- Review Request 62005: Used 'undiscardable' to protect a future in MesosContainerizer. - posted by Jie Yu <yu...@gmail.com> on 2017/08/31 00:32:13 UTC, 4 replies.
- Review Request 62016: Fixed unnecessary non-constness. - posted by Ilya Pronin <ip...@twopensource.com> on 2017/08/31 15:29:20 UTC, 2 replies.
- Review Request 62017: Allows port mapper plugin to have optional args. - posted by Deepak Goel <de...@gmail.com> on 2017/08/31 16:16:32 UTC, 2 replies.
- Review Request 62018: Updated logging::initialize to make flags optional. - posted by Armand Grillet <ag...@mesosphere.io> on 2017/08/31 16:47:28 UTC, 1 replies.