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

(Updated April 17, 2016, 5:15 p.m.)

Review request for mesos, Alexander Rukletsov, Ben Mahler, Greg Mann, Neil 
Conway, and Timothy Chen.


Update description.

Bugs: MESOS-1802

Repository: mesos

Description (updated)

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 (updated)

  src/tests/health_check_tests.cpp 1c4a554ab07731963a4a38e3ae40b0323bf317bb 

Diff: https://reviews.apache.org/r/46307/diff/


# 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.


haosdent huang

Reply via email to