Skip to content

Fix version mismatch between OpenSearch and Dashboards in CI binary installation workflow #205

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

Merged

Conversation

KishoreKicha14
Copy link
Contributor

Description

This PR updates the GitHub Actions workflow to ensure compatibility between OpenSearch and OpenSearch Dashboards versions during binary installation testing for the Query Insights Dashboards plugin.

Set both OPENSEARCH_VERSION and Dashboards version to 3.0.0 to avoid compatibility errors.

Prevents failure due to saved objects migration blocking startup when versions are mismatched.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Kishore Kumaar Natarajan added 2 commits June 6, 2025 12:34
Signed-off-by: Kishore Kumaar Natarajan <[email protected]>
Signed-off-by: Kishore Kumaar Natarajan <[email protected]>
@ansjcy
Copy link
Member

ansjcy commented Jun 6, 2025

Cypress test failed to bootstrap; looks like there are some issues when running the cluster. Could you double check?

@KishoreKicha14
Copy link
Contributor Author

Cypress test failed to bootstrap; looks like there are some issues when running the cluster. Could you double check?

Yeah Chenyang working on updating the Cypress workflow

@KishoreKicha14 KishoreKicha14 force-pushed the updates-the-GitHub-Actions branch from 961a95d to 3651f96 Compare June 9, 2025 08:47
Kishore Kumaar Natarajan added 3 commits June 9, 2025 01:57
Signed-off-by: Kishore Kumaar Natarajan <[email protected]>
Signed-off-by: Kishore Kumaar Natarajan <[email protected]>
Signed-off-by: Kishore Kumaar Natarajan <[email protected]>
Copy link
Member

@ansjcy ansjcy left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the fix!
one of the cypress test is failing, but I'll merge this PR now to unblock the pipeline.

@ansjcy ansjcy merged commit 01fa41d into opensearch-project:main Jun 9, 2025
8 of 10 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.

3 participants