openshift / openshift/oadp-operator
Remove kubevirt/kubevirt e2e workarounds once their upstream PRs merge
@kaovilai is already working on this.
Since Aug 31, 2026.
- Dominant language
- Go
- Stars
- 92
- Forks
- 93
- Avg merge
- 1d 23h
- Merged PRs (30d)
- 43
Description
openshift/oadp-operator#2404's kdm e2e suite carries three workarounds for still-open kubevirt/kubevirt bugs. Each should be removed once its blocking PR merges and a release picks it up.
(migtools/kubevirt-datamover-controller#207, migtools/kubevirt-datamover-controller#208, and migtools/kubevirt-datamover-controller#212 are expected to merge shortly and are intentionally not tracked here.)
Workarounds to remove
-
Stuck VMB finalizer blocks namespace deletion forever (kubevirt/kubevirt#18724)
- Workaround:
tests/e2e/lib/virt_helpers.go'sClearStuckVMBFinalizers(line 1674) /IsNamespaceDeletedClearingStuckVMBFinalizers(line 1715), called fromtests/e2e/backup_restore_suite_test.go'sdeleteNamespace(line 421) - Remove once: kubevirt/kubevirt#18725 merges. kubevirt/kubevirt#18289 alone is not sufficient -- it reduces how often the bug fires but leaves a timing/version-skew gap (confirmed by cross-check), so it complements kubevirt/kubevirt#18725 rather than replacing it.
- Workaround:
-
VirtualMachineBackup status freeze during target PVC attach (CNV-85377/CNV-89684)
- Workaround:
tests/e2e/lib/virt_helpers.go'sNudgeVmiToTriggerResync(line 1054) andVMBHasNoConditions(line 1646), called fromtests/e2e/virt_backup_restore_suite_test.go(lines 476, 482, 510) - Remove once: kubevirt/kubevirt#18949 merges.
- Workaround:
-
Stale reconcile can mark an already-completed VirtualMachineBackup Failed with SourceLost
- Workaround:
tests/e2e/lib/apps.go'sGetNamespaceEventMessages(line 386), called fromtests/e2e/virt_backup_restore_suite_test.go(lines 507, 530) - Remove once: kubevirt/kubevirt#18957 merges.
- Workaround:
Not tracked here (already a permanent improvement, not removable scaffolding)
openshift/oadp-operator#2404's hyperConvergedGVR() (preferring HCO's v1 API over v1beta1) stays useful even after kubevirt/hyperconverged-cluster-operator#4552 merges, so it's not a candidate for removal.
[!Note]
Responses generated with Claude
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.