----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52097/#review149890 -----------------------------------------------------------
src/slave/container_loggers/logrotate.cpp (lines 203 - 205) <https://reviews.apache.org/r/52097/#comment217652> And apparently, there's a helper that is more appropriate for changing the user (in this case): https://github.com/apache/mesos/blob/cd5e924ad1b0a003fd2a1b711631357ccde0edc0/3rdparty/stout/include/stout/os/posix/su.hpp#L296 - Joseph Wu On Sept. 20, 2016, 8:54 p.m., Sivaram Kannan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52097/ > ----------------------------------------------------------- > > (Updated Sept. 20, 2016, 8:54 p.m.) > > > Review request for mesos and Joseph Wu. > > > Bugs: MESOS-5856 > https://issues.apache.org/jira/browse/MESOS-5856 > > > Repository: mesos > > > Description > ------- > > Functionality to switch user when executor launches as non-root user. > > > Diffs > ----- > > src/slave/container_loggers/logrotate.cpp > 431bc3cbb54e94359078e4dae0b32ad301393640 > > Diff: https://reviews.apache.org/r/52097/diff/ > > > Testing > ------- > > > Thanks, > > Sivaram Kannan > >
