Re: Review Request 43854: Fixed chdir to an empty directory error.

2016-02-23 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43854/#review120420 --- Ship it! Ship It! - Jie Yu On Feb. 22, 2016, 10:52 p.m.,

Re: Review Request 43854: Fixed chdir to an empty directory error.

2016-02-23 Thread Jie Yu
> On Feb. 23, 2016, 6:40 a.m., Timothy Chen wrote: > > src/slave/containerizer/mesos/isolators/docker/runtime.cpp, line 354 > > > > > > Do we have tests to test the docker runtime isolator? Yeah, I think he's

Re: Review Request 43854: Fixed chdir to an empty directory error.

2016-02-22 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43854/#review120268 --- src/slave/containerizer/mesos/isolators/docker/runtime.cpp (line

Re: Review Request 43854: Fixed chdir to an empty directory error.

2016-02-22 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43854/#review120261 --- Patch looks great! Reviews applied: [43854] Passed command:

Review Request 43854: Fixed chdir to an empty directory error.

2016-02-22 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43854/ --- Review request for mesos, Artem Harutyunyan, Jie Yu, and Timothy Chen.

Re: Review Request 43854: Fixed chdir to an empty directory error.

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