- 
                Notifications
    You must be signed in to change notification settings 
- Fork 3
15084 add on test and obs da split #269
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
Conversation
| /run pipeline | 
| /run pipeline | 
| /run pipeline | 
| /run pipeline | 
| /run pipeline | 
| /run pipeline | 
| /run pipeline | 
Co-authored-by: Shikha Maheshwari <[email protected]>
Co-authored-by: Shikha Maheshwari <[email protected]>
| /run pipeline | 
| /run pipeline | 
| @mukulpalit-ibm What is the issue with the addons test? Im not going to approve this PR until those test pass | 
| 
 The permutation tests are still not ready to be released. The pipelines fail with error from project backend even though all the test cases pass Therefore skipping permutation tests | 
| /run pipeline | 
| @mukulpalit-ibm Please link the GIT issue tracking this - this will block this PR | 
| FYI, the permutation tests just passed for Secrets Manager PR | 
| 
 It is actually happening with hit and try sometimes it passes sometimes it fails | 
| 🎉 This PR is included in version 1.10.0 🎉 The release is available on: 
 Your semantic-release bot 📦🚀 | 

Description
This PR splits the add on Observability DA as well as add the default and permutation add-ons testing to pr_test
Release required?
x.x.X)x.X.x)X.x.x)Release notes content
Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
Checklist for reviewers
For mergers