Skip to content

ci: use GitHub-hosted Linux runners#50

Merged
willkill07 merged 1 commit into
NVIDIA:mainfrom
willkill07:refactor/github-hosted-linux-runners
May 3, 2026
Merged

ci: use GitHub-hosted Linux runners#50
willkill07 merged 1 commit into
NVIDIA:mainfrom
willkill07:refactor/github-hosted-linux-runners

Conversation

@willkill07
Copy link
Copy Markdown
Member

@willkill07 willkill07 commented May 3, 2026

Overview

Switch Linux CI jobs from custom runner labels to standard GitHub-hosted runner labels.

  • I confirm this contribution is my own work, or I have the right to submit it under this project's license.
  • I searched existing issues and open pull requests, and this does not duplicate existing work.

Details

  • Replaces linux-amd64-cpu4 with ubuntu-latest across workflow jobs and Linux AMD64 matrix entries.
  • Replaces linux-arm64-cpu4 with ubuntu-24.04-arm across Linux ARM64 matrix entries.
  • Leaves .github/actionlint.yaml unchanged so the existing self-hosted runner allow-list remains in place.

Where should the reviewer start?

Start with .github/workflows/ci_rust.yml, then compare the same matrix-label change across the Go, Node.js, Python, and WebAssembly workflows.

Related Issues: (use one of the action keywords Closes / Fixes / Resolves / Relates to)

  • Relates to: none

Summary by CodeRabbit

  • Chores
    • Updated CI/CD workflows to use GitHub-hosted runners for build and test jobs across all platforms (Go, Node.js, Python, Rust, and WebAssembly).

Signed-off-by: Will Killian <wkillian@nvidia.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 3, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Enterprise

Run ID: ca1af4e8-7629-4ce0-b3a3-fcb5e0aa3253

📥 Commits

Reviewing files that changed from the base of the PR and between c200917 and 1ba45cc.

📒 Files selected for processing (7)
  • .github/workflows/ci_check.yml
  • .github/workflows/ci_docs.yml
  • .github/workflows/ci_go.yml
  • .github/workflows/ci_node.yml
  • .github/workflows/ci_python.yml
  • .github/workflows/ci_rust.yml
  • .github/workflows/ci_wasm.yml

Walkthrough

GitHub Actions CI workflows are updated to use GitHub-hosted runners instead of custom CPU4 runners. Linux AMD64 jobs now target ubuntu-latest, and Linux ARM64 jobs now target ubuntu-24.04-arm, across all language and build pipelines.

Changes

CI Runner Migration

Layer / File(s) Summary
Standalone Job Configuration
​.github/workflows/ci_check.yml, ​.github/workflows/ci_docs.yml
Runner label updated from linux-amd64-cpu4 to ubuntu-latest.
Matrix-based Job Configuration
​.github/workflows/ci_go.yml, ​.github/workflows/ci_node.yml, ​.github/workflows/ci_python.yml, ​.github/workflows/ci_rust.yml, ​.github/workflows/ci_wasm.yml
Matrix entries updated: linux-amd64-cpu4ubuntu-latest and linux-arm64-cpu4ubuntu-24.04-arm.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Review rate limit: 9/10 reviews remaining, refill in 6 minutes.

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot added size:S PR is small ci labels May 3, 2026
@willkill07 willkill07 marked this pull request as ready for review May 3, 2026 22:05
@willkill07 willkill07 requested a review from a team as a code owner May 3, 2026 22:05
@willkill07 willkill07 merged commit 5719bcd into NVIDIA:main May 3, 2026
58 of 59 checks passed
@willkill07 willkill07 deleted the refactor/github-hosted-linux-runners branch May 3, 2026 22:06
@willkill07 willkill07 self-assigned this May 3, 2026
@willkill07 willkill07 added this to the 0.2.0 milestone May 4, 2026
@willkill07 willkill07 added the Maintenance CI or Build or general repository maintenance label May 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Maintenance CI or Build or general repository maintenance size:S PR is small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant