----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29896/#review68376 -----------------------------------------------------------
src/slave/flags.hpp <https://reviews.apache.org/r/29896/#comment112573> Could you document that this is relevant only for the "posix/disk" isolator? src/tests/disk_quota_tests.cpp <https://reviews.apache.org/r/29896/#comment112574> Whoops, this should be false, right? The fact that this test passed means it doesn't catch the kill of the task. Too bad we can't pause the clock! Maybe you need a small sleep here with a TODO? Or a way to capture the `_monitor` dispatch? - Ben Mahler On Jan. 14, 2015, 8:53 p.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29896/ > ----------------------------------------------------------- > > (Updated Jan. 14, 2015, 8:53 p.m.) > > > Review request for mesos, Ben Mahler, Ian Downes, and Vinod Kone. > > > Bugs: MESOS-1588 > https://issues.apache.org/jira/browse/MESOS-1588 > > > Repository: mesos-git > > > Description > ------- > > Added a slave flag to turn off disk quota enforcement. > > > Diffs > ----- > > src/slave/containerizer/isolators/posix/disk.cpp PRE-CREATION > src/slave/flags.hpp f1b8dfbb7391167b67a9498561742aa9ab9089a6 > src/tests/disk_quota_tests.cpp PRE-CREATION > > Diff: https://reviews.apache.org/r/29896/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Jie Yu > >
