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 upload-artifact to version 3 #1252

Merged
merged 3 commits into from
Sep 16, 2024

Conversation

Hailong-am
Copy link
Collaborator

@Hailong-am Hailong-am commented Sep 14, 2024

Description

upgrade upload-artifact to version 3 as v1 and v2 are deprecated per https://github.blog/changelog/2024-02-13-deprecation-notice-v1-and-v2-of-the-artifact-actions/

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.

@Hailong-am Hailong-am added maintenance improves code quality, but not the product backport 2.x labels Sep 14, 2024
Signed-off-by: Hailong Cui <[email protected]>
@Hailong-am Hailong-am added the v2.18.0 Issues targeting release v2.18.0 label Sep 14, 2024
@gargrajat
Copy link

@Hailong-am Can you check the linux workflows which are failing?

with:
name: index-management-plugin-ubuntu-latest
name: index-management-plugin-ubuntu-latest-${{ matrix.java }}

Choose a reason for hiding this comment

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

This probably should have
name: index-management-plugin-${{ matrix.os }}-${{ matrix.java }}

Signed-off-by: Hailong Cui <[email protected]>
@Hailong-am Hailong-am changed the title upgrade upload-artifact to version 4 upgrade upload-artifact to version 3 Sep 16, 2024
@Hailong-am
Copy link
Collaborator Author

CI failures are not related to upload-artifact

@vikasvb90 vikasvb90 merged commit 58add11 into opensearch-project:main Sep 16, 2024
27 of 29 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Sep 16, 2024
* upgrade upload-artifact to version 4

Signed-off-by: Hailong Cui <[email protected]>

* add overwrite as true

Signed-off-by: Hailong Cui <[email protected]>

* use v3 as v4 need node.js 20

Signed-off-by: Hailong Cui <[email protected]>

---------

Signed-off-by: Hailong Cui <[email protected]>
(cherry picked from commit 58add11)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.x maintenance improves code quality, but not the product v2.18.0 Issues targeting release v2.18.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants