-
Notifications
You must be signed in to change notification settings - Fork 183
Onboard to s3 snapshots #4320
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
Onboard to s3 snapshots #4320
Conversation
Signed-off-by: Darshit Chanpura <[email protected]>
Signed-off-by: Darshit Chanpura <[email protected]>
Hi @dhrubo-os @DarshitChanpura these two tests keep failling, any solutions? Thanks! |
will be fixed here: #4244 |
@DarshitChanpura Why it's passing for other workflows? |
Merging this as this is blocking #4244 which is the fix to this PR check failure issues. Thanks. |
@dhrubo-os I'm not familiar with setup but do other stages invoke security related tests? |
I don't think so, based on this: ml-commons/.github/workflows/CI-workflow.yml Lines 80 to 87 in 7eb0d80
but the docker workflow calls security tests: ml-commons/.github/workflows/CI-workflow.yml Lines 180 to 195 in 7eb0d80
hence other CI passed but docker one fails |
* Onboard to s3 snapshots Signed-off-by: Darshit Chanpura <[email protected]> * Remove oss sonatype maven reference Signed-off-by: Darshit Chanpura <[email protected]> --------- Signed-off-by: Darshit Chanpura <[email protected]> (cherry picked from commit 3950a87)
* Onboard to s3 snapshots Signed-off-by: Darshit Chanpura <[email protected]> * Remove oss sonatype maven reference Signed-off-by: Darshit Chanpura <[email protected]> --------- Signed-off-by: Darshit Chanpura <[email protected]> (cherry picked from commit 3950a87)
* Onboard to s3 snapshots * Remove oss sonatype maven reference --------- (cherry picked from commit 3950a87) Signed-off-by: Darshit Chanpura <[email protected]> Co-authored-by: Darshit Chanpura <[email protected]>
* Onboard to s3 snapshots Signed-off-by: Darshit Chanpura <[email protected]> * Remove oss sonatype maven reference Signed-off-by: Darshit Chanpura <[email protected]> --------- Signed-off-by: Darshit Chanpura <[email protected]> (cherry picked from commit 3950a87)
* Onboard to s3 snapshots Signed-off-by: Darshit Chanpura <[email protected]> * Remove oss sonatype maven reference Signed-off-by: Darshit Chanpura <[email protected]> --------- Signed-off-by: Darshit Chanpura <[email protected]> (cherry picked from commit 3950a87)
* Onboard to s3 snapshots (#4320) * Onboard to s3 snapshots Signed-off-by: Darshit Chanpura <[email protected]> * Remove oss sonatype maven reference Signed-off-by: Darshit Chanpura <[email protected]> --------- Signed-off-by: Darshit Chanpura <[email protected]> (cherry picked from commit 3950a87) * Update branch patterns for Maven publish workflow Signed-off-by: Peter Zhu <[email protected]> --------- Signed-off-by: Darshit Chanpura <[email protected]> Signed-off-by: Peter Zhu <[email protected]> Co-authored-by: Darshit Chanpura <[email protected]> Co-authored-by: Peter Zhu <[email protected]>
* Onboard to s3 snapshots (#4320) * Onboard to s3 snapshots Signed-off-by: Darshit Chanpura <[email protected]> * Remove oss sonatype maven reference Signed-off-by: Darshit Chanpura <[email protected]> --------- Signed-off-by: Darshit Chanpura <[email protected]> (cherry picked from commit 3950a87) * Update branch patterns for Maven publish workflow Signed-off-by: Peter Zhu <[email protected]> --------- Signed-off-by: Darshit Chanpura <[email protected]> Signed-off-by: Peter Zhu <[email protected]> Co-authored-by: Darshit Chanpura <[email protected]> Co-authored-by: Peter Zhu <[email protected]>
Description
Related to opensearch-project/opensearch-build#5360
Check List
--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.