> On Sept. 20, 2018, 10:15 p.m., Ilya Pronin wrote: > > src/tests/containerizer/xfs_quota_tests.cpp > > Lines 879-881 (original), 904-906 (patched) > > <https://reviews.apache.org/r/68403/diff/4/?file=2087669#file2087669line906> > > > > Nit: move `usage()` call below the started/working "locks" checking?
I'm going to drop this one. The `AWAIT` happens first so that we don't busy-poll on the filesystem state (it is basically rate-limiting the poll). - James ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68403/#review208818 ----------------------------------------------------------- On Aug. 16, 2018, 11:52 p.m., James Peach wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68403/ > ----------------------------------------------------------- > > (Updated Aug. 16, 2018, 11: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` resource statistics tests to ensure that > statistics for persistent volumes are collected and published > correctly. > > > Diffs > ----- > > src/tests/containerizer/xfs_quota_tests.cpp > 2b3a2e25f5075357f090d47826698e7bb6fdf969 > > > Diff: https://reviews.apache.org/r/68403/diff/4/ > > > Testing > ------- > > sudo make check (Fedora 28) > > > Thanks, > > James Peach > >
