> On Dec. 13, 2018, 12:13 a.m., Andrei Budnik wrote: > > src/tests/persistent_volume_tests.cpp > > Lines 2521 (patched) > > <https://reviews.apache.org/r/68163/diff/2/?file=2113016#file2113016line2521> > > > > Do we have a check in the Mesos agent that all related paths have read > > permissions for `group`/`other`?
I do not remember we have such check in agent code, but usually the mode of the dirs created by agent is `rwxr-xr-x` which I think depends on the `umask`. The reason that we need to do `chmod` here for the work dir is, the work dir is created by `mkdtemp` which creates directory with permissions 0700. - Qian ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68163/#review211241 ----------------------------------------------------------- On Dec. 11, 2018, 2:51 p.m., Qian Zhang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68163/ > ----------------------------------------------------------- > > (Updated Dec. 11, 2018, 2:51 p.m.) > > > Review request for mesos, Andrei Budnik, Gilbert Song, Greg Mann, Ilya > Pronin, and Jie Yu. > > > Bugs: MESOS-8813 > https://issues.apache.org/jira/browse/MESOS-8813 > > > Repository: mesos > > > Description > ------- > > Added a test `UNPRIVILEGED_USER_SharedPersistentVolume`. > > > Diffs > ----- > > src/tests/persistent_volume_tests.cpp > 6afc56cecbf6f06615dbc8632578332e6f7fb6ae > > > Diff: https://reviews.apache.org/r/68163/diff/2/ > > > Testing > ------- > > sudo make check > > > Thanks, > > Qian Zhang > >