> On Jan. 19, 2018, 5:03 p.m., Benjamin Bannier wrote: > > Thanks for this fix Greg. This will help remove some unsafe boilerplate > > from tests. We should probably follow up with a tech-debt ticket removing > > unnecessary clock resumes in tests.
Created https://issues.apache.org/jira/browse/MESOS-8466 to look for unnecessary calls to `Clock::resume()` in the tests. - Greg ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65232/#review195836 ----------------------------------------------------------- On Jan. 19, 2018, 5:36 p.m., Greg Mann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65232/ > ----------------------------------------------------------- > > (Updated Jan. 19, 2018, 5:36 p.m.) > > > Review request for mesos, Benjamin Bannier and Gaston Kleiman. > > > Repository: mesos > > > Description > ------- > > Because the `cgroups::destroy()` code path makes use of `delay()`, > the clock must not be paused in order for the destructor of the > test `Slave` to reliably destroy all remaining containers. > > This patch updates the destructor to check if the clock is paused > and, if so, resume it before destroying containers. > > > Diffs > ----- > > src/tests/cluster.cpp 568c9c7eda9d69e85bc32ce3259548b348ed9f25 > > > Diff: https://reviews.apache.org/r/65232/diff/2/ > > > Testing > ------- > > make check > > > Thanks, > > Greg Mann > >
