| Mesos ReviewBot |
Re: Review Request 38075: Added an Appc provisioner recovery test. |
Thu, 10 Sep, 05:08 |
| Mesos ReviewBot |
Re: Review Request 37505: Fix broken health check in docker executor. |
Thu, 10 Sep, 06:01 |
| Mesos ReviewBot |
Re: Review Request 37968: Add more comments for why not using const reference for some functions |
Thu, 10 Sep, 06:52 |
| Mesos ReviewBot |
Re: Review Request 37773: Docker: Adding registry client. |
Thu, 10 Sep, 07:52 |
| Mesos ReviewBot |
Re: Review Request 37903: stout: Fix bug in IPNetwork::create() with zero prefix. |
Thu, 10 Sep, 08:52 |
| Mesos ReviewBot |
Re: Review Request 38075: Added an Appc provisioner recovery test. |
Thu, 10 Sep, 09:42 |
| Mesos ReviewBot |
Re: Review Request 37505: Fix broken health check in docker executor. |
Thu, 10 Sep, 10:40 |
| Mesos ReviewBot |
Re: Review Request 38252: Remove namespaces method in LinuxFilesystemIsolatorProcess. |
Thu, 10 Sep, 11:59 |
| Mesos ReviewBot |
Re: Review Request 38137: Added Docker provisioner and local store |
Thu, 10 Sep, 12:51 |
| Mesos ReviewBot |
Re: Review Request 38251: FrameworkInfo should only be updated if the re-registration is valid |
Thu, 10 Sep, 13:44 |
| Mesos ReviewBot |
Re: Review Request 38253: [MESOS-2875] Add containerId to ResourceUsage to enable QoS controller to target a container |
Thu, 10 Sep, 14:42 |
| Mesos ReviewBot |
Re: Review Request 38256: Update framework-rate-limiting.md |
Thu, 10 Sep, 15:42 |
| Mesos ReviewBot |
Re: Review Request 38259: [MESOS-3340] Command-line flags should take precedence over OS Env variables |
Thu, 10 Sep, 16:41 |
| Mesos ReviewBot |
Re: Review Request 38056: Fix webui task informations not update bug. |
Thu, 10 Sep, 17:31 |
| Mesos ReviewBot |
Re: Review Request 36126: Removing '.json' extension in slave endpoints url |
Thu, 10 Sep, 18:04 |
| Mesos ReviewBot |
Re: Review Request 36127: Removing '.json' extension in files endpoints url |
Thu, 10 Sep, 19:02 |
| Mesos ReviewBot |
Re: Review Request 38154: Switched to type traits for checking whether a type is a protobuf message. |
Fri, 11 Sep, 02:39 |
| Mesos ReviewBot |
Re: Review Request 38158: Refactored Value::Ranges coalesce(). |
Fri, 11 Sep, 05:38 |
| Mesos ReviewBot |
Re: Review Request 37821: Join threads in libprocess when shutting down. |
Fri, 11 Sep, 06:58 |
| Mesos ReviewBot |
Re: Review Request 36127: Removing '.json' extension in files endpoints url |
Fri, 11 Sep, 08:05 |
| Mesos ReviewBot |
Re: Review Request 38143: Adding executor HTTP API protobuf to V1 |
Fri, 11 Sep, 09:50 |
| Mesos ReviewBot |
Re: Review Request 38284: Layers from Appc store should be the rootfs dir, not the top level dir. |
Fri, 11 Sep, 10:55 |
| Mesos ReviewBot |
Re: Review Request 38289: Handle bad request in Docker registry client. |
Fri, 11 Sep, 12:30 |
| Mesos ReviewBot |
Re: Review Request 38251: FrameworkInfo should only be updated if the re-registration is valid |
Fri, 11 Sep, 13:53 |
| Mesos ReviewBot |
Re: Review Request 38287: Check if the futrue is failed before dispatch in freeze() |
Fri, 11 Sep, 15:07 |
| Mesos ReviewBot |
Re: Review Request 38137: Added Docker provisioner and local store |
Fri, 11 Sep, 16:28 |
| Mesos ReviewBot |
Re: Review Request 38259: [MESOS-3340] Command-line flags should take precedence over OS Env variables |
Fri, 11 Sep, 17:38 |
| Mesos ReviewBot |
Re: Review Request 38300: Add a test case to reproduce EBUSY error. |
Fri, 11 Sep, 19:08 |
| Mesos ReviewBot |
Re: Review Request 37821: Join threads in libprocess when shutting down. |
Fri, 11 Sep, 21:44 |
| Mesos ReviewBot |
Re: Review Request 38077: [5/5] Integer Precision for JSON <-> Protobuf conversions. |
Fri, 11 Sep, 23:29 |
| Mesos ReviewBot |
Re: Review Request 38307: Libprocess: Removed namespace pollution. |
Sat, 12 Sep, 00:48 |
| Mesos ReviewBot |
Re: Review Request 38289: Handle bad request in Docker registry client. |
Sat, 12 Sep, 01:22 |
| Mesos ReviewBot |
Re: Review Request 38321: Adding upgrade specifications to docs for '.json' endpoints |
Sat, 12 Sep, 02:34 |
| Mesos ReviewBot |
Re: Review Request 38327: Added workaround for rmdir after umount in LinuxFilesystemIsolator. |
Sat, 12 Sep, 03:32 |
| Mesos ReviewBot |
Re: Review Request 38329: Ensured that the device containing slave work_dir is a shared mount when LinuxFilesystemIsolator is used. |
Sat, 12 Sep, 07:57 |
| Mesos ReviewBot |
Re: Review Request 38321: Adding upgrade specifications to docs for '.json' endpoints |
Sat, 12 Sep, 19:05 |
| Mesos ReviewBot |
Re: Review Request 38259: [MESOS-3340] Command-line flags should take precedence over OS Env variables |
Sun, 13 Sep, 06:21 |
| Mesos ReviewBot |
Re: Review Request 38333: Made container sandbox a shared mount to address MESOS-3349. |
Sun, 13 Sep, 07:12 |
| Mesos ReviewBot |
Re: Review Request 38334: Added Labels field of FrameworkInfo into v1 mesos.proto |
Sun, 13 Sep, 14:22 |
| Mesos ReviewBot |
Re: Review Request 38335: Add JSON::protobuf for google::protobuf::RepeatedPtrField |
Sun, 13 Sep, 16:39 |
| Mesos ReviewBot |
Re: Review Request 38337: Extract gz file in fetcher. |
Sun, 13 Sep, 18:01 |
| Mesos ReviewBot |
Re: Review Request 38338: Enhanced option for Docker cli volume plugin. |
Sun, 13 Sep, 19:02 |
| Mesos ReviewBot |
Re: Review Request 37821: Join threads in libprocess when shutting down. |
Mon, 14 Sep, 00:05 |
| Mesos ReviewBot |
Re: Review Request 38343: mesos: Fixed punctuation in log message. |
Mon, 14 Sep, 06:02 |
| Mesos ReviewBot |
Re: Review Request 38287: Check if the futrue is failed before dispatch in freeze() |
Mon, 14 Sep, 13:21 |
| Mesos ReviewBot |
Re: Review Request 37505: Fix broken health check in docker executor. |
Mon, 14 Sep, 18:35 |
| Mesos ReviewBot |
Re: Review Request 37168: MESOS-3063 (Add an example framework using dynamic reservation) |
Mon, 14 Sep, 23:13 |
| Mesos ReviewBot |
Re: Review Request 38375: Fixed process::collect and process::await to do discard propagation. |
Tue, 15 Sep, 10:30 |
| Mesos ReviewBot |
Re: Review Request 38382: Fixed the perf event isolator to continue sampling in the presence of failures. |
Tue, 15 Sep, 17:09 |
| Mesos ReviewBot |
Re: Review Request 38376: Fix 'operator=' spacing in the V1 Scheduler interface. |
Tue, 15 Sep, 18:37 |
| Mesos ReviewBot |
Re: Review Request 37969: Maintenance primitives: Tweak validation error messages to return JSON rather than protobuf. |
Tue, 15 Sep, 19:43 |
| Mesos ReviewBot |
Re: Review Request 38386: Fix 'operator=' spacing in the registry_client.hpp |
Tue, 15 Sep, 20:43 |
| Mesos ReviewBot |
Re: Review Request 38011: Maintenance Primitives: Use the parse<RepeatedPtrField<T>> helper instead of a plural MachineID protobuf. |
Tue, 15 Sep, 21:42 |
| Mesos ReviewBot |
Re: Review Request 37275: [2/2]Generate make batch file to build project in windows. |
Tue, 15 Sep, 22:33 |
| Mesos ReviewBot |
Re: Review Request 38158: Refactored Value::Ranges coalesce(). |
Tue, 15 Sep, 23:32 |
| Mesos ReviewBot |
Re: Review Request 37903: stout: Fix bug in IPNetwork::create() with zero prefix. |
Wed, 16 Sep, 01:32 |
| Mesos ReviewBot |
Re: Review Request 38370: Updated docker executor to set container IP in TaskStatus::NetworkInfo. |
Wed, 16 Sep, 01:57 |
| Mesos ReviewBot |
Re: Review Request 38143: Adding executor HTTP API protobuf to V1 |
Wed, 16 Sep, 02:52 |
| Mesos ReviewBot |
Re: Review Request 38417: Unified the implementations of image provisioners. |
Wed, 16 Sep, 04:05 |
| Mesos ReviewBot |
Re: Review Request 38416: Allow HTTP response codes to be checked with code. |
Wed, 16 Sep, 05:03 |
| Mesos ReviewBot |
Re: Review Request 37275: [2/2]Generate make batch file to build project in windows. |
Wed, 16 Sep, 06:43 |
| Mesos ReviewBot |
Re: Review Request 38124: Add V1 Support for QuiesceOffers |
Wed, 16 Sep, 08:53 |
| Mesos ReviewBot |
Re: Review Request 38056: Fix webui task informations not update bug. |
Wed, 16 Sep, 15:51 |
| Mesos ReviewBot |
Re: Review Request 38011: Maintenance Primitives: Use the parse<RepeatedPtrField<T>> helper instead of a plural MachineID protobuf. |
Wed, 16 Sep, 20:23 |
| Mesos ReviewBot |
Re: Review Request 38370: Updated docker executor to set container IP in TaskStatus::NetworkInfo. |
Wed, 16 Sep, 20:48 |
| Mesos ReviewBot |
Re: Review Request 38370: Updated docker executor to set container IP in TaskStatus::NetworkInfo. |
Wed, 16 Sep, 22:53 |
| Mesos ReviewBot |
Re: Review Request 38253: Add containerId to ResourceUsage to enable QoS controller to target a container |
Thu, 17 Sep, 00:21 |
| Mesos ReviewBot |
Re: Review Request 38443: Added layerid information to ManifestResponse |
Thu, 17 Sep, 02:08 |
| Mesos ReviewBot |
Re: Review Request 38444: Only unmount sandbox if the mount exists in LinuxFilesystemIsolator. |
Thu, 17 Sep, 03:11 |
| Mesos ReviewBot |
Re: Review Request 38447: Implemented a TODO to clean up host mounts irrelevant to the container in the container's mount namespace. |
Thu, 17 Sep, 07:23 |
| Mesos ReviewBot |
Re: Review Request 38452: Updated C++ style guide for namespace usage |
Thu, 17 Sep, 09:07 |
| Mesos ReviewBot |
Re: Review Request 38399: Add ACLs for the maintenance HTTP endpoints |
Thu, 17 Sep, 10:04 |
| Mesos ReviewBot |
Re: Review Request 38454: Fix failed test LimitedCpuIsolatorTest.ROOT_CGROUPS_Pids_and_Tids |
Thu, 17 Sep, 11:07 |
| Mesos ReviewBot |
Re: Review Request 38455: CMake: Remove thread_tests.cpp from Stout tests. |
Thu, 17 Sep, 12:01 |
| Mesos ReviewBot |
Re: Review Request 38457: CMake: Fix MESOS-3250, a dynamic load error in Stout tests on OS X. |
Thu, 17 Sep, 13:02 |
| Mesos ReviewBot |
Re: Review Request 38003: MESOS-3351 (duplicated slave id in master after master failover) |
Thu, 17 Sep, 16:03 |
| Mesos ReviewBot |
Re: Review Request 38443: Added layerid information to ManifestResponse |
Thu, 17 Sep, 17:51 |
| Mesos ReviewBot |
Re: Review Request 38253: Add containerId to ResourceUsage to enable QoS controller to target a container |
Thu, 17 Sep, 18:43 |
| Mesos ReviewBot |
Re: Review Request 38279: Enabled resources/attributes discovery |
Thu, 17 Sep, 19:08 |
| Mesos ReviewBot |
Re: Review Request 38077: [5/5] Integer Precision for JSON <-> Protobuf conversions. |
Thu, 17 Sep, 20:02 |
| Mesos ReviewBot |
Re: Review Request 38468: docs: Added discussion of finding a shepherd. |
Thu, 17 Sep, 20:55 |
| Mesos ReviewBot |
Re: Review Request 38468: docs: Added discussion of finding a shepherd. |
Thu, 17 Sep, 22:13 |
| Mesos ReviewBot |
Re: Review Request 38447: Implemented a TODO to clean up host mounts irrelevant to the container in the container's mount namespace. |
Thu, 17 Sep, 22:41 |
| Mesos ReviewBot |
Re: Review Request 38470: Maintenance Primitives: Fix error in master's Accept/Decline for inverse offers. |
Thu, 17 Sep, 23:39 |
| Mesos ReviewBot |
Re: Review Request 38003: MESOS-3351 (duplicated slave id in master after master failover) |
Fri, 18 Sep, 01:04 |
| Mesos ReviewBot |
Re: Review Request 38443: Added layerid information to ManifestResponse |
Fri, 18 Sep, 02:01 |
| Mesos ReviewBot |
Re: Review Request 38124: Add V1 Support for QuiesceOffers |
Fri, 18 Sep, 05:12 |
| Mesos ReviewBot |
Re: Review Request 38158: Refactored Value::Ranges coalesce(). |
Fri, 18 Sep, 06:05 |
| Mesos ReviewBot |
Re: Review Request 38158: Refactored Value::Ranges coalesce(). |
Fri, 18 Sep, 07:26 |
| Mesos ReviewBot |
Re: Review Request 38051: Only update the task status when its old status is not terminal. |
Fri, 18 Sep, 08:24 |
| Mesos ReviewBot |
Re: Review Request 38481: Fix UUID::random compile error in OSX. |
Fri, 18 Sep, 10:01 |
| Mesos ReviewBot |
Re: Review Request 38491: Logged requests related to dynamic reservations. |
Fri, 18 Sep, 15:31 |
| Mesos ReviewBot |
Re: Review Request 38470: Maintenance Primitives: Fix error in master's Accept/Decline for inverse offers. |
Fri, 18 Sep, 17:54 |
| Mesos ReviewBot |
Re: Review Request 38499: Tests for executor endpoint on agent |
Fri, 18 Sep, 18:53 |
| Mesos ReviewBot |
Re: Review Request 38347: Add HELP Message for Reserve/Unreserve endpoint |
Fri, 18 Sep, 19:11 |
| Mesos ReviewBot |
Re: Review Request 37585: Maintenance primitives: Add a user doc. |
Fri, 18 Sep, 20:30 |
| Mesos ReviewBot |
Re: Review Request 38475: Maintenance Primitives: Add test for inverse offer filters. |
Fri, 18 Sep, 23:33 |
| Mesos ReviewBot |
Re: Review Request 38074: Calculate schedule latency with trace events |
Sat, 19 Sep, 00:24 |
| Mesos ReviewBot |
Re: Review Request 38347: Add HELP Message for Reserve/Unreserve endpoint |
Sat, 19 Sep, 01:16 |
| Mesos ReviewBot |
Re: Review Request 38511: Added a warning comment to the copy provisioner backend. |
Sat, 19 Sep, 02:12 |