> On Oct. 13, 2016, 11:47 p.m., Jiang Yan Xu wrote: > > src/tests/slave_tests.cpp, line 932 > > <https://reviews.apache.org/r/52828/diff/2/?file=1534792#file1534792line932> > > > > "test" worked? Could you verify?
"test" didn't work since we are chowning so needs a real user. Replaced it with "nobody". - Megha ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52828/#review152606 ----------------------------------------------------------- On Oct. 14, 2016, 6:14 p.m., Megha Sharma wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52828/ > ----------------------------------------------------------- > > (Updated Oct. 14, 2016, 6:14 p.m.) > > > Review request for mesos and Jiang Yan Xu. > > > Bugs: mesos-5218 > https://issues.apache.org/jira/browse/mesos-5218 > > > Repository: mesos > > > Description > ------- > > Moved the uri.size() check in fetcher so that the chown to > task user of stdout/stderr in sandbox directory happens even > when there is no uri to be fetched. > > > Diffs > ----- > > src/slave/containerizer/fetcher.cpp > 11104d66e6dd05d8eb1d37a2e3250aca19278110 > src/tests/slave_tests.cpp 93b81d3e5b90d8036409e943f598c865fe335bcc > > Diff: https://reviews.apache.org/r/52828/diff/ > > > Testing > ------- > > make check on macOs and linux > > > Thanks, > > Megha Sharma > >
