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

update upload-artifact and download-artifact to v4 #1491

Merged
merged 2 commits into from
Feb 17, 2025

Conversation

ruanyl
Copy link
Member

@ruanyl ruanyl commented Feb 14, 2025

Description

Starting January 30th, 2025, GitHub Actions customers will no longer be able to use v3 of actions/upload-artifact or actions/download-artifact.
This PR updates actions/upload-artifact from v3 to v4.

Issues Resolved

Check List

  • New functionality includes testing.
  • New functionality has been documented.
  • All tests pass
    • yarn lint
    • yarn test-unit
  • Update CHANGELOG.md
  • 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.

Signed-off-by: Yulong Ruan <ruanyl@amazon.com>
Signed-off-by: Yulong Ruan <ruanyl@amazon.com>
@d-buckner
Copy link
Member

Nice! Are these artifacts only used in the CI/CD? Just wondering to make sure there aren't unintended side effects that aren't covered by our PR checks.

@virajsanghvi
Copy link
Collaborator

Nice! Are these artifacts only used in the CI/CD? Just wondering to make sure there aren't unintended side effects that aren't covered by our PR checks.

Also curious on this question

@ashwin-pc
Copy link
Member

I dont think they are used anywhere else today.

@ashwin-pc ashwin-pc merged commit 9d46d68 into opensearch-project:main Feb 17, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants