mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Bannier <>
Subject Review Request 71092: Added additionaly synchronization to a previously flaky test.
Date Wed, 17 Jul 2019 12:02:58 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for mesos, Benno Evers, Greg Mann, and Joseph Wu.

Bugs: MESOS-9895

Repository: mesos


This test did not make sure that the agent's task status update manager
had received a task status update acknowledgemtn before launching a
subsequent task. The agent then, given its state, correctly rejected the

This patch adds additional synchronization so the agent has finished
processing the task status update like expected.


  src/tests/slave_tests.cpp 1ed59ca5af69a6b53ceb8be035e608286299c2f1 



This test previously failed about 4% of the time in ~400 iterations (~20 failures) under a
lot of additional system stress. With this patch it did not fail once in 500 iterations.


Benjamin Bannier

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message