----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50915/#review145170 -----------------------------------------------------------
Ship it! Ship It! - Gilbert Song On Aug. 8, 2016, 9:39 p.m., Benjamin Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50915/ > ----------------------------------------------------------- > > (Updated Aug. 8, 2016, 9:39 p.m.) > > > Review request for mesos, Gilbert Song and Jie Yu. > > > Bugs: MESOS-5889 > https://issues.apache.org/jira/browse/MESOS-5889 > > > Repository: mesos > > > Description > ------- > > During TearDown of ContainerizerTest<slave::MesosContainerizer>, > we attempt to destroy all cgroups that we can find within the > cgroup root. We observed in CI that on some distributions it > appears that the freeze is not instantaneous and the freezer > delay is needed to complete the destroy process. Howevever, > the clock may be paused at this point which can lead to the > TearDown failing to destroy! > > There does not appear to be a listener hook to run code before > test tear down, so a one-off Clock::resume here seems to be the > simplest route. > > > Diffs > ----- > > src/tests/mesos.cpp 30492d7e3b4c5e9ae9d2b2446cadba62d43a3c65 > > Diff: https://reviews.apache.org/r/50915/diff/ > > > Testing > ------- > > Ran in internal CI on distributions that were seeing this flakiness. > > > Thanks, > > Benjamin Mahler > >
