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

Orchestrator analysis-locked-hypothesis test is unstable #1465

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

Conversation

kamil-holubicki
Copy link

#1464

Problem:
analysis-locked-hypothesis test is unstable

Cause:
By default instance is polled every 5 seconds, but we wait only for 2 seconds, so it may happen that instance is not polled before we get the analysis result.

Solution:
Wait for 7 seconds after disabling instance. It is 7 because it has to be > 5 but LockedSemiSyncMasterHypothesis is reported only within 6 secs window (InstancePollSeconds + ReasonableInstanceCheckSeconds), then it switches to LockedSemiSyncMaster status.

Related issue: https://github.com/openark/orchestrator/issues/0123456789

Description

This PR [briefly explain what is does]

openark#1464

Problem:
analysis-locked-hypothesis test is unstable

Cause:
By default instance is polled every 5 seconds, but we wait only for
2 seconds, so it may happen that instance is not polled before we
get the analysis result.

Solution:
Wait for 7 seconds after disabling instance. It is 7 because it has
to be > 5 but LockedSemiSyncMasterHypothesis is reported only within
6 secs window (InstancePollSeconds + ReasonableInstanceCheckSeconds),
then it switches to LockedSemiSyncMaster status.
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.

None yet

1 participant