Skip to content
This repository has been archived by the owner on Sep 30, 2024. It is now read-only.

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 subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant