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

Pin test-summary action to v2.2 to avoid the Node 20 requirement until RAPIDS drops CentOS 7 support. #178

Merged
merged 1 commit into from
Feb 22, 2024

Conversation

bdice
Copy link
Contributor

@bdice bdice commented Feb 22, 2024

The test-summary GitHub Action was updated to require Node 20, which doesn't work with CentOS 7. (Version 2.3, released 6 hours ago)
https://github.com/test-summary/action/releases/tag/v2.3

We'll have to pin to test-summary/[email protected] in shared-workflows so that they remain compatible with CentOS 7.

Generally, most GitHub Actions are moving to require Node 20, and we're unable to support that on CentOS 7 runners. We'll be dropping CentOS 7 support in RAPIDS 24.06, but we'll have to pin GitHub Actions to compatible (Node 16) versions until then. https://docs.rapids.ai/notices/rsn0037/

@bdice bdice changed the title Pin test-summary action to v2.2 to avoid the Node 20 requirement until we drop CentOS 7. Pin test-summary action to v2.2 to avoid the Node 20 requirement until RAPIDS drops CentOS 7 support. Feb 22, 2024
@bdice bdice requested a review from ajschmidt8 February 22, 2024 16:47
@bdice bdice merged commit 32fa139 into rapidsai:branch-24.04 Feb 22, 2024
@jakirkham
Copy link
Member

Thanks you both! 🙏

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