| Benjamin Bannier |
Review Request 52388: Added compile-time check for correct inheritance in Flags hierarchies. |
Thu, 29 Sep, 12:55 |
| Mesos ReviewBot |
Re: Review Request 52388: Added compile-time check for correct inheritance in Flags hierarchies. |
Thu, 29 Sep, 15:41 |
| Benjamin Bannier |
Review Request 52384: Correctly used variables in test shell scripts. |
Thu, 29 Sep, 12:58 |
| Alexander Rukletsov |
Re: Review Request 52384: Correctly used variables in test shell scripts. |
Thu, 29 Sep, 14:18 |
| Avinash sridharan |
Review Request 51617: Added the `remove` and `insert` methods. |
Thu, 29 Sep, 15:44 |
| Mesos ReviewBot |
Re: Review Request 51617: Added the `remove` and `insert` methods. |
Thu, 29 Sep, 19:19 |
| Gilbert Song |
Review Request 52393: Allowed malformatted docker config file from '--docker_config'. |
Thu, 29 Sep, 15:50 |
| haosdent huang |
Review Request 52395: Updated the document link of 'Image Provisioning Design'. |
Thu, 29 Sep, 16:25 |
| Mesos ReviewBot |
Re: Review Request 52395: Updated the document link of 'Image Provisioning Design'. |
Fri, 30 Sep, 03:44 |
|
Re: Review Request 49906: Add lawrencew to contributors |
|
| Alexander Rukletsov |
Re: Review Request 49906: Add lawrencew to contributors |
Thu, 29 Sep, 17:18 |
| Sivaram Kannan |
Review Request 52392: Switch the uid to passed user before spawning the process. |
Thu, 29 Sep, 18:55 |
| Jie Yu |
Review Request 52405: Skipped isolator callbacks for nested containers if needed. |
Thu, 29 Sep, 20:15 |
| Gilbert Song |
Re: Review Request 52405: Skipped isolator callbacks for nested containers if needed. |
Thu, 29 Sep, 20:43 |
| Jie Yu |
Review Request 52406: Marked 'volume/sandbox_path' as nesting aware. |
Thu, 29 Sep, 20:18 |
| Joseph Wu |
Re: Review Request 52406: Marked 'volume/sandbox_path' as nesting aware. |
Thu, 29 Sep, 20:47 |
| Gilbert Song |
Re: Review Request 52406: Marked 'volume/sandbox_path' as nesting aware. |
Fri, 30 Sep, 00:25 |
|
Re: Review Request 51009: Collect throttle related cpu.stat for Docker Containerizer. |
|
| Zhitao Li |
Re: Review Request 51009: Collect throttle related cpu.stat for Docker Containerizer. |
Thu, 29 Sep, 20:37 |
| Mesos ReviewBot |
Re: Review Request 51009: Collect throttle related cpu.stat for Docker Containerizer. |
Fri, 30 Sep, 10:49 |
| Anand Mazumdar |
Review Request 52407: Added `RecoveryInfo` struct to the agent. |
Thu, 29 Sep, 22:08 |
| Vinod Kone |
Re: Review Request 52407: Added `RecoveryInfo` struct to the agent. |
Thu, 29 Sep, 22:13 |
| Anand Mazumdar |
Review Request 52408: Disallowed HTTP executors to subscribe before containerizer recovery. |
Thu, 29 Sep, 22:08 |
| Vinod Kone |
Re: Review Request 52408: Disallowed HTTP executors to subscribe before containerizer recovery. |
Thu, 29 Sep, 22:27 |
| Anand Mazumdar |
Re: Review Request 52408: Disallowed HTTP executors to subscribe before containerizer recovery. |
Thu, 29 Sep, 22:36 |
| Vinod Kone |
Re: Review Request 52408: Disallowed HTTP executors to subscribe before containerizer recovery. |
Thu, 29 Sep, 22:59 |
|
Re: Review Request 50491: Added PIDGroup to libprocess. |
|
| Joseph Wu |
Re: Review Request 50491: Added PIDGroup to libprocess. |
Fri, 30 Sep, 00:00 |
|
Re: Review Request 50493: Added `base` to PIDGroup. |
|
| Joseph Wu |
Re: Review Request 50493: Added `base` to PIDGroup. |
Fri, 30 Sep, 00:37 |
|
Re: Review Request 50498: Added --pid_group flag in master. |
|
| Joseph Wu |
Re: Review Request 50498: Added --pid_group flag in master. |
Fri, 30 Sep, 00:59 |
| Gilbert Song |
Review Request 52412: Supported logger with nested containers in Mesos Containerizer. |
Fri, 30 Sep, 01:14 |
| Gilbert Song |
Re: Review Request 52412: Supported logger with nested containers in Mesos Containerizer. |
Fri, 30 Sep, 05:19 |
| Mesos ReviewBot |
Re: Review Request 52412: Supported logger with nested containers in Mesos Containerizer. |
Fri, 30 Sep, 14:09 |
|
Review Request 52415: Extended buildPath to support more modes. |
|
| Jie Yu |
Review Request 52415: Extended buildPath to support more modes. |
Fri, 30 Sep, 05:14 |
| Kevin Klues |
Re: Review Request 52415: Extended buildPath to support more modes. |
Fri, 30 Sep, 15:36 |
| Jie Yu |
Re: Review Request 52415: Extended buildPath to support more modes. |
Fri, 30 Sep, 20:51 |
| Benjamin Hindman |
Re: Review Request 52415: Extended buildPath to support more modes. |
Fri, 30 Sep, 17:20 |
| Jie Yu |
Re: Review Request 52415: Extended buildPath to support more modes. |
Fri, 30 Sep, 20:49 |
| Benjamin Hindman |
Re: Review Request 52415: Extended buildPath to support more modes. |
Fri, 30 Sep, 21:13 |
| Jie Yu |
Re: Review Request 52415: Extended buildPath to support more modes. |
Fri, 30 Sep, 21:17 |
| Jie Yu |
Review Request 52416: Made 'parse' method in linux launcher take a real cgroup path. |
Fri, 30 Sep, 05:15 |
| Mesos ReviewBot |
Re: Review Request 52416: Made 'parse' method in linux launcher take a real cgroup path. |
Fri, 30 Sep, 12:06 |
| Benjamin Hindman |
Re: Review Request 52416: Made 'parse' method in linux launcher take a real cgroup path. |
Fri, 30 Sep, 15:19 |
| Benjamin Hindman |
Re: Review Request 52416: Made 'parse' method in linux launcher take a real cgroup path. |
Fri, 30 Sep, 15:20 |
| Jie Yu |
Re: Review Request 52416: Made 'parse' method in linux launcher take a real cgroup path. |
Fri, 30 Sep, 15:24 |
| Sivaram Kannan |
Review Request 52424: Fix removed empty lines from slave.cpp. |
Fri, 30 Sep, 14:48 |
|
Review Request 52431: Reduced boilerplate from health check tests. |
|
| Gastón Kleiman |
Review Request 52431: Reduced boilerplate from health check tests. |
Fri, 30 Sep, 17:15 |
| haosdent huang |
Re: Review Request 52431: Reduced boilerplate from health check tests. |
Fri, 30 Sep, 17:16 |
| Gastón Kleiman |
Review Request 52431: Reduced boilerplate from health check tests. |
Fri, 30 Sep, 17:15 |
| Gastón Kleiman |
Review Request 52432: Reenabled the 'HealthCheckTest.GracePeriod' test. |
Fri, 30 Sep, 17:15 |
| haosdent huang |
Re: Review Request 52432: Reenabled the 'HealthCheckTest.GracePeriod' test. |
Fri, 30 Sep, 17:23 |
| haosdent huang |
Re: Review Request 52432: Reenabled the 'HealthCheckTest.GracePeriod' test. |
Fri, 30 Sep, 18:41 |
| Gastón Kleiman |
Re: Review Request 52432: Reenabled the 'HealthCheckTest.GracePeriod' test. |
Fri, 30 Sep, 18:12 |
| Gastón Kleiman |
Review Request 52433: Improved the naming of variables in HealthCheckTest.HealthStatusChange. |
Fri, 30 Sep, 17:15 |
| Gastón Kleiman |
Review Request 52434: Improved handling of tmp file creation in health check test. |
Fri, 30 Sep, 17:15 |
| haosdent huang |
Re: Review Request 52434: Improved handling of tmp file creation in health check test. |
Fri, 30 Sep, 17:24 |
| Gilbert Song |
Review Request 52435: Fixed children list issue and log in containerizer destroy. |
Fri, 30 Sep, 18:15 |
| Gilbert Song |
Re: Review Request 52435: Fixed children list issue and log in containerizer destroy. |
Fri, 30 Sep, 18:20 |
| Kunal Thakar |
Review Request 52438: Reduce log level to verbose MESOS-6295. |
Fri, 30 Sep, 19:17 |
| Zhitao Li |
Re: Review Request 52438: Reduce log level to verbose MESOS-6295. |
Fri, 30 Sep, 19:20 |
| Zhitao Li |
Review Request 52439: Add Zhitao Li to contributor.yaml. |
Fri, 30 Sep, 19:18 |
| Vinod Kone |
Re: Review Request 52439: Add Zhitao Li to contributor.yaml. |
Fri, 30 Sep, 19:30 |
| Xiaojian Huang |
Re: Review Request 52439: Add Zhitao Li to contributor.yaml. |
Fri, 30 Sep, 20:06 |
| Zhitao Li |
Re: Review Request 52439: Add Zhitao Li to contributor.yaml. |
Fri, 30 Sep, 20:08 |
| Zhitao Li |
Re: Review Request 52439: Add Zhitao Li to contributor.yaml. |
Fri, 30 Sep, 20:08 |
| Xiaojian Huang |
Re: Review Request 52439: Add Zhitao Li to contributor.yaml. |
Fri, 30 Sep, 20:14 |
| Vinod Kone |
Review Request 52440: Fixed master to properly handle pending tasks. |
Fri, 30 Sep, 19:40 |
| Vinod Kone |
Re: Review Request 52440: Fixed master to properly handle pending tasks. |
Fri, 30 Sep, 20:21 |
| Anand Mazumdar |
Re: Review Request 52440: Fixed master to properly handle pending tasks. |
Fri, 30 Sep, 20:23 |
| Vinod Kone |
Re: Review Request 52440: Fixed master to properly handle pending tasks. |
Fri, 30 Sep, 20:30 |
| Jie Yu |
Review Request 52442: Added logging for MesosContainerizer internal state. |
Fri, 30 Sep, 20:28 |
| Gilbert Song |
Re: Review Request 52442: Added logging for MesosContainerizer internal state. |
Fri, 30 Sep, 20:40 |
| Jie Yu |
Review Request 52444: Fixed the MesosContainerizer destroy issue. |
Fri, 30 Sep, 20:32 |
| Gilbert Song |
Re: Review Request 52444: Fixed the MesosContainerizer destroy issue. |
Fri, 30 Sep, 21:01 |
| Jie Yu |
Re: Review Request 52444: Fixed the MesosContainerizer destroy issue. |
Fri, 30 Sep, 21:14 |
| Jie Yu |
Review Request 52445: Used posix launcher for multiple slave tests. |
Fri, 30 Sep, 20:33 |
| Gilbert Song |
Re: Review Request 52445: Used posix launcher for multiple slave tests. |
Fri, 30 Sep, 20:49 |
| Jie Yu |
Review Request 52448: Fixed a CNI test that might cause a failed container destroy. |
Fri, 30 Sep, 20:45 |
| Avinash sridharan |
Re: Review Request 52448: Fixed a CNI test that might cause a failed container destroy. |
Fri, 30 Sep, 21:09 |
| Kevin Klues |
Review Request 52446: Updated 'destroy()' to checkpoint termination state of nested container. |
Fri, 30 Sep, 20:53 |
| Kevin Klues |
Re: Review Request 52446: Updated 'destroy()' to checkpoint termination state of nested container. |
Fri, 30 Sep, 20:57 |
| Vinod Kone |
Re: Review Request 52446: Updated 'destroy()' to checkpoint termination state of nested container. |
Fri, 30 Sep, 21:03 |
| Kevin Klues |
Re: Review Request 52446: Updated 'destroy()' to checkpoint termination state of nested container. |
Fri, 30 Sep, 21:26 |
| Kevin Klues |
Re: Review Request 52446: Updated 'destroy()' to checkpoint termination state of nested container. |
Fri, 30 Sep, 21:27 |
| Jie Yu |
Re: Review Request 52446: Updated 'destroy()' to checkpoint termination state of nested container. |
Fri, 30 Sep, 21:52 |
| Kevin Klues |
Re: Review Request 52446: Updated 'destroy()' to checkpoint termination state of nested container. |
Fri, 30 Sep, 23:08 |
| Jie Yu |
Re: Review Request 52446: Updated 'destroy()' to checkpoint termination state of nested container. |
Fri, 30 Sep, 23:10 |
| Kevin Klues |
Re: Review Request 52446: Updated 'destroy()' to checkpoint termination state of nested container. |
Fri, 30 Sep, 23:08 |
| Jie Yu |
Re: Review Request 52446: Updated 'destroy()' to checkpoint termination state of nested container. |
Fri, 30 Sep, 23:12 |
| Kevin Klues |
Re: Review Request 52446: Updated 'destroy()' to checkpoint termination state of nested container. |
Fri, 30 Sep, 23:22 |
| Kevin Klues |
Re: Review Request 52446: Updated 'destroy()' to checkpoint termination state of nested container. |
Fri, 30 Sep, 23:22 |
| Kevin Klues |
Re: Review Request 52446: Updated 'destroy()' to checkpoint termination state of nested container. |
Fri, 30 Sep, 23:22 |
| Jie Yu |
Review Request 52450: Simplied the buildPath logic. |
Fri, 30 Sep, 21:35 |
| Kevin Klues |
Re: Review Request 52450: Simplied the buildPath logic. |
Fri, 30 Sep, 23:41 |
| Jie Yu |
Review Request 52459: Ignored error in posix launcher destroy if the container is unknown. |
Fri, 30 Sep, 23:17 |
| Gilbert Song |
Re: Review Request 52459: Ignored error in posix launcher destroy if the container is unknown. |
Fri, 30 Sep, 23:27 |
| Jie Yu |
Review Request 52460: Ignored cleanup request for unknown containers in posix isolators. |
Fri, 30 Sep, 23:18 |
| Gilbert Song |
Re: Review Request 52460: Ignored cleanup request for unknown containers in posix isolators. |
Fri, 30 Sep, 23:27 |