Closed lyarwood closed 3 months ago
Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all
/hold cancel
/retest-required
+ ./cluster-up/ssh.sh node01 -- ip l show eth1
selecting podman as container runtime
Device "eth1" does not exist.
I can't reproduce this locally so trying another run to ensure it isn't transient in CI.
/retest-required
+ ./cluster-up/ssh.sh node01 -- ip l show eth1 selecting podman as container runtime Device "eth1" does not exist.
I can't reproduce this locally so trying another run to ensure it isn't transient in CI.
+ ../gocli/build/cli provision 1.30 --phases k8s
time="2024-04-16T17:06:15Z" level=info msg="Using remote image quay.io/kubevirtci/centos9:2404020400-62457f2"
Ah I see it now, the issue is the tests don't also rebuild centos so the vm.sh
script within is old?
I assume I need to break these changes out of https://github.com/kubevirt/kubevirtci/pull/1171/commits/2618f6633d8a7698a116f4b893b4c5cc32653c0e into their own PR, land that and have a fresh centos image created?
/hold
/hold cancel
/cc
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: brianmcarey
The full list of commands accepted by this bot can be found here.
The pull request process is described here
@lyarwood: The following test failed, say /retest
to rerun all failed tests or /retest-required
to rerun all mandatory failed tests:
Test name | Commit | Details | Required | Rerun command |
---|---|---|---|---|
check-up-kind-1.27-vgpu | 4151e66316dfab1d8bf22f9c999c6620ddd09f7d | link | false | /test check-up-kind-1.27-vgpu |
What this PR does / why we need it:
These new env variables being useful when testing
dedicatedCpuPlacement
andguestMappingPassthrough
without requiring a physical host with multiple NUMA nodes etc.Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged): Fixes #Special notes for your reviewer:
Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR. Approvers are expected to review this list.
Release note: