----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38333/#review98764 -----------------------------------------------------------
Patch looks great! Reviews applied: [38333] All tests passed. - Mesos ReviewBot On Sept. 13, 2015, 6:33 a.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38333/ > ----------------------------------------------------------- > > (Updated Sept. 13, 2015, 6:33 a.m.) > > > Review request for mesos, haosdent huang, Timothy Chen, Vinod Kone, and Jiang > Yan Xu. > > > Bugs: MESOS-3349 > https://issues.apache.org/jira/browse/MESOS-3349 > > > Repository: mesos > > > Description > ------- > > Made container sandbox a shared mount to address MESOS-3349. > > See the discussion in https://reviews.apache.org/r/38329/ for more context. > > The idea is to mark container sandbox a shared mount (do a self bind mount > first) so that persistent volume mounts can be propagated. > > This is less invasive than marking '/' as a shared mount. > > One followup for this patch is to set the default filesystem isolator to > posix as the linux isolator will manipulate host mount table. > > We also need to address the TODO in `LinuxFilesystemIsolator::_recover` so > that tests do not leak mounts in the host mount table. > > > Diffs > ----- > > src/slave/containerizer/isolators/filesystem/linux.cpp > 0970b3d48b13d5e9d2e0160df5cf14a3dcd0acc9 > src/slave/containerizer/provisioners/appc/provisioner.cpp > cd29a00fa0db8af294c10bb7a2e0cb4252bd2993 > src/slave/containerizer/provisioners/backends/bind.cpp > 1cdae61786790dc6a475ae5f73c8cc92d2bbf739 > > Diff: https://reviews.apache.org/r/38333/diff/ > > > Testing > ------- > > sudo make check on Centos5 and Centos6 > > > Thanks, > > Jie Yu > >
