> On May 20, 2014, 12:34 a.m., Tobias Weingartner wrote: > > src/slave/containerizer/mesos_containerizer.cpp, line 442 > > <https://reviews.apache.org/r/20958/diff/3/?file=584749#file584749line442> > > > > Should contain a mode (0666).
Aye, that is what is biting us now in MESOS-1402 - Till ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20958/#review43447 ----------------------------------------------------------- On May 19, 2014, 10:57 p.m., Ian Downes wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20958/ > ----------------------------------------------------------- > > (Updated May 19, 2014, 10:57 p.m.) > > > Review request for mesos, Ben Mahler, Jie Yu, and Vinod Kone. > > > Repository: mesos-git > > > Description > ------- > > We *may* have observed deadlocks in the forked child when testing some other > new code so this makes the execute() function in MesosContainerizer > completely async-signal safe. > > Reviewers: I'd appreciate it if you could carefully check that I've made this > completely async signal safe. > > > Replace all stout calls with just the appropriate system call. > > Construct Envp and pass into child and use execle for the environment. > > Use open and dup2 rather than freopen. > > Determine gid and uid and pass into child. > > > Diffs > ----- > > src/slave/containerizer/mesos_containerizer.cpp > 2a4816ec34f2298b71e8591c1e92d77f94ef4cb3 > > Diff: https://reviews.apache.org/r/20958/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Ian Downes > >
