> On Aug. 27, 2015, 12:03 a.m., Timothy Chen wrote: > > src/slave/containerizer/mesos/containerizer.cpp, line 145 > > <https://reviews.apache.org/r/37812/diff/1/?file=1055255#file1055255line145> > > > > This is quite subtle, I think we should log and also put some > > documentation around slave that we only choose linux filesystem by default > > with root.
The logging is already there (see code below): `LOG(INFO) << "Using isolation: " << isolation;` Adjusted the comments above. - Jie ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37812/#review96615 ----------------------------------------------------------- On Aug. 26, 2015, 9:04 p.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37812/ > ----------------------------------------------------------- > > (Updated Aug. 26, 2015, 9:04 p.m.) > > > Review request for mesos, Timothy Chen, Vinod Kone, and Jiang Yan Xu. > > > Repository: mesos > > > Description > ------- > > Used linux filesystem isolator by default if possible. > > > Diffs > ----- > > src/slave/containerizer/mesos/containerizer.cpp > 377de50da90edd64dab812fda3730fa4d7d63b13 > > Diff: https://reviews.apache.org/r/37812/diff/ > > > Testing > ------- > > sudo make check > > > Thanks, > > Jie Yu > >
