Re: Review Request 36181: Port CFS support to Docker Containerizer

2016-08-12 Thread Zhitao Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36181/#review145638 --- src/slave/containerizer/docker.cpp (lines 145 - 147)

Re: Review Request 36181: Port CFS support to Docker Containerizer

2015-10-04 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36181/#review101441 --- Patch looks great! Reviews applied: [36181] All tests passed. -

Re: Review Request 36181: Port CFS support to Docker Containerizer

2015-10-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36181/ --- (Updated Oct. 4, 2015, 6:40 a.m.) Review request for mesos and Timothy Chen.

Re: Review Request 36181: Port CFS support to Docker Containerizer

2015-10-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36181/ --- (Updated Oct. 3, 2015, 5:14 p.m.) Review request for mesos and Timothy Chen.

Re: Review Request 36181: Port CFS support to Docker Containerizer

2015-07-11 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36181/#review91391 --- Patch looks great! Reviews applied: [36181] All tests passed. - M

Re: Review Request 36181: Port CFS support to Docker Containerizer

2015-07-11 Thread haosdent huang
> On July 4, 2015, 5:34 p.m., Timothy Chen wrote: > > src/slave/containerizer/docker.cpp, line 282 > > > > > > These flags are supported only after a certain docker version right? > > > > I think we need to lo

Re: Review Request 36181: Port CFS support to Docker Containerizer

2015-07-11 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36181/ --- (Updated July 11, 2015, 8:59 a.m.) Review request for mesos and Timothy Chen.

Re: Review Request 36181: Port CFS support to Docker Containerizer

2015-07-04 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36181/#review90399 --- Patch looks great! Reviews applied: [36181] All tests passed. - M

Re: Review Request 36181: Port CFS support to Docker Containerizer

2015-07-04 Thread haosdent huang
> On July 4, 2015, 5:34 p.m., Timothy Chen wrote: > > src/slave/containerizer/docker.cpp, line 282 > > > > > > These flags are supported only after a certain docker version right? > > > > I think we need to lo

Re: Review Request 36181: Port CFS support to Docker Containerizer

2015-07-04 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36181/#review90392 --- src/slave/containerizer/docker.cpp (line 282)