----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46307/ -----------------------------------------------------------
(Updated April 17, 2016, 3:11 p.m.) Review request for mesos, Ben Mahler, Neil Conway, and Timothy Chen. Bugs: MESOS-1802 https://issues.apache.org/jira/browse/MESOS-1802 Repository: mesos Description ------- We need to ignore subsequent status updates in HealthStatusChange tests. In our test cases, we set `consecutive_failures` to 3 in HealthCheck message definition. But the counter for `consecutiveFailures` in `mesos-health-check` would be reset to 0 after a success check. It is possible to continue to receive status updates before we stop the driver. Diffs ----- src/tests/health_check_tests.cpp 1c4a554ab07731963a4a38e3ae40b0323bf317bb Diff: https://reviews.apache.org/r/46307/diff/ Testing (updated) ------- # I still could not reproduce the problem in old code after repeatedly tests. So seems no way to verify whether my assumption is correct or not. Thanks, haosdent huang