Skip to content

Commit c4e0568

Browse files
Ahmad-HafeRedHatQE
authored andcommitted
[storage][T2][Main] Wait for PVC creation before VM deploy in public registry DV setup (RedHatQE#3626)
##### Short description: Wait for PVC creation before VM deploy in public registry DV setup Ensure public-registry DataVolume PVCs exist before creating VMs. ##### More details: Add a short wait after DataVolume creation so the corresponding PVC is created before VM deployment reads data_volume.pvc.instance. ##### What this PR does / why we need it: Fixes a setup race where the VM deployment fails with 404 because the PVC isn’t created yet. ##### Which issue(s) this PR fixes: N/A (test setup race for test_public_registry_multiple_data_volume) ##### Special notes for reviewer: ##### jira-ticket: <!-- full-ticket-url needs to be provided. This would add a link to the pull request to the jira and close it when the pull request is merged If the task is not tracked by a Jira ticket, just write "NONE". --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Tests** * Improved storage import test reliability by waiting for each persistent storage claim to be ready before proceeding (replacing a fixed timeout), reducing race conditions and flaky failures during VM deployment. * Increases determinism of test runs and makes CI outcomes more stable and predictable. <sub>✏️ Tip: You can customize this high-level summary in your review settings.</sub> <!-- end of auto-generated comment: release notes by coderabbit.ai -->
1 parent cbb3406 commit c4e0568

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

tests/storage/cdi_import/conftest.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -197,6 +197,7 @@ def dvs_and_vms_from_public_registry(namespace, storage_class_name_scope_functio
197197
)
198198
dv.create()
199199
dvs.append(dv)
200+
dv.pvc.wait(timeout=TIMEOUT_1MIN)
200201

201202
for dv in dvs:
202203
vm = VirtualMachineForTests(

0 commit comments

Comments
 (0)