Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added 2 simulation tests to explicitly reproduce the issue when the quarantine latency is bigger than 1000ms and verify the fix. #674

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rachelhanhan
Copy link
Contributor

This PR adds 2 simulation tests that simulates the following scenario:
A host is unhealthy and needs to be quarantined, when it's quarantined, the quarantine latency is greater than 1000ms (with 4.0 as low relative factor, avgClusterLatency > 250ms)

I explicitly verified when the last fix is not in place, we will receive the error stating the quarantine latency is greater than 1000ms. After the fix, the quarantine logic works well. These 2 simulation tests gives us confidence about our last fix in 29.21.0.

…uarantine latency is bigger than 1000ms and verify the fix.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant