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

[2.7-branch] ci: update download-artifact action to v4 #83067

Open
wants to merge 1 commit into
base: v2.7-branch
Choose a base branch
from

Conversation

nashif
Copy link
Member

@nashif nashif commented Dec 16, 2024

v2 is deprecated and not supported anymore.

Signed-off-by: Anas Nashif anas.nashif@intel.com

v2 is deprecated and not supported anymore.

Signed-off-by: Anas Nashif <anas.nashif@intel.com>
@nashif nashif requested a review from galak as a code owner December 16, 2024 21:25
@nashif nashif changed the title ci: update download-artifact action to v4 [2.7-branch] ci: update download-artifact action to v4 Dec 16, 2024
@nashif nashif requested review from stephanosio, kartben and cfriedt and removed request for galak December 16, 2024 21:31
Copy link
Collaborator

@kartben kartben left a comment

Choose a reason for hiding this comment

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

However, this is a major update that includes breaking changes. Artifacts created with versions v3 and below are not compatible with the v4 actions. Uploads and downloads must use the same major actions versions. There are also key differences from previous versions that may require updates to your workflows.

Please also update upload-artifact versions otherwise it looks like we'll be in trouble :) (even if they do seem already out-of-sync in some places, e.g the codecov action, v4 seems to basically and very explicitly be incompatible with artifacts uploaded with older versions)

Copy link
Member

@cfriedt cfriedt left a comment

Choose a reason for hiding this comment

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

Could do uploads here or in a separate pr.

@kartben
Copy link
Collaborator

kartben commented Dec 17, 2024

Could do uploads here or in a separate pr.

See my comment above. I would strongly recommend doing it in the same PR as GitHub explicitly says uploads from previous versions of the upload action DON'T work
https://github.blog/news-insights/product-news/get-started-with-v4-of-github-actions-artifacts/#compatibility

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants