| Zhitao Li |
Re: Review Request 56721: Implemented pruneImages with a mark and sweep in docker store. |
Tue, 26 Sep, 18:15 |
| Mesos Reviewbot Windows |
Re: Review Request 62554: Added the ability to prune the gone agent list from the registry. |
Tue, 26 Sep, 18:23 |
| Vinod Kone |
Re: Review Request 62475: Added `MARK_AGENT_GONE` call to the v1 Master API. |
Tue, 26 Sep, 18:30 |
| Vinod Kone |
Re: Review Request 62477: Added the gone agents to the master registry. |
Tue, 26 Sep, 18:38 |
| Vinod Kone |
Re: Review Request 62477: Added the gone agents to the master registry. |
Tue, 26 Sep, 18:42 |
| Vinod Kone |
Re: Review Request 62581: Added `__removeSlave` function and made `_markUnreachable` use it. |
Tue, 26 Sep, 18:43 |
| Vinod Kone |
Re: Review Request 62478: Added the mark agent gone handler on the master. |
Tue, 26 Sep, 18:46 |
| Vinod Kone |
Re: Review Request 62479: Removed the logic for removing the latest symlink on the agent. |
Tue, 26 Sep, 18:52 |
| Vinod Kone |
Re: Review Request 62481: Fixed tests impacted by no longer removing the agent symlink. |
Tue, 26 Sep, 18:58 |
| Vinod Kone |
Re: Review Request 62479: Removed the logic for removing the latest symlink on the agent. |
Tue, 26 Sep, 18:58 |
| Vinod Kone |
Re: Review Request 62554: Added the ability to prune the gone agent list from the registry. |
Tue, 26 Sep, 19:02 |
| Zhitao Li |
Re: Review Request 55334: Checkpoint and recover `ContainerConfig` in Mesos containerizer. |
Tue, 26 Sep, 19:09 |
| Zhitao Li |
Re: Review Request 55335: Backfilled required fields in TaskInfo in MesosContainerizer* tests. |
Tue, 26 Sep, 20:14 |
| Mesos Reviewbot Windows |
Re: Review Request 62579: [WIP] Added a test `BlkioIsolatorTest.ROOT_BlkioUsage`. |
Tue, 26 Sep, 20:19 |
| Anand Mazumdar |
Re: Review Request 62554: Added the ability to prune the gone agent list from the registry. |
Tue, 26 Sep, 20:23 |
| Anand Mazumdar |
Re: Review Request 62477: Added the gone agents to the master registry. |
Tue, 26 Sep, 20:25 |
| Anand Mazumdar |
Re: Review Request 62554: Added the ability to prune the gone agent list from the registry. |
Tue, 26 Sep, 20:30 |
| Vinod Kone |
Re: Review Request 62554: Added the ability to prune the gone agent list from the registry. |
Tue, 26 Sep, 20:43 |
| Anand Mazumdar |
Re: Review Request 62481: Fixed tests impacted by no longer removing the agent symlink. |
Tue, 26 Sep, 20:45 |
| Anand Mazumdar |
Re: Review Request 62481: Fixed tests impacted by no longer removing the agent symlink. |
Tue, 26 Sep, 20:45 |
| Chun-Hung Hsiao |
Re: Review Request 62230: Avoid GC pruning events from blocking other processes. |
Tue, 26 Sep, 22:04 |
| Mesos Reviewbot Windows |
Re: Review Request 62554: Added the ability to prune the gone agent list from the registry. |
Tue, 26 Sep, 22:11 |
| Mesos Reviewbot Windows |
Re: Review Request 60511: Added MockMetadataManager and test concurrent prune and pull. |
Tue, 26 Sep, 22:22 |
| Till Toenshoff |
Review Request 62587: Added HTTP authenticatee interface definition. |
Tue, 26 Sep, 22:54 |
| Till Toenshoff |
Review Request 62591: Modularized HTTP authenticatee. |
Tue, 26 Sep, 22:54 |
| Till Toenshoff |
Review Request 62592: Added basic HTTP authenticatee implementation. |
Tue, 26 Sep, 22:54 |
| Till Toenshoff |
Review Request 62594: Updated scheduler library for modularized HTTP authenticatee use. |
Tue, 26 Sep, 22:54 |
| Till Toenshoff |
Review Request 62595: Added test module wrapping the basic HTTP authenticatee. |
Tue, 26 Sep, 22:54 |
| Jiang Yan Xu |
Re: Review Request 62230: Avoid GC pruning events from blocking other processes. |
Tue, 26 Sep, 23:51 |
| Mesos Reviewbot Windows |
Re: Review Request 62595: Added test module wrapping the basic HTTP authenticatee. |
Wed, 27 Sep, 00:53 |
| Benjamin Hindman |
Review Request 62609: Protect against Future callbacks deleting instance. |
Wed, 27 Sep, 03:16 |
| Benjamin Mahler |
Re: Review Request 62609: Protect against Future callbacks deleting instance. |
Wed, 27 Sep, 03:58 |
| Mesos Reviewbot Windows |
Re: Review Request 62609: Protect against Future callbacks deleting instance. |
Wed, 27 Sep, 05:21 |
| Qian Zhang |
Re: Review Request 62579: Added a test `BlkioIsolatorTest.ROOT_BlkioUsage`. |
Wed, 27 Sep, 06:24 |
| Mesos Reviewbot Windows |
Re: Review Request 62579: Added a test `BlkioIsolatorTest.ROOT_BlkioUsage`. |
Wed, 27 Sep, 08:21 |
| Jan Schlicht |
Re: Review Request 61682: Fixed usage of 'ATOMIC_FLAG_INIT'. |
Wed, 27 Sep, 08:48 |
| Qian Zhang |
Re: Review Request 62518: Fixed default executor handling of nested container status. |
Wed, 27 Sep, 09:03 |
| Armand Grillet |
Re: Review Request 62067: Added 'mesos container list' command to CLI. |
Wed, 27 Sep, 12:39 |
| Armand Grillet |
Re: Review Request 62065: Added 'mesos agent list' command to CLI. |
Wed, 27 Sep, 12:50 |
| Kevin Klues |
Re: Review Request 62067: Added 'mesos container list' command to CLI. |
Wed, 27 Sep, 12:53 |
| Mesos Reviewbot Windows |
Re: Review Request 62065: Added 'mesos agent list' command to CLI. |
Wed, 27 Sep, 13:01 |
| Mesos Reviewbot Windows |
Re: Review Request 62067: Added 'mesos container list' command to CLI. |
Wed, 27 Sep, 13:01 |
| Kevin Klues |
Re: Review Request 62065: Added 'mesos agent list' command to CLI. |
Wed, 27 Sep, 13:06 |
| Kapil Arya |
Review Request 62616: [WIP] Binary packaging for Centos 7. |
Wed, 27 Sep, 13:09 |
| Kapil Arya |
Re: Review Request 62616: [WIP] Binary packaging for CentOS. |
Wed, 27 Sep, 13:09 |
| Kevin Klues |
Re: Review Request 62214: Added JavaScript linter. |
Wed, 27 Sep, 13:29 |
| Mesos Reviewbot Windows |
Re: Review Request 62616: [WIP] Binary packaging for CentOS. |
Wed, 27 Sep, 15:18 |
| Gastón Kleiman |
Re: Review Request 62168: Added a test using a file URI and the DefaultExecutor. |
Wed, 27 Sep, 19:07 |
| Mesos Reviewbot Windows |
Re: Review Request 62168: Added a test using a file URI and the DefaultExecutor. |
Wed, 27 Sep, 19:31 |
| Benjamin Mahler |
Re: Review Request 52284: Implemented more quota validation tests and validate duplicate name. |
Wed, 27 Sep, 20:57 |
| Greg Mann |
Re: Review Request 62592: Added basic HTTP authenticatee implementation. |
Wed, 27 Sep, 21:21 |
| Till Toenshoff |
Re: Review Request 62592: Added basic HTTP authenticatee implementation. |
Wed, 27 Sep, 21:24 |
| Kapil Arya |
Re: Review Request 62616: Binary packaging for Centos 7. |
Wed, 27 Sep, 21:50 |
| Gaston Kleiman |
Review Request 62632: Added a test using Docker, a file URI, and the DefaultExecutor. |
Wed, 27 Sep, 21:50 |
| Greg Mann |
Re: Review Request 62594: Updated scheduler library for modularized HTTP authenticatee use. |
Wed, 27 Sep, 22:07 |
| Greg Mann |
Re: Review Request 62595: Added test module wrapping the basic HTTP authenticatee. |
Wed, 27 Sep, 22:17 |
| Greg Mann |
Re: Review Request 62587: Added HTTP authenticatee interface definition. |
Wed, 27 Sep, 22:18 |
| James Peach |
Review Request 62638: Removed support for platforms without O_CLOEXEC. |
Wed, 27 Sep, 22:20 |
| Greg Mann |
Re: Review Request 62591: Modularized HTTP authenticatee. |
Wed, 27 Sep, 22:20 |
| Greg Mann |
Re: Review Request 59746: Stopped accounting aborted container launches as failures. |
Wed, 27 Sep, 22:31 |
| Mesos Reviewbot Windows |
Re: Review Request 62616: Binary packaging for Centos 7. |
Wed, 27 Sep, 23:51 |
| Mesos Reviewbot Windows |
Re: Review Request 62632: Added a test using Docker, a file URI, and the DefaultExecutor. |
Wed, 27 Sep, 23:52 |
| Andrew Schwartzmeyer |
Re: Review Request 62638: Removed support for platforms without O_CLOEXEC. |
Wed, 27 Sep, 23:56 |
| James Peach |
Review Request 62648: Added a test for nested container resource limitations. |
Thu, 28 Sep, 00:19 |
| James Peach |
Review Request 62647: Show resource limitations in mesos-execute. |
Thu, 28 Sep, 00:19 |
| James Peach |
Review Request 62646: Propagated new wait response fields from the default executor. |
Thu, 28 Sep, 00:19 |
| James Peach |
Review Request 62645: Propagated new wait response fields from the default executor. |
Thu, 28 Sep, 00:19 |
| James Peach |
Review Request 62644: Added resources to the TaskStatus message. |
Thu, 28 Sep, 00:19 |
| James Peach |
Review Request 62643: Propagated the container termination to the agent API. |
Thu, 28 Sep, 00:19 |
| James Peach |
Review Request 62642: Propagated the termination info down the container tree. |
Thu, 28 Sep, 00:20 |
| James Peach |
Review Request 62641: Removed multiple reasons from ContainerTermination. |
Thu, 28 Sep, 00:20 |
| James Peach |
Review Request 62640: Stopped keeping multiple limitations in MesosContainerizer. |
Thu, 28 Sep, 00:20 |
| Mesos Reviewbot Windows |
Re: Review Request 62638: Removed support for platforms without O_CLOEXEC. |
Thu, 28 Sep, 01:49 |
| Mesos Reviewbot Windows |
Re: Review Request 62648: Added a test for nested container resource limitations. |
Thu, 28 Sep, 03:44 |
| Mesos Reviewbot Windows |
Re: Review Request 62645: Propagated new wait response fields from the default executor. |
Thu, 28 Sep, 05:36 |
| Benjamin Bannier |
Review Request 62655: Made 'UpdateSlaveMessage' a union of possible updates. |
Thu, 28 Sep, 06:42 |
| Benjamin Bannier |
Re: Review Request 62158: Rescinded offers possibly affected by updates to agent total resources. |
Thu, 28 Sep, 06:42 |
| Benjamin Bannier |
Re: Review Request 61183: Triggered 'UpdateSlaveMessage' when 'ResourceProviderManager' updates. |
Thu, 28 Sep, 06:42 |
| Andrei Budnik |
Review Request 62661: Added --disable-libtool-wrapper configuration to Mesos. |
Thu, 28 Sep, 11:37 |
| Armand Grillet |
Review Request 62663: Updated maximum acceptable number of similar lines in Python files. |
Thu, 28 Sep, 13:39 |
| Alexander Rukletsov |
Re: Review Request 62663: Updated maximum acceptable number of similar lines in Python files. |
Thu, 28 Sep, 13:40 |
| Andrei Budnik |
Re: Review Request 62518: Fixed default executor handling of nested container status. |
Thu, 28 Sep, 13:47 |
| Mesos Reviewbot Windows |
Re: Review Request 62661: Added --disable-libtool-wrapper configuration to Mesos. |
Thu, 28 Sep, 13:50 |
| Andrei Budnik |
Re: Review Request 62381: Removed `docker exec` when performing health checks in docker executor. |
Thu, 28 Sep, 14:54 |
| Benjamin Bannier |
Re: Review Request 62587: Added HTTP authenticatee interface definition. |
Thu, 28 Sep, 15:54 |
| Benjamin Bannier |
Re: Review Request 62661: Added --disable-libtool-wrapper configuration to Mesos. |
Thu, 28 Sep, 17:22 |
| Andrei Budnik |
Re: Review Request 62661: Added --disable-libtool-wrapper configuration to Mesos. |
Thu, 28 Sep, 18:24 |
| Gaston Kleiman |
Re: Review Request 62381: Removed `docker exec` when performing health checks in docker executor. |
Thu, 28 Sep, 18:39 |
| Jie Yu |
Re: Review Request 62655: Made 'UpdateSlaveMessage' a union of possible updates. |
Thu, 28 Sep, 19:14 |
| Jie Yu |
Re: Review Request 62158: Rescinded offers possibly affected by updates to agent total resources. |
Thu, 28 Sep, 19:19 |
| Mesos Reviewbot Windows |
Re: Review Request 62661: Added --disable-libtool-wrapper configuration to Mesos. |
Thu, 28 Sep, 20:20 |
| James Peach |
Re: Review Request 61473: Do not kill non partition aware tasks. |
Thu, 28 Sep, 21:32 |
| Mesos Reviewbot Windows |
Re: Review Request 62638: Removed support for platforms without O_CLOEXEC. |
Thu, 28 Sep, 22:19 |
| Qian Zhang |
Re: Review Request 61109: Used the default value when parsing an optional enum field from JSON. |
Fri, 29 Sep, 00:24 |
| Qian Zhang |
Re: Review Request 61174: Added a test `ProtobufTest.ParseJSONOptionalEnum`. |
Fri, 29 Sep, 00:26 |
| Mesos Reviewbot |
Re: Review Request 62616: Binary packaging for Centos 7. |
Fri, 29 Sep, 01:20 |
| Jie Yu |
Re: Review Request 61183: Triggered 'UpdateSlaveMessage' when 'ResourceProviderManager' updates. |
Fri, 29 Sep, 03:33 |
| Mesos Reviewbot Windows |
Re: Review Request 61174: Added a test `ProtobufTest.ParseJSONOptionalEnum`. |
Fri, 29 Sep, 03:42 |
| Qian Zhang |
Re: Review Request 62326: Always send TASK_KILLED when the task is killed. |
Fri, 29 Sep, 06:55 |
| Qian Zhang |
Review Request 62685: Updated the comments of TASK_FINISHED to make it more clear. |
Fri, 29 Sep, 07:32 |