----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21968/ -----------------------------------------------------------
Review request for mesos, Adam B and Niklas Nielsen. Bugs: MESOS-878 https://issues.apache.org/jira/browse/MESOS-878 Repository: mesos-git Description ------- Added a sentence to check the slave's state in doReliableRegistration() to make sure that the function returns when the slave's state is TERMINATING. Also, to write the test, I need to make sure there's no additional doReliableRegistration() calls, so I added NO_FUTURE_PROTOBUFS() and NO_FUTURE_MESSAGES() to make sure there's no more SlaveReregisteredMessages. Diffs ----- 3rdparty/libprocess/include/process/gmock.hpp daba7e3 src/slave/slave.cpp 7fbedb1 src/tests/mesos.hpp 1b0f358 src/tests/slave_tests.cpp 80fe3cf Diff: https://reviews.apache.org/r/21968/diff/ Testing ------- Slave_tests.cpp:TerminatingSlaveDoesNotReregister Thanks, Yifan Gu