> On Jan. 25, 2017, 7:31 p.m., Joseph Wu wrote: > > If I understand this correctly, the test is failing (when run as ROOT) > > because: > > > > * Agent 1 starts a container. > > * Agent 1 is partitioned. > > * Agent 2 starts. > > * Agent 2 observes (via global cgroups) Agent 1's container as an orphan. > > * Agent 2 kills Agent 1's container. > > * During test teardown, Agent 1 tries to clean up the container destroyed > > by Agent 2, fails. > > > > To fix this, you are simply cleaning up Agent 1 after you've partitioned > > it, as it doesn't come back during the lifetime of the test anyway. Can > > you add something like this to the review description?
That's right -- will update the review description. - Neil ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55911/#review163007 ----------------------------------------------------------- On Jan. 25, 2017, 2:38 a.m., Neil Conway wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55911/ > ----------------------------------------------------------- > > (Updated Jan. 25, 2017, 2:38 a.m.) > > > Review request for mesos and Vinod Kone. > > > Bugs: MESOS-6983 > https://issues.apache.org/jira/browse/MESOS-6983 > > > Repository: mesos > > > Description > ------- > > Fixed flakiness in TaskValidationTest.TaskReusesUnreachableTaskID. > > > Diffs > ----- > > src/tests/master_validation_tests.cpp > ce10ea4502d0a13faca28d288dd16cd5cb864f6e > > Diff: https://reviews.apache.org/r/55911/diff/ > > > Testing > ------- > > `make check` > > Without this change, this test fails on most platforms in the internal CI. > With this change, no failures of this test are observed. > > > Thanks, > > Neil Conway > >
