From 6881f105c365bba97cfe9b5ca580fa74e7e91551 Mon Sep 17 00:00:00 2001 From: Edward Haas Date: Sun, 25 Jun 2023 14:36:49 +0300 Subject: [PATCH] tests, hotplug: Remove redundant migration When testing multiple hotplug interfaces a redundant migration was executed after the second hotplug. By executing the migration before the new interface propagated to the VMI, flakes are seen (pod interface missing on the target). Therefore, drop one migration and depend on the one performed as part of the `verifyDynamicInterfaceChange` helper. Signed-off-by: Edward Haas --- tests/network/hotplug.go | 4 ---- 1 file changed, 4 deletions(-) diff --git a/tests/network/hotplug.go b/tests/network/hotplug.go index a058f73f8716..1689a8ab4b60 100644 --- a/tests/network/hotplug.go +++ b/tests/network/hotplug.go @@ -206,10 +206,6 @@ var _ = SIGDescribe("nic-hotplug", func() { ), ).To(Succeed()) - if plugMethod == migrationBased { - migrate(hotPluggedVMI) - } - By("wait for the second network to appear in the VMI spec") EventuallyWithOffset(1, func() []v1.Network { var err error