----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65232/#review195760 -----------------------------------------------------------
Ship it! Ship It! - Gaston Kleiman On Jan. 18, 2018, 4:40 p.m., Greg Mann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65232/ > ----------------------------------------------------------- > > (Updated Jan. 18, 2018, 4:40 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/1/ > > > Testing > ------- > > make check > > > Thanks, > > Greg Mann > >