mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Greg Mann <>
Subject Review Request 69581: Fixed an interleaving bug on the master actor.
Date Tue, 18 Dec 2018 09:06:49 GMT

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

Review request for mesos, Benno Evers, Chun-Hung Hsiao, Gastón Kleiman, and Meng Zhu.

Bugs: MESOS-9460

Repository: mesos


Upon receipt of an operation via the operator API, the
master would previously update the allocator and then
update its `Slave` struct in two different continuations.
This created the possibility that the allocator could be
updated in between those two continuations, leading to
inconsistency between the allocator state and the
master's state.

This patch invokes these two blocks of code
synchronously to avoid this issue.


  src/master/master.cpp ae5b24093156a6ba8ffa3e94a46e600eb4def5fa 



Ran the test in the subsequent patch both before and after this fix. Before the fix, the test
fails reliably. After the fix, the test passes reliably.


Greg Mann

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