Skip to content

ci: move Linux and Windows jobs to GitHub-hosted runners#6175

Merged
Xuanwo merged 2 commits intolance-format:mainfrom
Xuanwo:xuanwo/use-github-hosted-runners
Mar 12, 2026
Merged

ci: move Linux and Windows jobs to GitHub-hosted runners#6175
Xuanwo merged 2 commits intolance-format:mainfrom
Xuanwo:xuanwo/use-github-hosted-runners

Conversation

@Xuanwo
Copy link
Copy Markdown
Collaborator

@Xuanwo Xuanwo commented Mar 12, 2026

This PR moves the Linux and Windows workflows that currently run on Warp onto GitHub-hosted runners. The goal is to reduce reliance on custom runners and take advantage of the sponsored larger GitHub-hosted machines for the slowest CI paths.

This is focused on the current CI bottlenecks we observed in recent successful PR runs, especially Rust ARM and Python Windows jobs, while keeping the existing macOS and benchmark-specific runners unchanged until we verify equivalent GitHub-hosted options for them.

Context:

  • Recent PR history shows Rust linux-arm and Python windows as the dominant critical-path jobs.
  • This change upgrades those jobs to larger GitHub-hosted runners where available (ubuntu-24.04-8x, ubuntu-24.04-arm64-8x, windows-latest-4x) and aligns the remaining Linux/Windows workflows with the same runner family.
  • I validated the workflow YAML locally after the runner migration; no product code or test logic changed.

Updates:

  • Rust linux-arm:40.7 -> 19.4,about -52%
  • Rust windows-build:27.7 -> 21.0,about -24%
  • Python windows:36.5 -> 23.1,about -37%
  • Python Linux 3.13 ARM:26.9 -> 20.7,about -23%
  • Python Linux 3.13 x86_64:26.8 -> 19.1,about -29%
  • Python Linux 3.9 x86_64:25.9 -> 19.2,about -26%

@github-actions github-actions Bot added the ci Github Action or Test issues label Mar 12, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Review

Clean CI migration from Warp to GitHub-hosted runners. One observation:

Runner size upgrades (intentional?): linux-build and linux-arm in rust.yml are upgraded from 4x to 8x cores, while other jobs maintain their existing sizes. The PR description explains these are the CI bottleneck jobs, so the upgrades make sense — just flagging for cost awareness since larger runners are more expensive per minute.

No issues with the rest of the changes — straightforward 1:1 runner label swaps. CI is still running at time of review; worth confirming all jobs pass on the new runners before merging.

LGTM pending green CI.

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@Xuanwo Xuanwo merged commit 98bb073 into lance-format:main Mar 12, 2026
40 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Github Action or Test issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants