Closed lyarwood closed 4 months ago
@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 | d2ca1f2fb2a0f9958da803e4ec7137819cddf3bf | link | false | /test check-up-kind-1.27-vgpu |
/cc @vladikr /cc @brianmcarey
Hey folks, would you mind taking a look at this review and the wider series in https://github.com/kubevirt/kubevirtci/pull/1171, any and all comments welcome!
/test ?
@brianmcarey: The following commands are available to trigger required jobs:
/test check-provision-alpine-with-test-tooling
/test check-provision-centos-base
/test check-provision-k8s-1.27
/test check-provision-k8s-1.28
/test check-provision-k8s-1.29
/test check-provision-k8s-1.30
/test check-provision-manager
/test check-up-kind-sriov
The following commands are available to trigger optional jobs:
/test check-gocli
/test check-up-kind-1.27-vgpu
Use /test all
to run the following jobs that were automatically triggered:
check-provision-alpine-with-test-tooling
check-provision-k8s-1.27
check-provision-k8s-1.28
check-provision-k8s-1.29
check-provision-k8s-1.30
check-provision-manager
check-up-kind-1.27-vgpu
check-up-kind-sriov
/test check-provision-centos-base
[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
/retest-required
It looks good for me as well. Thanks @lyarwood ! /lgtm
What this PR does / why we need it:
This change adds basic support to the vm.sh script for defining a number of NUMA nodes per host with vCPUs and memory spread equally between them.
This change is broken out from the rest of the series wiring up gocli and cluster-up as it needs to be present in the current centos9 image to be testable in such a PR.
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:
See https://github.com/kubevirt/kubevirtci/pull/1171 for how this will be used.
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: