Re: Review Request 53296: Added cgroup namespace support for unified container.

2017-05-07 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53296/ --- (Updated May 8, 2017, 2:51 a.m.) Review request for mesos, Jason Lai, Jie Yu, J

Re: Review Request 53296: Added cgroup namespace support for unified container.

2017-01-26 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53296/ --- (Updated Jan. 26, 2017, 8:18 a.m.) Review request for mesos, Jason Lai, Jie Yu,

Re: Review Request 53296: Added cgroup namespace support for unified container.

2016-11-27 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53296/ --- (Updated Nov. 27, 2016, 3:34 p.m.) Review request for mesos, Jie Yu, Qian Zhang

Re: Review Request 53296: Added cgroup namespace support for unified container.

2016-11-27 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53296/ --- (Updated Nov. 27, 2016, 2:22 p.m.) Review request for mesos, Jie Yu, Qian Zhang

Re: Review Request 53296: Added cgroup namespace support for unified container.

2016-11-06 Thread Jie Yu
> On Nov. 1, 2016, 4:43 a.m., Jie Yu wrote: > > src/slave/containerizer/mesos/isolators/namespaces/cgroup.hpp, line 28 > > > > > > Instead of creating a new namespace/cgroup isolator, I would suggest we > > add the s

Re: Review Request 53296: Added cgroup namespace support for unified container.

2016-11-06 Thread haosdent huang
> On Nov. 1, 2016, 4:43 a.m., Jie Yu wrote: > > src/slave/containerizer/mesos/isolators/namespaces/cgroup.hpp, line 28 > > > > > > Instead of creating a new namespace/cgroup isolator, I would suggest we > > add the s

Re: Review Request 53296: Added cgroup namespace support for unified container.

2016-11-06 Thread Jie Yu
> On Nov. 1, 2016, 4:43 a.m., Jie Yu wrote: > > src/slave/containerizer/mesos/isolators/namespaces/cgroup.hpp, line 28 > > > > > > Instead of creating a new namespace/cgroup isolator, I would suggest we > > add the s

Re: Review Request 53296: Added cgroup namespace support for unified container.

2016-11-06 Thread haosdent huang
> On Nov. 1, 2016, 4:43 a.m., Jie Yu wrote: > > src/slave/containerizer/mesos/isolators/namespaces/cgroup.hpp, line 28 > > > > > > Instead of creating a new namespace/cgroup isolator, I would suggest we > > add the s

Re: Review Request 53296: Added cgroup namespace support for unified container.

2016-11-06 Thread Jie Yu
> On Nov. 1, 2016, 4:43 a.m., Jie Yu wrote: > > src/slave/containerizer/mesos/isolators/namespaces/cgroup.hpp, line 28 > > > > > > Instead of creating a new namespace/cgroup isolator, I would suggest we > > add the s

Re: Review Request 53296: Added cgroup namespace support for unified container.

2016-11-06 Thread haosdent huang
> On Nov. 1, 2016, 4:43 a.m., Jie Yu wrote: > > src/slave/containerizer/mesos/isolators/namespaces/cgroup.hpp, line 28 > > > > > > Instead of creating a new namespace/cgroup isolator, I would suggest we > > add the s

Re: Review Request 53296: Added cgroup namespace support for unified container.

2016-11-06 Thread haosdent huang
> On Nov. 1, 2016, 4:43 a.m., Jie Yu wrote: > > src/slave/containerizer/mesos/isolators/namespaces/cgroup.hpp, line 28 > > > > > > Instead of creating a new namespace/cgroup isolator, I would suggest we > > add the s

Re: Review Request 53296: Added cgroup namespace support for unified container.

2016-11-06 Thread Jie Yu
> On Nov. 1, 2016, 4:43 a.m., Jie Yu wrote: > > src/slave/containerizer/mesos/isolators/namespaces/cgroup.hpp, line 28 > > > > > > Instead of creating a new namespace/cgroup isolator, I would suggest we > > add the s

Re: Review Request 53296: Added cgroup namespace support for unified container.

2016-11-06 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53296/ --- (Updated Nov. 6, 2016, 12:47 p.m.) Review request for mesos, Jie Yu, Qian Zhang

Re: Review Request 53296: Added cgroup namespace support for unified container.

2016-11-06 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53296/#review155080 --- Patch looks great! Reviews applied: [53296] Passed command: expo

Re: Review Request 53296: Added cgroup namespace support for unified container.

2016-11-06 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53296/ --- (Updated Nov. 6, 2016, 7:35 a.m.) Review request for mesos, Jie Yu, Qian Zhang,

Re: Review Request 53296: Added cgroup namespace support for unified container.

2016-11-05 Thread haosdent huang
> On Nov. 1, 2016, 4:43 a.m., Jie Yu wrote: > > src/slave/containerizer/mesos/isolators/namespaces/cgroup.hpp, line 28 > > > > > > Instead of creating a new namespace/cgroup isolator, I would suggest we > > add the s

Re: Review Request 53296: Added cgroup namespace support for unified container.

2016-11-05 Thread Jie Yu
> On Nov. 1, 2016, 4:43 a.m., Jie Yu wrote: > > src/slave/containerizer/mesos/isolators/namespaces/cgroup.hpp, line 28 > > > > > > Instead of creating a new namespace/cgroup isolator, I would suggest we > > add the s

Re: Review Request 53296: Added cgroup namespace support for unified container.

2016-11-01 Thread haosdent huang
> On Nov. 1, 2016, 4:43 a.m., Jie Yu wrote: > > src/slave/containerizer/mesos/isolators/namespaces/cgroup.hpp, line 28 > > > > > > Instead of creating a new namespace/cgroup isolator, I would suggest we > > add the s

Re: Review Request 53296: Added cgroup namespace support for unified container.

2016-10-31 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53296/#review154371 --- src/slave/containerizer/mesos/isolators/namespaces/cgroup.hpp (li

Re: Review Request 53296: Added cgroup namespace support for unified container.

2016-10-31 Thread James Peach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53296/#review154370 --- src/slave/containerizer/mesos/isolators/namespaces/cgroup.cpp (li

Re: Review Request 53296: Added cgroup namespace support for unified container.

2016-10-31 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53296/#review154250 --- Patch looks great! Reviews applied: [53296] Passed command: expo

Re: Review Request 53296: Added cgroup namespace support for unified container.

2016-10-30 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53296/ --- (Updated Oct. 30, 2016, 4:45 p.m.) Review request for mesos, Jie Yu, Qian Zhang

Review Request 53296: Added cgroup namespace support for unified container.

2016-10-30 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53296/ --- Review request for mesos, Jie Yu, Qian Zhang, and Jiang Yan Xu. Bugs: MESOS-541