Re: Review Request 52004: Supported docker runtime isolator to be nested aware.

2016-09-26 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52004/#review150524 --- Ship it! Ship It! - Jie Yu On Sept. 27, 2016, 4:53 a.m., Gil

Re: Review Request 52004: Supported docker runtime isolator to be nested aware.

2016-09-26 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52004/ --- (Updated Sept. 26, 2016, 9:53 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 52004: Supported docker runtime isolator to be nested aware.

2016-09-26 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52004/#review150518 --- src/slave/containerizer/mesos/isolators/docker/runtime.cpp (line

Re: Review Request 52004: Supported docker runtime isolator to be nested aware.

2016-09-26 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52004/ --- (Updated Sept. 26, 2016, 5:13 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 52004: Supported docker runtime isolator to be nested aware.

2016-09-26 Thread Guangya Liu
> On 九月 18, 2016, 9:03 p.m., Jie Yu wrote: > > src/slave/containerizer/mesos/isolators/docker/runtime.cpp, lines 77-82 > > > > > > We can remove this. The default behavior is this. > > Guangya Liu wrote: > There

Re: Review Request 52004: Supported docker runtime isolator to be nested aware.

2016-09-26 Thread Gilbert Song
> On Sept. 18, 2016, 2:03 p.m., Jie Yu wrote: > > src/slave/containerizer/mesos/isolators/docker/runtime.cpp, lines 77-82 > > > > > > We can remove this. The default behavior is this. > > Guangya Liu wrote: > The

Re: Review Request 52004: Supported docker runtime isolator to be nested aware.

2016-09-18 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52004/#review149421 --- Ditto for appc run time isolator here: https://reviews.apache.org/

Re: Review Request 52004: Supported docker runtime isolator to be nested aware.

2016-09-18 Thread Guangya Liu
> On 九月 18, 2016, 9:03 p.m., Jie Yu wrote: > > src/slave/containerizer/mesos/isolators/docker/runtime.cpp, lines 77-82 > > > > > > We can remove this. The default behavior is this. There is already a patch here http

Re: Review Request 52004: Supported docker runtime isolator to be nested aware.

2016-09-18 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52004/#review149385 --- src/slave/containerizer/mesos/isolators/docker/runtime.cpp (lines

Re: Review Request 52004: Supported docker runtime isolator to be nested aware.

2016-09-18 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52004/#review149382 --- Fix it, then Ship it! src/slave/containerizer/mesos/isolators/