reviews
Thread
Date
Later messages
Messages by Thread
Re: Review Request 71955: Added a new parameter `resourceLimits` to the `createTask` methods.
Greg Mann
Re: Review Request 71955: Added a new parameter `resourceLimits` to the `createTask` methods.
Qian Zhang
Review Request 72217: Added tests for resource limit and shared cgroup validation.
Greg Mann
Re: Review Request 72217: Added tests for resource limit and shared cgroup validation.
Mesos Reviewbot
Re: Review Request 72217: Added tests for resource limit and shared cgroup validation.
Greg Mann
Re: Review Request 72217: Added tests for resource limit and shared cgroup validation.
Mesos Reviewbot
Re: Review Request 72217: Added tests for resource limit and shared cgroup validation.
Andrei Budnik
Re: Review Request 72217: Added tests for master validation of limits and shared cgroups.
Greg Mann
Re: Review Request 72217: Added tests for master validation of limits and shared cgroups.
Greg Mann
Re: Review Request 72217: Added tests for master validation of limits and shared cgroups.
Qian Zhang
Re: Review Request 72217: Added tests for master validation of limits and shared cgroups.
Greg Mann
Review Request 72216: Added master validation for task resource limits and shared cgroups.
Greg Mann
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Andrei Budnik
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Greg Mann
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Greg Mann
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Greg Mann
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Greg Mann
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Greg Mann
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Greg Mann
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Andrei Budnik
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Greg Mann
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Greg Mann
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Qian Zhang
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Qian Zhang
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Greg Mann
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Greg Mann
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Qian Zhang
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Greg Mann
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Greg Mann
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Greg Mann
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Greg Mann
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Greg Mann
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Qian Zhang
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Greg Mann
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Greg Mann
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Greg Mann
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Greg Mann
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Greg Mann
Re: Review Request 72216: Added master validation for task resource limits and shared cgroups.
Qian Zhang
Review Request 72215: Added resource limits to one overload of 'createTask()' helper.
Greg Mann
Re: Review Request 72215: Added resource limits to one overload of 'createTask()' helper.
Greg Mann
Review Request 72211: Added a test `ROOT_DOCKER_CGROUPS_CFS_CommandTaskInfiniteLimits`.
Qian Zhang
Re: Review Request 72211: Added a test `ROOT_DOCKER_CGROUPS_CFS_CommandTaskInfiniteLimits`.
Qian Zhang
Re: Review Request 72211: Added a test `ROOT_DOCKER_CGROUPS_CFS_CommandTaskInfiniteLimits`.
Greg Mann
Re: Review Request 72211: Added a test `ROOT_DOCKER_CGROUPS_CFS_CommandTaskInfiniteLimits`.
Qian Zhang
Re: Review Request 72211: Added a test `ROOT_DOCKER_CGROUPS_CFS_CommandTaskInfiniteLimits`.
Qian Zhang
Re: Review Request 72211: Added a test `ROOT_DOCKER_CGROUPS_CFS_CommandTaskInfiniteLimits`.
Qian Zhang
Re: Review Request 72211: Added a test `ROOT_DOCKER_CGROUPS_CFS_CommandTaskInfiniteLimits`.
Greg Mann
Review Request 72210: Added a test `ROOT_CGROUPS_CFS_CommandTaskInfiniteLimits`.
Qian Zhang
Re: Review Request 72210: Added a test `ROOT_CGROUPS_CFS_CommandTaskInfiniteLimits`.
Qian Zhang
Re: Review Request 72210: Added a test `ROOT_CGROUPS_CFS_CommandTaskInfiniteLimits`.
Greg Mann
Re: Review Request 72210: Added a test `ROOT_CGROUPS_CFS_CommandTaskInfiniteLimits`.
Qian Zhang
Re: Review Request 72210: Added a test `ROOT_CGROUPS_CFS_CommandTaskInfiniteLimits`.
Qian Zhang
Re: Review Request 72210: Added a test `ROOT_CGROUPS_CFS_CommandTaskInfiniteLimits`.
Qian Zhang
Re: Review Request 72210: Added a test `ROOT_CGROUPS_CFS_CommandTaskInfiniteLimits`.
Greg Mann
Re: Review Request 72027: Added a new test `ROOT_DOCKER_CGROUPS_CFS_CommandTaskLimits`.
Qian Zhang
Re: Review Request 72027: Added a new test `ROOT_DOCKER_CGROUPS_CFS_CommandTaskLimits`.
Qian Zhang
Re: Review Request 72027: Added a new test `ROOT_DOCKER_CGROUPS_CFS_CommandTaskLimits`.
Qian Zhang
Re: Review Request 72027: Added a new test `ROOT_DOCKER_CGROUPS_CFS_CommandTaskLimits`.
Qian Zhang
Review Request 72208: Ensured that MasterAPITest.ReservationUpdate waits for changes on agent.
Andrei Sekretenko
Re: Review Request 72208: Ensured that MasterAPITest.ReservationUpdate waits for changes on agent.
Benjamin Mahler
Re: Review Request 72208: Ensured that MasterAPITest.ReservationUpdate waits for changes on agent.
Mesos Reviewbot
[GitHub] [mesos] wavesoft opened a new pull request #351: Ensure that long CNI labels are short enough
GitBox
Re: Review Request 72040: Updated the `LaunchContainer` agent API to support resource limits.
Greg Mann
Re: Review Request 72040: Updated the `LaunchContainer` agent API to support resource limits.
Qian Zhang
Re: Review Request 72040: Updated the `LaunchContainer` agent API to support resource limits.
Qian Zhang
Re: Review Request 72027: Added a new test `ROOT_DOCKER_CGROUPS_CFS_CgroupsEnableCFS`.
Greg Mann
Re: Review Request 72027: Added a new test `ROOT_DOCKER_CGROUPS_CFS_CommandTaskLimits`.
Qian Zhang
Re: Review Request 72121: Moved cgroup path helpers to `paths.hpp`.
Qian Zhang
Re: Review Request 72022: Set resource limits and OOM score adjustment in Docker executor.
Greg Mann
Re: Review Request 72022: Set resource limits and OOM score adjustment in Docker executor.
Qian Zhang
Re: Review Request 72022: Set resource limits and OOM score adjustment in Docker executor.
Greg Mann
Re: Review Request 72022: Set resource limits and OOM score adjustment in Docker executor.
Qian Zhang
Re: Review Request 72022: Set resource limits and OOM score adjustment in Docker executor.
Qian Zhang
Re: Review Request 72022: Set resource limits and OOM score adjustment in Docker executor.
Greg Mann
Re: Review Request 72022: Set resource limits and OOM score adjustment in Docker executor.
Qian Zhang
Re: Review Request 72022: Set resource limits and OOM score adjustment in Docker executor.
Greg Mann
Re: Review Request 72022: Set resource limits and OOM score adjustment in Docker executor.
Greg Mann
Review Request 72202: Fixed the broken PathTest.PathIteration on windows.
Benjamin Mahler
Re: Review Request 72202: Fixed the broken PathTest.PathIteration on windows.
Mesos Reviewbot
Re: Review Request 72202: Fixed the broken PathTest.PathIteration on windows.
Andrei Sekretenko
Review Request 72201: Fixed the broken PathTest.Relative on Windows.
Benjamin Mahler
Re: Review Request 72201: Fixed the broken PathTest.Relative on Windows.
Andrei Sekretenko
Re: Review Request 71983: Added a test `CgroupsIsolatorTest.ROOT_CGROUPS_CFS_TaskGroupLimits`.
Greg Mann
Re: Review Request 71983: Added a test `CgroupsIsolatorTest.ROOT_CGROUPS_CFS_TaskGroupLimits`.
Qian Zhang
Re: Review Request 71983: Added a test `CgroupsIsolatorTest.ROOT_CGROUPS_CFS_TaskGroupLimits`.
Qian Zhang
Re: Review Request 71983: Added a test `CgroupsIsolatorTest.ROOT_CGROUPS_CFS_TaskGroupLimits`.
Qian Zhang
Re: Review Request 71983: Added a test `CgroupsIsolatorTest.ROOT_CGROUPS_CFS_TaskGroupLimits`.
Qian Zhang
Re: Review Request 71983: Added a test `CgroupsIsolatorTest.ROOT_CGROUPS_CFS_TaskGroupLimits`.
Qian Zhang
Re: Review Request 71983: Added a test `CgroupsIsolatorTest.ROOT_CGROUPS_CFS_TaskGroupLimits`.
Greg Mann
Re: Review Request 71983: Added a test `CgroupsIsolatorTest.ROOT_CGROUPS_CFS_TaskGroupLimits`.
Qian Zhang
Re: Review Request 71983: Added a test `CgroupsIsolatorTest.ROOT_CGROUPS_CFS_TaskGroupLimits`.
Greg Mann
Re: Review Request 71983: Added a test `CgroupsIsolatorTest.ROOT_CGROUPS_CFS_TaskGroupLimits`.
Greg Mann
Re: Review Request 71983: Added a test `CgroupsIsolatorTest.ROOT_CGROUPS_CFS_TaskGroupLimits`.
Qian Zhang
Re: Review Request 71956: Added a test `ROOT_CGROUPS_CFS_CommandTaskLimits`.
Greg Mann
Re: Review Request 71956: Added a test `ROOT_CGROUPS_CFS_CommandTaskLimits`.
Qian Zhang
Re: Review Request 71956: Added a test `ROOT_CGROUPS_CFS_CommandTaskLimits`.
Qian Zhang
Re: Review Request 71956: Added a test `ROOT_CGROUPS_CFS_CommandTaskLimits`.
Qian Zhang
Re: Review Request 71956: Added a test `ROOT_CGROUPS_CFS_CommandTaskLimits`.
Qian Zhang
Re: Review Request 71956: Added a test `ROOT_CGROUPS_CFS_CommandTaskLimits`.
Qian Zhang
Re: Review Request 71953: Updated the test `ROOT_CGROUPS_CFS_EnableCfs` to check CFS quota.
Greg Mann
Re: Review Request 71953: Updated the test `ROOT_CGROUPS_CFS_EnableCfs` to check CFS quota.
Qian Zhang
Re: Review Request 71953: Updated the test `ROOT_CGROUPS_CFS_EnableCfs` to check CFS quota.
Greg Mann
Re: Review Request 71955: Add a new parameter `resourceLimits` to the `createTask` methods.
Greg Mann
Re: Review Request 71955: Add a new parameter `resourceLimits` to the `createTask` methods.
Greg Mann
Re: Review Request 71955: Added a new parameter `resourceLimits` to the `createTask` methods.
Qian Zhang
Re: Review Request 71952: Set resource limits when updating executor container.
Greg Mann
Re: Review Request 71952: Set resource limits when updating executor container.
Greg Mann
Re: Review Request 71952: Set resource limits when updating executor container.
Qian Zhang
Re: Review Request 71952: Set resource limits when updating executor container.
Greg Mann
Re: Review Request 71952: Set resource limits when updating executor container.
Qian Zhang
Re: Review Request 71951: Added resource limits into the `Task` protobuf message.
Greg Mann
Re: Review Request 71951: Added resource limits into the `Task` protobuf message.
Qian Zhang
Re: Review Request 71951: Added resource limits into the `Task` protobuf message.
Greg Mann
Re: Review Request 71951: Added resource limits into the `Task` protobuf message.
Qian Zhang
Re: Review Request 71943: Set container's `memory.limit_in_bytes` to its memory limit.
Greg Mann
Re: Review Request 71943: Set container's `memory.limit_in_bytes` to its memory limit.
Qian Zhang
Re: Review Request 71943: Set container's `memory.limit_in_bytes` to its memory limit.
Qian Zhang
Re: Review Request 71943: Set container's `memory.limit_in_bytes` to its memory limit.
Greg Mann
Re: Review Request 71943: Set container's `memory.limit_in_bytes` to its memory limit.
Qian Zhang
Re: Review Request 71886: Set container's `cpu.cfs_quota_us` to its CPU resource limit.
Greg Mann
Re: Review Request 71886: Set container's `cpu.cfs_quota_us` to its CPU resource limit.
Qian Zhang
Re: Review Request 71886: Set container's `cpu.cfs_quota_us` to its CPU resource limit.
Qian Zhang
Re: Review Request 71885: Updated the `update()` method of subsystem to handle resource limits.
Greg Mann
Re: Review Request 71885: Updated the `update()` method of subsystem to handle resource limits.
Qian Zhang
Re: Review Request 71884: Updated the `update()` method of isolator to handle resource limits.
Greg Mann
Re: Review Request 71884: Updated the `update()` method of isolator to handle resource limits.
Qian Zhang
Review Request 72196: Removed stale MPI framework.
Vinod Kone
Re: Review Request 72196: Removed stale MPI framework.
Benjamin Mahler
Re: Review Request 72162: Accommodated the "Infinity" value in the JSON <-> Protobuf conversion.
Greg Mann
Re: Review Request 72162: Accommodated the "Infinity" value in the JSON <-> Protobuf conversion.
Qian Zhang
Re: Review Request 72162: Accommodated the "Infinity" value in the JSON <-> Protobuf conversion.
Qian Zhang
Re: Review Request 72161: Added patch for RapidJSON.
Greg Mann
Re: Review Request 72161: Added patch for RapidJSON.
Greg Mann
Re: Review Request 72161: Added patch for RapidJSON.
Qian Zhang
Re: Review Request 72161: Added patch for RapidJSON.
Qian Zhang
Re: Review Request 72161: Added patch for RapidJSON.
Greg Mann
Re: Review Request 72161: Added patch for RapidJSON.
Qian Zhang
Re: Review Request 71965: Cgroups isolator: added support for nested cgroups during launch.
Andrei Budnik
Re: Review Request 71965: Cgroups isolator: added support for nested cgroups during launch.
Andrei Budnik
Re: Review Request 71965: Cgroups isolator: added support for nested cgroups during launch.
Qian Zhang
Re: Review Request 71965: Cgroups isolator: added support for nested cgroups during launch.
Greg Mann
Re: Review Request 71965: Cgroups isolator: added support for nested cgroups during launch.
Qian Zhang
Re: Review Request 71965: Cgroups isolator: added support for nested cgroups during launch.
Andrei Budnik
Re: Review Request 71965: Cgroups isolator: added support for nested cgroups during launch.
Greg Mann
Re: Review Request 71965: Cgroups isolator: added support for nested cgroups during launch.
Qian Zhang
Re: Review Request 71965: Cgroups isolator: added support for nested cgroups during launch.
Qian Zhang
Re: Review Request 71965: Cgroups isolator: added support for nested cgroups during launch.
Greg Mann
Re: Review Request 71965: Cgroups isolator: added support for nested cgroups during launch.
Greg Mann
Re: Review Request 71965: Cgroups isolator: added support for nested cgroups during launch.
Andrei Budnik
Re: Review Request 71965: Cgroups isolator: added support for nested cgroups during launch.
Andrei Budnik
Re: Review Request 71965: Cgroups isolator: added support for nested cgroups during launch.
Qian Zhang
Re: Review Request 71965: Cgroups isolator: added support for nested cgroups during launch.
Qian Zhang
Re: Review Request 71965: Cgroups isolator: added support for nested cgroups during launch.
Qian Zhang
Re: Review Request 71965: Cgroups isolator: added support for nested cgroups during launch.
Qian Zhang
Review Request 72190: Added a test `LaunchNestedShareCgroups`.
Andrei Budnik
Re: Review Request 72190: Added a test `LaunchNestedShareCgroups`.
Mesos Reviewbot
Re: Review Request 72190: Added a test `LaunchNestedShareCgroups`.
Mesos Reviewbot
Re: Review Request 72190: Added a test `LaunchNestedShareCgroups`.
Mesos Reviewbot
Re: Review Request 72190: Added a test `LaunchNestedShareCgroups`.
Qian Zhang
Re: Review Request 72190: Added a test `LaunchNestedShareCgroups`.
Mesos Reviewbot
Review Request 72189: Updated nested mesos containerizer tests to support `shared_cgroups`.
Andrei Budnik
Re: Review Request 72189: Updated nested mesos containerizer tests to support `shared_cgroups`.
Qian Zhang
Re: Review Request 72189: Updated nested mesos containerizer tests to support `shared_cgroups`.
Greg Mann
Re: Review Request 72041: Updated default executor to call the `LaunchContainer` agent API.
Mesos Reviewbot
Re: Review Request 72041: Updated default executor to call the `LaunchContainer` agent API.
Qian Zhang
Re: Review Request 72041: Updated default executor to call the `LaunchContainer` agent API.
Mesos Reviewbot
Re: Review Request 72041: Updated default executor to call the `LaunchContainer` agent API.
Mesos Reviewbot
Re: Review Request 72041: Updated default executor to call the `LaunchContainer` agent API.
Greg Mann
Re: Review Request 72041: Updated default executor to call the `LaunchContainer` agent API.
Mesos Reviewbot
Re: Review Request 72041: Updated default executor to call the `LaunchContainer` agent API.
Mesos Reviewbot
Re: Review Request 72041: Updated default executor to call the `LaunchContainer` agent API.
Mesos Reviewbot
Re: Review Request 72041: Updated default executor to call the `LaunchContainer` agent API.
Mesos Reviewbot
Re: Review Request 72041: Updated default executor to call the `LaunchContainer` agent API.
Mesos Reviewbot
Re: Review Request 72041: Updated default executor to call the `LaunchContainer` agent API.
Mesos Reviewbot
Re: Review Request 72041: Updated default executor to call the `LaunchContainer` agent API.
Mesos Reviewbot
Re: Review Request 72041: Updated default executor to call the `LaunchContainer` agent API.
Mesos Reviewbot
Re: Review Request 72041: Updated default executor to call the `LaunchContainer` agent API.
Qian Zhang
Re: Review Request 72041: Updated default executor to call the `LaunchContainer` agent API.
Mesos Reviewbot
Re: Review Request 72041: Updated default executor to call the `LaunchContainer` agent API.
Mesos Reviewbot
Re: Review Request 72041: Updated default executor to call the `LaunchContainer` agent API.
Mesos Reviewbot
Re: Review Request 72041: Updated default executor to call the `LaunchContainer` agent API.
Mesos Reviewbot
Re: Review Request 72041: Updated default executor to call the `LaunchContainer` agent API.
Mesos Reviewbot
Re: Review Request 71858: Set resource limits when launching executor container.
Qian Zhang
Re: Review Request 71858: Set resource limits when launching executor container.
Greg Mann
Re: Review Request 71858: Set resource limits when launching executor container.
Greg Mann
Re: Review Request 71858: Set resource limits when launching executor container.
Qian Zhang
Re: Review Request 71858: Set resource limits when launching executor container.
Greg Mann
Re: Review Request 71858: Set resource limits when launching executor container.
Greg Mann
Re: Review Request 71858: Set resource limits when launching executor container.
Qian Zhang
Re: Review Request 71858: Set resource limits when launching executor container.
Greg Mann
Re: Review Request 71858: Set resource limits when launching executor container.
Qian Zhang
Re: Review Request 71950: Updated containerizer's `update()` method to handle resource limits.
Qian Zhang
Re: Review Request 71950: Updated containerizer's `update()` method to handle resource limits.
Greg Mann
Re: Review Request 71950: Updated containerizer's `update()` method to handle resource limits.
Greg Mann
Re: Review Request 71950: Updated containerizer's `update()` method to handle resource limits.
Qian Zhang
Re: Review Request 71950: Updated containerizer's `update()` method to handle resource limits.
Qian Zhang
Re: Review Request 72187: Fixed build on platforms not treating initializer_list<enum> as literal.
Andrei Sekretenko
Re: Review Request 72187: Fixed build on platforms not treating initializer_list<enum> as literal.
Benjamin Mahler
Re: Review Request 72187: Fixed build on platforms not treating initializer_list<enum> as literal.
Mesos Reviewbot
Later messages