Re: Review Request 43083: Supported working dir in docker runtime isolator.

2016-02-05 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43083/#review118003 --- Patch looks great! Reviews applied: [43019, 43020, 43021, 43022,

Re: Review Request 43083: Supported working dir in docker runtime isolator.

2016-02-05 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43083/ --- (Updated Feb. 5, 2016, 12:13 a.m.) Review request for mesos, Artem Harutyunyan,

Re: Review Request 43083: Supported working dir in docker runtime isolator.

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

Re: Review Request 43083: Supported working dir in docker runtime isolator.

2016-02-03 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43083/#review117758 --- Patch looks great! Reviews applied: [43019, 43020, 43021, 43022,

Re: Review Request 43083: Supported working dir in docker runtime isolator.

2016-02-03 Thread Gilbert Song
> On Feb. 2, 2016, 4:16 p.m., Jie Yu wrote: > > src/slave/containerizer/mesos/containerizer.cpp, lines 1029-1034 > > > > > > We shouldn't allow executor to cd into an arbitrary directory if > > filesystem isolation

Re: Review Request 43083: Supported working dir in docker runtime isolator.

2016-02-03 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43083/ --- (Updated Feb. 3, 2016, 4:49 p.m.) Review request for mesos, Artem Harutyunyan,

Re: Review Request 43083: Supported working dir in docker runtime isolator.

2016-02-02 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43083/#review117522 --- src/slave/containerizer/mesos/containerizer.cpp (lines 1029 - 103

Re: Review Request 43083: Supported working dir in docker runtime isolator.

2016-02-02 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43083/#review117448 --- Patch looks great! Reviews applied: [43019, 43020, 43021, 43022,

Re: Review Request 43083: Supported working dir in docker runtime isolator.

2016-02-02 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43083/ --- (Updated Feb. 2, 2016, 9:18 a.m.) Review request for mesos, Artem Harutyunyan,