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-04 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. -

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 https://reviews.apache.org/r/36181/diff/1/?file=999124#file999124line282 These flags are supported only after a certain docker version right? I think we need to log and disable

Review Request 36181: Port CFS support to Docker Containerizer

2015-07-04 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36181/ --- Review request for mesos and Timothy Chen. Bugs: MESOS-2154

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. -