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

CI/CD: Bump upload-artifact to v4 #1592

Merged
merged 1 commit into from
Oct 25, 2024

Conversation

cmuellner
Copy link
Collaborator

We recently updated the download-artifact action to v4, because earlier versions were deprecated and stopped working.

Now we ran into the situation, that the release pipeline broke, because no toolchains are found by the download-artifact action ("Found 0 artifact(s)" although all toolchain builds succeeded and the resulting toolchain was successfully archived and uploaded).

Looking into the documention of download-artifact@v4 shows the issue: One of the breaking changes of download-artifact@v4 is "Downloading artifacts that were created from action/upload-artifact@v3 and below are not supported." ([1]). Our pipeline uses upload-artifact@v3, which is why they are not found later on.

Let's bump the upload-artifact action to v4 so that download-artifact@v4 will find the built artifacts again.

[1] https://github.com/actions/download-artifact?tab=readme-ov-file#v4---whats-new

We recently updated the download-artifact action to v4, because earlier
versions were deprecated and stopped working.

Now we ran into the situation, that the release pipeline broke, because
no toolchains are found by the download-artifact action
("Found 0 artifact(s)" although all toolchain builds succeeded and the
resulting toolchain was successfully archived and uploaded).

Looking into the documention of download-artifact@v4 shows the issue:
One of the breaking changes of download-artifact@v4 is "Downloading
artifacts that were created from action/upload-artifact@v3 and below
are not supported." ([1]). Our pipeline uses upload-artifact@v3, which
is why they are not found later on.

Let's bump the upload-artifact action to v4 so that download-artifact@v4
will find the built artifacts again.

[1] https://github.com/actions/download-artifact?tab=readme-ov-file#v4---whats-new

Signed-off-by: Christoph Müllner <[email protected]>
@pz9115
Copy link
Contributor

pz9115 commented Oct 23, 2024

LGTM,thanks!

@cmuellner
Copy link
Collaborator Author

I'm merging this without a formal review from somebody else.
This PR was available for two days and people are waiting for the CI/CD to work again.

@cmuellner cmuellner merged commit c4e32b3 into riscv-collab:master Oct 25, 2024
26 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.

2 participants