> On Aug. 8, 2019, 1:33 a.m., Qian Zhang wrote: > > src/tests/containerizer/docker_volume_isolator_tests.cpp > > Lines 1275 (patched) > > <https://reviews.apache.org/r/71249/diff/2/?file=2160249#file2160249line1275> > > > > I think we need `UNPRIVILEGED_USER_` in the test name to use > > `SUDO_USER` in the test. And then we do not need `NonRootUser` in the test > > name.
I was trying to use that prefix but it makes it too long. - Gilbert ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71249/#review217118 ----------------------------------------------------------- On Aug. 7, 2019, 11:59 p.m., Gilbert Song wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71249/ > ----------------------------------------------------------- > > (Updated Aug. 7, 2019, 11:59 p.m.) > > > Review request for mesos, Andrei Budnik and Qian Zhang. > > > Bugs: MESOS-9908 > https://issues.apache.org/jira/browse/MESOS-9908 > > > Repository: mesos > > > Description > ------- > > Added a unit test for the docker volume chown support. > > > Diffs > ----- > > src/tests/containerizer/docker_volume_isolator_tests.cpp > cd5c028991b0ef2438dd5c61a08d48747a109585 > > > Diff: https://reviews.apache.org/r/71249/diff/2/ > > > Testing > ------- > > make check > > > Thanks, > > Gilbert Song > >
