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

Upgrade github workflow upload artifact to v3 #1656

Merged
merged 1 commit into from
Sep 10, 2024

Conversation

jowg-amazon
Copy link
Collaborator

@jowg-amazon jowg-amazon commented Sep 10, 2024

Description

upload-artifact@v1 and upload-artifact@v2 was deprecated. This pr upgrades the github test workflow upload-artifact from @v1 to @V3.

Related Issues

Resolves #[Issue number to be closed when this PR is merged]

Check List

  • New functionality includes testing.
  • New functionality has been documented.
  • API changes companion pull request created.
  • Commits are signed per the DCO using --signoff.
  • Public documentation issue/PR created.

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: Joanne Wang <jowg@amazon.com>
@jowg-amazon jowg-amazon changed the title Upgrade github workflow upload artifact to v4 Upgrade github workflow upload artifact to v3 Sep 10, 2024
@jowg-amazon jowg-amazon marked this pull request as ready for review September 10, 2024 21:30
Copy link
Collaborator

@AWSHurneyt AWSHurneyt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jowg-amazon If possible, could you also have someone from the release team take a look at this PR to make sure it's correct?

@jowg-amazon
Copy link
Collaborator Author

@jowg-amazon If possible, could you also have someone from the release team take a look at this PR to make sure it's correct?

@gaiksaya confirmed that changing this version doesn't affect any release related artifacts and other plugins are also currently on v3 (ex. https://github.com/opensearch-project/job-scheduler/blob/1c282b0342183a743e166af9650b7c0bde68a11c/.github/workflows/ci.yml#L55)

@jowg-amazon jowg-amazon merged commit e113dbb into opensearch-project:main Sep 10, 2024
18 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Sep 10, 2024
Signed-off-by: Joanne Wang <jowg@amazon.com>
(cherry picked from commit e113dbb)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Sep 10, 2024
Signed-off-by: Joanne Wang <jowg@amazon.com>
(cherry picked from commit e113dbb)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
AWSHurneyt pushed a commit that referenced this pull request Sep 17, 2024
(cherry picked from commit e113dbb)

Signed-off-by: Joanne Wang <jowg@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
AWSHurneyt pushed a commit that referenced this pull request Sep 18, 2024
(cherry picked from commit e113dbb)

Signed-off-by: Joanne Wang <jowg@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Sep 26, 2024
(cherry picked from commit e113dbb)

Signed-off-by: Joanne Wang <jowg@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
(cherry picked from commit 01a2f64)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Sep 26, 2024
(cherry picked from commit e113dbb)

Signed-off-by: Joanne Wang <jowg@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
(cherry picked from commit 01a2f64)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
sbcd90 pushed a commit that referenced this pull request Sep 26, 2024
Signed-off-by: Joanne Wang <jowg@amazon.com>
sbcd90 pushed a commit that referenced this pull request Sep 26, 2024
Signed-off-by: Joanne Wang <jowg@amazon.com>
sbcd90 pushed a commit to sbcd90/alerting that referenced this pull request Sep 26, 2024
eirsep pushed a commit that referenced this pull request Sep 26, 2024
Signed-off-by: Joanne Wang <jowg@amazon.com>
Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com>
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