Skip to content

ci: use self-hosted KubeVirt runners for amd64 jobs#787

Draft
mangelajo wants to merge 1 commit into
mainfrom
kubevirt-runners
Draft

ci: use self-hosted KubeVirt runners for amd64 jobs#787
mangelajo wants to merge 1 commit into
mainfrom
kubevirt-runners

Conversation

@mangelajo

Copy link
Copy Markdown
Member

Switch E2E and pytest amd64/Linux jobs from ubuntu-24.04 to arc-runner-kubevirt, running on self-hosted KubeVirt VMs on the beast cluster. ARM64 and macOS jobs are unchanged.

Switch E2E and pytest amd64/Linux jobs from ubuntu-24.04 to
arc-runner-kubevirt, running on self-hosted KubeVirt VMs on the
beast cluster. ARM64 and macOS jobs are unchanged.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 3e2b8761-72a5-4054-913c-0d70c146c6e1

📥 Commits

Reviewing files that changed from the base of the PR and between a147739 and eb6328c.

📒 Files selected for processing (2)
  • .github/workflows/e2e.yaml
  • .github/workflows/python-tests.yaml

📝 Walkthrough

Walkthrough

This PR updates GitHub Actions workflow runner configurations across the E2E and Python test pipelines, migrating from ubuntu-24.04 to arc-runner-kubevirt for job execution while maintaining the existing matrix structures for architecture coverage.

Changes

Runner Infrastructure Migration

Layer / File(s) Summary
E2E workflow jobs runner migration
.github/workflows/e2e.yaml
Build controller and operator image jobs, build-python-wheels, e2e-tests, and compatibility jobs switch from ubuntu-24.04 to arc-runner-kubevirt in their job matrices and runner configurations.
Python tests workflow runner migration
.github/workflows/python-tests.yaml
Pytest-matrix job updates its runs-on strategy from ubuntu-24.04 to arc-runner-kubevirt while retaining macos-15.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 In the CI clouds we hop and bound,
New runners found, a better ground,
Arc-runner-kubevirt, swift and true,
Let tests and builds run spry and new!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and concisely summarizes the main change: switching CI runners from ubuntu-24.04 to self-hosted KubeVirt runners for amd64 jobs.
Description check ✅ Passed The description clearly explains the changeset: switching E2E and pytest amd64/Linux jobs to self-hosted KubeVirt runners while keeping ARM64 and macOS jobs unchanged.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch kubevirt-runners

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@kirkbrauer kirkbrauer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This looks good to me, hopefully we get quite a speedup here!

@mangelajo mangelajo marked this pull request as draft June 15, 2026 07:31
@mangelajo

Copy link
Copy Markdown
Member Author

I am experimenting @kirkbrauer , the operator I am trying for the github actions in K8s + kubevirt seems to be a bit slow to rotate VMs, and grab jobs. Unless we get to improve that we would have to look for an alternative option

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