----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68404/#review208819 -----------------------------------------------------------
src/tests/containerizer/xfs_quota_tests.cpp Lines 1260-1262 (patched) <https://reviews.apache.org/r/68404/#comment293031> Should we also check that quota is still set like we do for the sandbox? src/tests/containerizer/xfs_quota_tests.cpp Lines 1401-1403 (patched) <https://reviews.apache.org/r/68404/#comment293032> Ditto re: quota check. - Ilya Pronin On Aug. 16, 2018, 4:52 p.m., James Peach wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68404/ > ----------------------------------------------------------- > > (Updated Aug. 16, 2018, 4:52 p.m.) > > > Review request for mesos, Chun-Hung Hsiao, Ilya Pronin, Jie Yu, Joseph Wu, > and Jiang Yan Xu. > > > Bugs: MESOS-5158 > https://issues.apache.org/jira/browse/MESOS-5158 > > > Repository: mesos > > > Description > ------- > > Updated the `disk/xfs` recovery tests to ensure that projects IDs on > persistent volumes are correctly accounted for across agent restarts. > > > Diffs > ----- > > src/tests/containerizer/xfs_quota_tests.cpp > 2b3a2e25f5075357f090d47826698e7bb6fdf969 > > > Diff: https://reviews.apache.org/r/68404/diff/4/ > > > Testing > ------- > > sudo make check (Fedora 28) > > > Thanks, > > James Peach > >