Skip to content
This repository was archived by the owner on Aug 2, 2022. It is now read-only.

Fix multiversion test failure - merge 2.1.x #10054

Merged
merged 11 commits into from
Feb 17, 2021

Conversation

bogniq
Copy link
Collaborator

@bogniq bogniq commented Feb 17, 2021

Change Description

The multiversion unit test is fixed to resolve occasional test failures, which mainly occurred at three checking points:

  1. Nodes are not in sync before preactivation after the testing cluster is launched. To fix this, areNodesInSync method adds a waiting time when checking if nodes are in sync.
  2. 1st node should contain PREACTIVATE FEATURE after the 1st node tries to activate preactivate feature. The fix adds retries to activate the feature and then check its status.
  3. 1st node should be out of sync with the rest nodes after the 1st node has activated the preactivate feature. This is fixed by not resuming (and then re-pausing) all nodes until the sync check is finished.

Thank @brianjohnson5972 for the help.

Change Type

Select ONE:

  • Documentation
  • Stability bug fix
  • Other
  • Other - special case

Testing Changes

Select ANY that apply:

  • New Tests
  • Existing Tests
  • Test Framework
  • CI System
  • Other

Consensus Changes

  • Consensus Changes

API Changes

  • API Changes

Documentation Additions

  • Documentation Additions

@brianjohnson5972 brianjohnson5972 changed the title Fix multiversion test failure Fix multiversion test failure - merge 2.1.x Feb 17, 2021
@bogniq bogniq merged commit 2b907ca into release/2.1.x Feb 17, 2021
@bogniq bogniq deleted the qy-multiversion-test-failure-2.1.x branch February 17, 2021 17:42
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.

2 participants