----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69656/#review212468 -----------------------------------------------------------
Patch looks great! Reviews applied: [69656] Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose --disable-libtool-wrappers --disable-parallel-test-execution' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker-build.sh - Mesos Reviewbot 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 > >
