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 & update-artifact actions to v4 #83067

Conversation

nashif
Copy link
Member

@nashif nashif commented Dec 16, 2024

v2 is deprecated and not supported anymore.

Signed-off-by: Anas Nashif [email protected]

@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)

stephanosio
stephanosio previously approved these changes Dec 16, 2024
cfriedt
cfriedt previously approved these changes Dec 17, 2024
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

v2 & v3 are deprecated and not supported anymore.

Signed-off-by: Anas Nashif <[email protected]>
Co-authored-by: Benjamin Cabé <[email protected]>
@kartben kartben dismissed stale reviews from cfriedt and stephanosio via e480d0a January 9, 2025 16:11
@kartben kartben force-pushed the topic/ci/download_artifact_version_v2 branch from 909195f to e480d0a Compare January 9, 2025 16:11
@kartben kartben changed the title [2.7-branch] ci: update download-artifact action to v4 [2.7-branch] ci: update download-artifact & update-artifact actions to v4 Jan 9, 2025
@kartben
Copy link
Collaborator

kartben commented Jan 9, 2025

@nashif I wonder if the update can be tested on zephyr-testing first, just in case?

@cfriedt
Copy link
Member

cfriedt commented Jan 9, 2025

Assuming all of this will work without a hitch ;-) Merge away @nashif

@nashif
Copy link
Member Author

nashif commented Jan 9, 2025

@nashif I wonder if the update can be tested on zephyr-testing first, just in case?

we can try this if it still fails after merging, there is nothing to lose right now, it is broken anyways.

@nashif nashif merged commit 62ad640 into zephyrproject-rtos:v2.7-branch Jan 9, 2025
12 of 14 checks passed
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