----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65348/ -----------------------------------------------------------
(Updated Jan. 26, 2018, 2:31 p.m.) Review request for mesos, Alexander Rukletsov and Benjamin Bannier. Changes ------- Removed TODO. Bugs: MESOS-8490 https://issues.apache.org/jira/browse/MESOS-8490 Repository: mesos Description (updated) ------- Under rare circumstances, the clock would have to be advances to trigger a new allocation cycle after 'UPDATE_SLAVE' has been handled in the master. We also wait for old offers to be rescinded before waiting for updated offers as sometimes the old offer could be received erroneously. Diffs (updated) ----- src/tests/master_tests.cpp 7112bb4efcc312dc4f68bdc44fb685c7624002b1 Diff: https://reviews.apache.org/r/65348/diff/2/ Changes: https://reviews.apache.org/r/65348/diff/1-2/ Testing ------- make check `src/mesos-tests --gtest_filter=MasterTest.UpdateSlaveMessageWithPendingOffers --gtest_repeat=10000 --gtest_break_on_failure` while running `stress -c 4 -i 8 -m 2 -d 6` on a 4 core machine. It usually takes a few thousand iterations to trigger the flakyness in the old code. And only when the host is under heavy load. Thanks, Jan Schlicht