-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31114/
-----------------------------------------------------------
(Updated Feb. 18, 2015, 10:05 a.m.)
Review request for mesos, Ben Mahler, Till Toenshoff, and Vinod Kone.
Changes
-------
Uses a `Not()` matcher to prevent catching messages intended to different
destinataries.
Bugs: MESOS-2355
https://issues.apache.org/jira/browse/MESOS-2355
Repository: mesos-incubating
Description
-------
Sets the "to" end to all calls to `FUTURE_PROTOBUF` within the
MasterTest.SlavesEndpointTwoSlaves.
Failing to do that causes the test to fail on very rare ocasions, due to the
first slave registering twice to the server (this is normal behavior according
to
[slave.cpp](https://github.com/apache/mesos/blob/master/src/slave/slave.cpp#L1087)
and
[master.cpp](https://github.com/apache/mesos/blob/master/src/master/master.cpp#L2912).
It those ocasions, where the first slave registers twice, the master's sends a
reponse to both registration requests before the second slave's turn, thus
setting the future which was expecting data for the second slave with incorrect
data.
Diffs (updated)
-----
src/tests/master_tests.cpp e69a7fb39e93a9a999e0991985f5d060b72aac65
Diff: https://reviews.apache.org/r/31114/diff/
Testing
-------
/bin/mesos-tests.sh --gtest_filter="MasterTest.SlavesEndpoint*"
--gtest_repeat=1000 --gtest_break_on_failure
Thanks,
Alexander Rojas
