-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
WIP: Add QEMU functional tests #19299
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: spowelljr The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
ab6bdad
to
4c452c9
Compare
if detect.IsAmd64M1Emulation() { | ||
return "tcg" | ||
} | ||
if runtime.GOARCH == "arm64" && detect.GithubActionRunner() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Gemini tells me
Flexibility: TCG allows QEMU to emulate a wide range of CPU architectures, even those not directly supported by the host system's hardware.
- Introduction — QEMU documentation
www.qemu.org
Portability: Since TCG is software-based, QEMU can run on various platforms without requiring specialized hardware for each architecture.
so I am guessing this might mean we can use tcg on windows too
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only for the slow path, which is not something that is recommended for running Kubernetes with
It should start with "whpx", and only use tcg if running on the wrong Architecture (like on Windows ARM)
PR needs rebase. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
The Kubernetes project currently lacks enough contributors to adequately respond to all PRs. This bot triages PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
The Kubernetes project currently lacks enough active contributors to adequately respond to all PRs. This bot triages PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle rotten |
Added functional test for QEMU that run on
macos-14
runners (arm64).If using the QEMU driver on GitHub Action runners, doubled the number of attempts to get the VM IP from the
dhcpd_leases
file (from 60 to 120).Set the
-cpu
QEMU flag tomax
& the-accel
QEMU flag totcg
if running on an arm64 macOS GitHub Action runner.