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] Update upload-artifacts and download-artifcats. #8090

Merged
merged 1 commit into from
Jan 16, 2025

Conversation

mikeurbach
Copy link
Contributor

The v3 of these actions is deprecated:
https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/.

While GitHub has indicated this will stop working on January 30, we seem to have already hit a failure related to this: https://github.com/llvm/circt/actions/runs/12815228763/job/35733456176?pr=8089#step:1:25.

This updates to use the v4 version of these actions where we were on v3, and in the Python wheel building specifically, it makes one small tweak following the migration guide:
https://github.blog/news-insights/product-news/get-started-with-v4-of-github-actions-artifacts/#compatibility.

The v3 of these actions is deprecated:
https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/.

While GitHub has indicated this will stop working on January 30, we
seem to have already hit a failure related to this:
https://github.com/llvm/circt/actions/runs/12815228763/job/35733456176?pr=8089#step:1:25.

This updates to use the v4 version of these actions where we were on
v3, and in the Python wheel building specifically, it makes one small
tweak following the migration guide:
https://github.blog/news-insights/product-news/get-started-with-v4-of-github-actions-artifacts/#compatibility.
@mikeurbach mikeurbach requested a review from teqdruid as a code owner January 16, 2025 18:45
@mikeurbach
Copy link
Contributor Author

mikeurbach commented Jan 16, 2025

Manually kicked off the workflows that aren't tested in pre-merge.

unifiedBuildTestAndInstall.yml: https://github.com/llvm/circt/actions/runs/12816324652

uploadWheels.yml (including push to PyPI): https://github.com/llvm/circt/actions/runs/12815570875

Copy link
Contributor

@teqdruid teqdruid left a comment

Choose a reason for hiding this comment

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

I'm probably gonna have to update the esi runtime and pycde pipelines as well.

@mikeurbach
Copy link
Contributor Author

@teqdruid I took a look for anything using these actions, and ESI and PyCDE are already on v4.

@teqdruid
Copy link
Contributor

Excellent! Looks like I'm ahead of the curve. Thanks for checking.

@mikeurbach
Copy link
Contributor Author

After a couple false start (pebkac), the Build, Test, and Install looks good.

Screen Shot 2025-01-16 at 12 45 15 PM

@mikeurbach
Copy link
Contributor Author

mikeurbach commented Jan 16, 2025

Upload Wheels looks good too.

Screen Shot 2025-01-16 at 2 33 50 PM
Screen Shot 2025-01-16 at 2 34 02 PM

@mikeurbach mikeurbach merged commit 730c9cd into main Jan 16, 2025
9 of 11 checks passed
@mikeurbach mikeurbach deleted the mikeurbach/deprecated-action branch January 16, 2025 21:35
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.

3 participants