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

[Campaign] Add distribution level Backward Compatiblity tests to all plugins #3671

Open
zelinh opened this issue Jun 23, 2023 · 6 comments
Open

Comments

@zelinh
Copy link
Member

zelinh commented Jun 23, 2023

Followup to this issue https://github.com/opensearch-build/issues/2870 when we onboarded BWC distribution tests to few plugins that have BWC tests running on plugin level already. However, there are numbers of plugins that are not included in our CI BWC tests on distribution bundle basis.

The reason we didn't onboard these plugins are:

  • Plugins were new and recently added to our OpenSearch distribution so they don't have their own BWC tests yet.
  • Plugins' BWC tests on plugins level were failing consistently and need fixed.

We are planning to add BWC tests to all plugins that we launch with the distribution bundle. This issue serves as a tracker to keep track of the BWC tests status of each plugins.

Component Plugin level BWC tests Distribution level BWC tests Comment
alerting
anomaly-detection
asynchronous-search opensearch-project/asynchronous-search#517
reporting opensearch-project/reporting#946
index-management
k-NN opensearch-project/k-NN#1434
performance-analyzer opensearch-project/performance-analyzer#614
security opensearch-project/security#4005
sql
observability
notifications
cross-cluster-replication opensearch-project/cross-cluster-replication#1314
geospatial opensearch-project/geospatial#5
ml-commons opensearch-project/ml-commons#1962
security-analytics opensearch-project/security-analytics#470
neural-search opensearch-project/neural-search#202
custom-codecs opensearch-project/custom-codecs#167
flow-framework opensearch-project/flow-framework#787
skills opensearch-project/skills#354
@github-actions github-actions bot added the untriaged Issues that have not yet been triaged label Jun 23, 2023
@zelinh zelinh added test ci-test-automation and removed untriaged Issues that have not yet been triaged labels Jun 26, 2023
@gaiksaya
Copy link
Member

Hi @zelinh @bbarani Do we need to create child issues in all repositories missing the distribution level BWC tests?

@bbarani
Copy link
Member

bbarani commented Sep 18, 2023

@zelinh Did we not create child issues in corresponding repos?

@rishabh6788
Copy link
Collaborator

@zelinh to create child issues and tag here.

@zelinh
Copy link
Member Author

zelinh commented Jan 30, 2024

Child issues for requests on BWC distribution level tests are created in each component repo which doesn't have it right now. This issue will serve as an campaign to keep track of each of those issues.

@jmazanec15
Copy link
Member

@zelinh is this change applicable to both 2.x and main? In AD, I see it was only added to 2.x branch: opensearch-project/anomaly-detection#766.

@zelinh
Copy link
Member Author

zelinh commented Feb 2, 2024

@zelinh is this change applicable to both 2.x and main? In AD, I see it was only added to 2.x branch: opensearch-project/anomaly-detection#766.

Since main branch is a major version change of OpenSearch. We expect there might/might not be breaking changes that fails the test and main branch is still under development. That's why we didn't add distribution level BWC test to main yet. If you find out that it's running on main branch. I'm good to add it now.

@zelinh zelinh added the v2.17.0 label Aug 14, 2024
@prudhvigodithi prudhvigodithi moved this from 🏗 In progress to Backlog in Engineering Effectiveness Board Sep 9, 2024
@peterzhuamazon peterzhuamazon moved this from Backlog to Now(This Quarter) in Engineering Effectiveness Board Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Now(This Quarter)
Status: In Progress
Development

No branches or pull requests

6 participants