> On July 4, 2016, 10:06 a.m., Jie Yu wrote: > > Please see the test section of https://reviews.apache.org/r/49569/ > > > > Let me know if that's ok or not. Thanks! > > Ian Downes wrote: > Hey, thanks for looking at this. I looked at the review for > `mesos-containerizer launch` and it just optionally creates a new mount ns. > So, to actually "chroot" a task we still need something like the > functionality in this review?
Actually, ignore my comment here, I'm reading more of the functionality contained within the launch command and it may be sufficient. :-) - Ian ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49273/#review140687 ----------------------------------------------------------- On June 27, 2016, 10:09 a.m., Ian Downes wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/49273/ > ----------------------------------------------------------- > > (Updated June 27, 2016, 10:09 a.m.) > > > Review request for mesos, Joshua Cohen and Jie Yu. > > > Repository: mesos > > > Description > ------- > > Uses the same code as the agent uses for chroot'ing on Linux, i.e., > pivot_root and setting up /dev etc. Intention is that executors (like > Aurora's Thermos) can use it to chroot tasks. > > Currently, the root path is specified as a flag and the remaining arguments > are exec'ed. Joshua has also requested that the root path could be specified > as the first arg. @Jie, thoughts? > > > Diffs > ----- > > src/Makefile.am 86c39fdf379ada470c9b1f86be263ef71dc47c41 > src/slave/containerizer/mesos-chroot.cpp PRE-CREATION > > Diff: https://reviews.apache.org/r/49273/diff/ > > > Testing > ------- > > Manual. > > > Thanks, > > Ian Downes > >