----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69656/#review211965 -----------------------------------------------------------
PASS: Mesos patch 69656 was successfully built and tested. Reviews applied: `['69656']` All the build artifacts available at: http://dcos-win.westus2.cloudapp.azure.com/artifacts/mesos-reviewbot-testing/2773/mesos-review-69656 - Mesos Reviewbot Windows On Jan. 3, 2019, 5:16 a.m., Meng Zhu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/69656/ > ----------------------------------------------------------- > > (Updated Jan. 3, 2019, 5:16 a.m.) > > > Review request for mesos, Joseph Wu and Till Toenshoff. > > > Bugs: MESOS-5048 > https://issues.apache.org/jira/browse/MESOS-5048 > > > Repository: mesos > > > Description > ------- > > `MesosContainerizerSlaveRecoveryTest.ResourceStatistics` is flaky > due to a race between executor shutdown (due to never getting any > tasks) and the test querying resource statistics. If the executor > is shutdown before the statistics query, the test will fail. > > This patch fixes the test by explicitly waiting for the task to > be delivered and task status transition to `TASK_RUNNING` before > restarting the agent. This way, the executor will not be shutdown > after agent restart. Hence there will be no race. > > > Diffs > ----- > > src/tests/slave_recovery_tests.cpp 0eb47e2bdf6a46fc21b59bb85b4b89181087ccd3 > > > Diff: https://reviews.apache.org/r/69656/diff/1/ > > > Testing > ------- > > ran `MesosContainerizerSlaveRecoveryTest.ResourceStatistics` continuously > without failure. > > > Thanks, > > Meng Zhu > >
