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

tests/upgrade: Allow DBs to settle before post-upgrade test. #96

Merged
merged 1 commit into from
Nov 6, 2023

Conversation

fnordahl
Copy link
Member

@fnordahl fnordahl commented Nov 5, 2023

The upgrade test currently performs an upgrade of all nodes in quick succession, and then immediately runs tests. This leads to spurious errors because the database clusters may or may not have settled on a new leader after all nodes being restarted.

Use ovsdb-client to check that each member of database cluster is connected prior to continuning with the test.

The upgrade test currently performs an upgrade of all nodes in
quick succession, and then immediately runs tests.  This leads to
spurious errors because the database clusters may or may not have
settled on a new leader after all nodes being restarted.

Use `ovsdb-client` to check that each member of database cluster
is connected prior to continuning with the test.

Signed-off-by: Frode Nordahl <[email protected]>
Copy link
Contributor

@mkalcok mkalcok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@fnordahl fnordahl merged commit 0e37df3 into canonical:main Nov 6, 2023
17 checks passed
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.

2 participants