Skip to content

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