Re: Review Request 38569: Made slave's work_dir a shared mount in LinuxFilesystemIsolator.

2015-09-22 Thread Cong Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38569/#review99927 --- src/slave/containerizer/isolators/filesystem/linux.cpp (line 87)

Re: Review Request 38569: Made slave's work_dir a shared mount in LinuxFilesystemIsolator.

2015-09-21 Thread Jie Yu
> On Sept. 21, 2015, 10:35 p.m., Jiang Yan Xu wrote: > > src/slave/containerizer/isolators/filesystem/linux.cpp, lines 73-74 > > > > > > After then sentense `using the bind backend)` add > > > > `because

Re: Review Request 38569: Made slave's work_dir a shared mount in LinuxFilesystemIsolator.

2015-09-21 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38569/#review99895 --- Ship it! Ship It! - haosdent huang On Sept. 21, 2015, 9:23

Re: Review Request 38569: Made slave's work_dir a shared mount in LinuxFilesystemIsolator.

2015-09-21 Thread Jiang Yan Xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38569/#review99869 --- Ship it! Ship It! - Jiang Yan Xu On Sept. 21, 2015, 2:23 p.m.,

Re: Review Request 38569: Made slave's work_dir a shared mount in LinuxFilesystemIsolator.

2015-09-21 Thread haosdent huang
> On Sept. 21, 2015, 10:35 p.m., Jiang Yan Xu wrote: > > src/slave/containerizer/isolators/filesystem/linux.cpp, lines 97-98 > > > > > > So this seems to work but oh my godness the the way `mount` command > >

Re: Review Request 38569: Made slave's work_dir a shared mount in LinuxFilesystemIsolator.

2015-09-21 Thread Jiang Yan Xu
> On Sept. 21, 2015, 3:35 p.m., Jiang Yan Xu wrote: > > src/slave/containerizer/isolators/filesystem/linux.cpp, lines 97-98 > > > > > > So this seems to work but oh my godness the the way `mount` command > >

Re: Review Request 38569: Made slave's work_dir a shared mount in LinuxFilesystemIsolator.

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

Re: Review Request 38569: Made slave's work_dir a shared mount in LinuxFilesystemIsolator.

2015-09-21 Thread Jiang Yan Xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38569/#review99831 --- src/slave/containerizer/isolators/filesystem/linux.cpp (lines 73 -

Re: Review Request 38569: Made slave's work_dir a shared mount in LinuxFilesystemIsolator.

2015-09-21 Thread Jie Yu
> On Sept. 21, 2015, 10:35 p.m., Jiang Yan Xu wrote: > > src/slave/containerizer/isolators/filesystem/linux.cpp, lines 97-98 > > > > > > So this seems to work but oh my godness the the way `mount` command > >

Review Request 38569: Made slave's work_dir a shared mount in LinuxFilesystemIsolator.

2015-09-21 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38569/ --- Review request for mesos, Jojy Varghese, Timothy Chen, and Jiang Yan Xu. Bugs: