Closed aerosouund closed 1 month ago
cc: @xpivarc @brianmcarey @dhiller
@aerosouund: 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 | 8623467419437b20523f71de3fdaa94f390bf68b | link | false | /test check-up-kind-1.27-vgpu |
/cc @akalenyu
/lgtm thank you!
[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
What this PR does / why we need it:
Fixes the problem mentioned with ceph storage class labeling in: https://github.com/kubevirt/kubevirt/pull/12677#issuecomment-2336737660
By executing the patch commands through the
sshClient
and adds the expected calls to the test suites of ceph and runWhich 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:
The fix uses kubectl and bash to avoid modifying the k8s client interface before deploying the fix, another PR that will introduce a
Patch
method in the k8s client will follow shortlyChecklist
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.