| Qian Zhang |
Review Request 71943: WIP: Set container's `memory.limit_in_bytes` to its memory limit. |
Thu, 02 Jan, 01:14 |
| Qian Zhang |
Review Request 71944: Set container process's OOM score adjust. |
Thu, 02 Jan, 01:15 |
| Mesos Reviewbot |
Re: Review Request 71944: Set container process's OOM score adjust. |
Thu, 02 Jan, 04:07 |
| Andrei Sekretenko |
Re: Review Request 71859: Moved creating authorization Object out of `Master::authorizeTask`. |
Thu, 02 Jan, 20:00 |
| Andrei Sekretenko |
Re: Review Request 71860: Moved creating authorization Object out of `Master::authorizeFramework`. |
Thu, 02 Jan, 20:01 |
| Andrei Sekretenko |
Re: Review Request 71861: Moved creating authz Object out of `Master::authorizeResizeVolume`. |
Thu, 02 Jan, 20:01 |
| Andrei Sekretenko |
Re: Review Request 71862: Moved creating authorization Object out of `Master::authorize*Disk`. |
Thu, 02 Jan, 20:01 |
| Andrei Sekretenko |
Re: Review Request 71863: Moved creating authorization Object out of `Master::authorize*Volume`. |
Thu, 02 Jan, 20:01 |
| Andrei Sekretenko |
Re: Review Request 71864: Moved creating authorization Object out of `Master::authorize*reserve`. |
Thu, 02 Jan, 20:01 |
| Andrei Sekretenko |
Re: Review Request 71865: Moved creating authorization Object out of `Master::authorizeSlave`. |
Thu, 02 Jan, 20:01 |
| Andrei Sekretenko |
Re: Review Request 71859: Moved creating authorization Object out of `Master::authorizeTask`. |
Thu, 02 Jan, 20:06 |
| Andrei Sekretenko |
Re: Review Request 71859: Moved creating authorization Object out of `Master::authorizeTask`. |
Thu, 02 Jan, 20:07 |
| Andrei Sekretenko |
Re: Review Request 71859: Moved creating authorization Object out of `Master::authorizeTask`. |
Thu, 02 Jan, 20:10 |
| Andrei Sekretenko |
Re: Review Request 71861: Moved creating authz Object out of `Master::authorizeResizeVolume`. |
Thu, 02 Jan, 20:16 |
| Andrei Sekretenko |
Re: Review Request 71863: Moved creating authorization Object out of `Master::authorize*Volume`. |
Thu, 02 Jan, 20:37 |
| Andrei Sekretenko |
Re: Review Request 71864: Moved creating authorization Object out of `Master::authorize*reserve`. |
Thu, 02 Jan, 20:58 |
| Mesos Reviewbot |
Re: Review Request 71865: Moved creating authorization Object out of `Master::authorizeSlave`. |
Thu, 02 Jan, 22:50 |
| Benno Evers |
Review Request 71947: Handled embedded null bytes in abstract domain socket names. |
Fri, 03 Jan, 01:38 |
| Mesos Reviewbot |
Re: Review Request 71947: Handled embedded null bytes in abstract domain socket names. |
Fri, 03 Jan, 04:27 |
| Andrei Sekretenko |
Re: Review Request 71906: Updated protobuf -> JSON mapping to emit empty repeated fields / maps. |
Fri, 03 Jan, 13:33 |
| Andrei Sekretenko |
Re: Review Request 71907: Adjusted mesos tests to expect empty arrays/maps in protobuf json. |
Fri, 03 Jan, 13:34 |
| Andrei Sekretenko |
Re: Review Request 71908: Updated protobuf -> JSON mapping to emit empty repeated fields / maps. |
Fri, 03 Jan, 13:52 |
| Andrei Sekretenko |
Re: Review Request 71865: Moved creating authorization Object out of `Master::authorizeSlave`. |
Fri, 03 Jan, 14:06 |
| Andrei Sekretenko |
Re: Review Request 71864: Moved creating authorization Object out of `Master::authorize*reserve`. |
Fri, 03 Jan, 14:12 |
| Mesos Reviewbot |
Re: Review Request 71865: Moved creating authorization Object out of `Master::authorizeSlave`. |
Fri, 03 Jan, 17:18 |
| GitBox |
[GitHub] [mesos] vinodkone opened a new pull request #347: Removed the link the slackarchive.io |
Fri, 03 Jan, 23:27 |
| GitBox |
[GitHub] [mesos] vinodkone merged pull request #347: Removed the link to mesos.slackarchive.io |
Fri, 03 Jan, 23:32 |
| Qian Zhang |
Re: Review Request 71884: Updated the `update()` method of isolator to handle resource limits. |
Sun, 05 Jan, 14:02 |
| Qian Zhang |
Re: Review Request 71885: Updated the `update()` method of subsystem to handle resource limits. |
Sun, 05 Jan, 14:03 |
| Qian Zhang |
Re: Review Request 71886: WIP: Set container's `cpu.cfs_quota_us` to its CPU resource limit. |
Sun, 05 Jan, 14:04 |
| Qian Zhang |
Re: Review Request 71943: WIP: Set container's `memory.limit_in_bytes` to its memory limit. |
Sun, 05 Jan, 14:04 |
| Qian Zhang |
Re: Review Request 71944: Set container process's OOM score adjust. |
Sun, 05 Jan, 14:06 |
| Qian Zhang |
Review Request 71950: Updated containierzer's `update()` method to handle resource limits. |
Sun, 05 Jan, 14:06 |
| Qian Zhang |
Review Request 71951: Added resource limits into the `Task` protobuf message. |
Sun, 05 Jan, 14:07 |
| Qian Zhang |
Review Request 71952: WIP: Set resource limits when updating executor container. |
Sun, 05 Jan, 14:08 |
| Mesos Reviewbot |
Re: Review Request 71952: WIP: Set resource limits when updating executor container. |
Sun, 05 Jan, 16:50 |
| Qian Zhang |
Review Request 71953: Updated the test `ROOT_CGROUPS_CFS_EnableCfs` to check CFS quota. |
Mon, 06 Jan, 06:21 |
| Qian Zhang |
Review Request 71955: Add a new parameter `resourceLimits` to the `createTask` methods. |
Mon, 06 Jan, 08:49 |
| Qian Zhang |
Review Request 71956: Added a test `ROOT_CGROUPS_CFS_EnableCpuLimit`. |
Mon, 06 Jan, 08:50 |
| Mesos Reviewbot |
Re: Review Request 71953: Updated the test `ROOT_CGROUPS_CFS_EnableCfs` to check CFS quota. |
Mon, 06 Jan, 09:23 |
| Mesos Reviewbot |
Re: Review Request 71956: Added a test `ROOT_CGROUPS_CFS_EnableCpuLimit`. |
Mon, 06 Jan, 12:00 |
| Benjamin Bannier |
Re: Review Request 71947: Handled embedded null bytes in abstract domain socket names. |
Mon, 06 Jan, 12:59 |
| Benno Evers |
Re: Review Request 71934: Bumped site's rack to rack-1.16.12. |
Mon, 06 Jan, 13:04 |
| Benjamin Bannier |
Review Request 71717: Made sure all targets are build for cmake `tests` target. |
Mon, 06 Jan, 13:05 |
| Till Toenshoff via Review Board |
Re: Review Request 71934: Bumped site's rack to rack-1.16.12. |
Mon, 06 Jan, 13:06 |
| Benno Evers |
Re: Review Request 71717: Made sure all targets are build for cmake `tests` target. |
Mon, 06 Jan, 13:08 |
| Benno Evers |
Re: Review Request 71538: Allowed disabling use of NVML headers. |
Mon, 06 Jan, 13:19 |
| Benjamin Bannier |
Re: Review Request 71538: Allowed disabling use of NVML headers. |
Mon, 06 Jan, 14:43 |
| Benjamin Bannier |
Re: Review Request 71538: Allowed disabling use of NVML headers. |
Mon, 06 Jan, 14:45 |
| Mesos Reviewbot |
Re: Review Request 71538: Allowed disabling use of NVML headers. |
Mon, 06 Jan, 17:13 |
| Benjamin Mahler |
Re: Review Request 71859: Moved creating authorization Object out of `Master::authorizeTask`. |
Mon, 06 Jan, 18:26 |
| Benjamin Mahler |
Re: Review Request 71859: Moved creating authorization Object out of `Master::authorizeTask`. |
Mon, 06 Jan, 18:26 |
| Benjamin Mahler |
Re: Review Request 71860: Moved creating authorization Object out of `Master::authorizeFramework`. |
Mon, 06 Jan, 18:47 |
| Benno Evers |
Review Request 71960: Added support for deprecated attribute to stout. |
Tue, 07 Jan, 12:24 |
| Benno Evers |
Review Request 71961: Added deprecated absolute() function for backwards compatibility. |
Tue, 07 Jan, 12:24 |
| Benjamin Bannier |
Re: Review Request 71960: Added support for deprecated attribute to stout. |
Tue, 07 Jan, 12:29 |
| Benjamin Bannier |
Re: Review Request 71961: Added deprecated absolute() function for backwards compatibility. |
Tue, 07 Jan, 12:29 |
| Qian Zhang |
Re: Review Request 71953: Updated the test `ROOT_CGROUPS_CFS_EnableCfs` to check CFS quota. |
Tue, 07 Jan, 15:05 |
| Qian Zhang |
Re: Review Request 71956: Added a test `ROOT_CGROUPS_CFS_CommandTaskLimits`. |
Tue, 07 Jan, 15:06 |
| Andrei Budnik |
Review Request 71965: Cgroups isolator: added support for nested cgroups during launch. |
Tue, 07 Jan, 17:06 |
| Andrei Budnik |
Review Request 71966: Cgroups isolator: added support for nested cgroups during recovery. |
Tue, 07 Jan, 17:06 |
| Benjamin Mahler |
Re: Review Request 71861: Moved creating authz Object out of `Master::authorizeResizeVolume`. |
Tue, 07 Jan, 17:22 |
| Benjamin Mahler |
Re: Review Request 71862: Moved creating authorization Object out of `Master::authorize*Disk`. |
Tue, 07 Jan, 17:28 |
| Benjamin Mahler |
Re: Review Request 71863: Moved creating authorization Object out of `Master::authorize*Volume`. |
Tue, 07 Jan, 18:09 |
| Benjamin Mahler |
Re: Review Request 71864: Moved creating authorization Object out of `Master::authorize*reserve`. |
Tue, 07 Jan, 18:11 |
| Benjamin Mahler |
Re: Review Request 71864: Moved creating authorization Object out of `Master::authorize*reserve`. |
Tue, 07 Jan, 18:42 |
| Benjamin Mahler |
Re: Review Request 71865: Moved creating authorization Object out of `Master::authorizeSlave`. |
Tue, 07 Jan, 19:01 |
| Mesos Reviewbot |
Re: Review Request 71956: Added a test `ROOT_CGROUPS_CFS_CommandTaskLimits`. |
Tue, 07 Jan, 19:17 |
| Mesos Reviewbot |
Re: Review Request 71966: Cgroups isolator: added support for nested cgroups during recovery. |
Tue, 07 Jan, 20:56 |
| Greg Mann |
Re: Review Request 71858: WIP: Set resource limits when launching executor container. |
Tue, 07 Jan, 21:43 |
| Greg Mann |
Re: Review Request 71886: WIP: Set container's `cpu.cfs_quota_us` to its CPU resource limit. |
Tue, 07 Jan, 22:11 |
| Greg Mann |
Re: Review Request 71943: WIP: Set container's `memory.limit_in_bytes` to its memory limit. |
Tue, 07 Jan, 22:34 |
| Greg Mann |
Re: Review Request 71944: Set container process's OOM score adjust. |
Tue, 07 Jan, 23:07 |
| Greg Mann |
Re: Review Request 71950: Updated containierzer's `update()` method to handle resource limits. |
Wed, 08 Jan, 00:45 |
| Greg Mann |
Re: Review Request 71950: Updated containierzer's `update()` method to handle resource limits. |
Wed, 08 Jan, 00:46 |
| Greg Mann |
Re: Review Request 71951: Added resource limits into the `Task` protobuf message. |
Wed, 08 Jan, 00:47 |
| Greg Mann |
Re: Review Request 71858: WIP: Set resource limits when launching executor container. |
Wed, 08 Jan, 01:24 |
| Qian Zhang |
Re: Review Request 71858: WIP: Set resource limits when launching executor container. |
Wed, 08 Jan, 02:08 |
| Qian Zhang |
Re: Review Request 71952: WIP: Set resource limits when updating executor container. |
Wed, 08 Jan, 02:08 |
| Qian Zhang |
Re: Review Request 71858: WIP: Set resource limits when launching executor container. |
Wed, 08 Jan, 02:31 |
| Qian Zhang |
Re: Review Request 71858: WIP: Set resource limits when launching executor container. |
Wed, 08 Jan, 02:40 |
| Qian Zhang |
Re: Review Request 71952: WIP: Set resource limits when updating executor container. |
Wed, 08 Jan, 02:40 |
| Qian Zhang |
Re: Review Request 71886: WIP: Set container's `cpu.cfs_quota_us` to its CPU resource limit. |
Wed, 08 Jan, 02:45 |
| Qian Zhang |
Re: Review Request 71943: WIP: Set container's `memory.limit_in_bytes` to its memory limit. |
Wed, 08 Jan, 02:46 |
| Mesos Reviewbot |
Re: Review Request 71956: Added a test `ROOT_CGROUPS_CFS_CommandTaskLimits`. |
Wed, 08 Jan, 05:01 |
| Benno Evers |
Review Request 71971: Added special case to stout's handling of C++ attributes. |
Wed, 08 Jan, 12:15 |
| Benjamin Bannier |
Re: Review Request 71971: Added special case to stout's handling of C++ attributes. |
Wed, 08 Jan, 13:28 |
| Benno Evers |
Re: Review Request 71971: Added special case to stout's handling of C++ attributes. |
Wed, 08 Jan, 13:45 |
| Benjamin Bannier |
Re: Review Request 71971: Added special case to stout's handling of C++ attributes. |
Wed, 08 Jan, 13:45 |
| Qian Zhang |
Re: Review Request 71944: Set container process's OOM score adjust. |
Wed, 08 Jan, 14:59 |
| Qian Zhang |
Re: Review Request 71944: Set container process's OOM score adjust. |
Wed, 08 Jan, 15:28 |
| Benno Evers |
Re: Review Request 71971: Added special case to stout's handling of C++ attributes. |
Wed, 08 Jan, 16:00 |
| Mesos Reviewbot |
Re: Review Request 71971: Added special case to stout's handling of C++ attributes. |
Wed, 08 Jan, 16:14 |
| Mesos Reviewbot |
Re: Review Request 71956: Added a test `ROOT_CGROUPS_CFS_CommandTaskLimits`. |
Wed, 08 Jan, 19:48 |
| Qian Zhang |
Re: Review Request 71950: Updated containierzer's `update()` method to handle resource limits. |
Thu, 09 Jan, 01:32 |
| Qian Zhang |
Re: Review Request 71950: Updated containerizer's `update()` method to handle resource limits. |
Thu, 09 Jan, 01:37 |
| Qian Zhang |
Re: Review Request 71950: Updated containerizer's `update()` method to handle resource limits. |
Thu, 09 Jan, 01:37 |
| Qian Zhang |
Re: Review Request 71944: Set container process's OOM score adjust. |
Thu, 09 Jan, 02:42 |
| Mesos Reviewbot |
Re: Review Request 71956: Added a test `ROOT_CGROUPS_CFS_CommandTaskLimits`. |
Thu, 09 Jan, 05:02 |
| Benjamin Mahler |
Review Request 71978: Updated all GET_* v1 master calls to be served in parallel. |
Thu, 09 Jan, 17:48 |