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

Add CCR tests to Manifest #3267

Merged

Conversation

monusingh-1
Copy link
Contributor

Signed-off-by: Monu Singh [email protected]

Description

All CCR tests to run on multiple remote clusters so that all the eligible tests run during release process.

Issues Resolved

opensearch-project/cross-cluster-replication#532
#2066

Signed-off-by: Monu Singh [email protected]
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.

@codecov-commenter
Copy link

codecov-commenter commented Mar 6, 2023

Codecov Report

Merging #3267 (a7121c3) into main (6191810) will increase coverage by 0.02%.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             main    #3267      +/-   ##
==========================================
+ Coverage   91.72%   91.74%   +0.02%     
==========================================
  Files         172      172              
  Lines        4991     4991              
==========================================
+ Hits         4578     4579       +1     
+ Misses        413      412       -1     
Impacted Files Coverage Δ
src/system/os.py 100.00% <0.00%> (+5.26%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@gaiksaya
Copy link
Member

gaiksaya commented Mar 6, 2023

Hi @monu-aws Can you fix the yaml lint issue?
Thanks!

Signed-off-by: Monu Singh <[email protected]>
@monusingh-1
Copy link
Contributor Author

@gaiksaya, I have fixed the yaml lint issue.
I would also like to know which branch should I backport this change to so that the Integ tests run for 2.7 release as well as for 1.3.9.

@gaiksaya
Copy link
Member

gaiksaya commented Mar 7, 2023

@gaiksaya, I have fixed the yaml lint issue. I would also like to know which branch should I backport this change to so that the Integ tests run for 2.7 release as well as for 1.3.9.

You can add the same to 2.7.0 and 1.3.9 test manifest as well located in https://github.com/opensearch-project/opensearch-build/tree/main/manifests

@gaiksaya gaiksaya merged commit 25a003a into opensearch-project:main Mar 7, 2023
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