-
Notifications
You must be signed in to change notification settings - Fork 164
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
Removes branch filter on test workflow #930
Removes branch filter on test workflow #930
Conversation
ITs failed. |
yes...because it is unable to find the latest 1.3.1.0 image for security-plugin:
|
In this change you also need to resolve the bad references in the integration test run for 1.3 |
Should I revert the wget commands in the ymls to use 1.3.0.0 instead of 1.3.1.0? Or is it something else that you are pointing to? |
This change is being merged onto 1.3 and from the error, the 1.3.1 build isn't resolving as its getting a 403. How are we planning on fixing this?
|
[Offline] We haven't yet added the security backend changes into the 1.3.1 manifest, after this has been done and a build has been created the URL will start to work. |
Once this PR: opensearch-project/opensearch-build#1840 gets merged we can re-run the ITs for this branch. |
PR: #1840 has been merged and I have re-run IT job now but it is still unable to locate the 1.3.1.0 artifact. |
…rkflows Signed-off-by: Darshit Chanpura <[email protected]>
2c592ae
to
e2c6a25
Compare
@cliu123 @peternied can i get your review on this PR? |
Description
Updates unit and integration test workflow to run on all branches rather than just the main branch
Category
Maintenance
Why these changes are required?
To run test on all PRs
Check List
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.