-
Notifications
You must be signed in to change notification settings - Fork 105
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 option to run bwc tests against bundle #735
Conversation
Signed-off-by: Zelin Hao <[email protected]>
Codecov Report
@@ Coverage Diff @@
## 2.x #735 +/- ##
============================================
- Coverage 75.31% 75.24% -0.08%
Complexity 116 116
============================================
Files 125 125
Lines 6819 6819
Branches 1020 1020
============================================
- Hits 5136 5131 -5
- Misses 1153 1155 +2
- Partials 530 533 +3
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
Signed-off-by: Zelin Hao <[email protected]>
@zelinh Is there a reason this cant be merged to main instead of 2.x? |
@lezzago Currently the distribution build for our 3.0.0 are not stable and I haven't validated these tests for major version update. So I'm targeting to 2.5 version for now. I could draft a new PR to backport on main when we getting close to it. |
Why do we need the new security credentials files? We already have them here for the security tests: https://github.com/opensearch-project/alerting/tree/2.x/alerting/src/test/resources |
* Add option to run bwc tests against bundle Signed-off-by: Zelin Hao <[email protected]> * Move security certs Signed-off-by: Zelin Hao <[email protected]> Signed-off-by: Zelin Hao <[email protected]> (cherry picked from commit 433f4c9)
* Add option to run bwc tests against bundle Signed-off-by: Zelin Hao <[email protected]> * Move security certs Signed-off-by: Zelin Hao <[email protected]> Signed-off-by: Zelin Hao <[email protected]> (cherry picked from commit 433f4c9) Co-authored-by: Zelin Hao <[email protected]>
Signed-off-by: Zelin Hao [email protected]
Description
Add option to run BWC tests in distribution level, which is running tests in the test cluster with latest distribution bundle installed and all plugins (included in the latest manifest) present.
The command to run BWC tests in distribution level would be
./gradlew bwcTestSuite -Dtests.security.manager=false -PcustomDistributionDownloadType=bundle
If property
customDistributionDownloadType
is not set nor set tobundle
, the BWC tests would be default to run in plugins level as previously configured.Issues Resolved
Part of opensearch-project/opensearch-build#2870
CheckList:
[ ] Commits are signed per the DCO using --signoff
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.