Closed brianmcarey closed 4 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
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: Once this PR has been reviewed and has the lgtm label, please ask for approval from brianmcarey. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.
/test check-provision-k8s-1.30
@brianmcarey: 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-provision-k8s-1.30 | 83b357d3fdd1af27682d0bdbb985bc239ab7c968 | link | true | /test check-provision-k8s-1.30 |
PR needs rebase.
/close
@brianmcarey: Closed this PR.
What this PR does / why we need it:
The manifests are being included in the centos base image - these are not required and should be removed.
This also causes issues with bumps of CDI as there are multiple versions of the manifests on the node[1]
[1] https://prow.ci.kubevirt.io/view/gs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirtci/1158/check-provision-k8s-1.29/1774972729217257472#1:build-log.txt%3A2363
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: /cc none
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: