From 95341f851487142792a1d98f26856cf104324dc2 Mon Sep 17 00:00:00 2001 From: Or Mergi Date: Sun, 4 Apr 2021 20:32:23 +0300 Subject: [PATCH 1/2] test, sriov ipv6 ping: Rename CIDR and IP's variables Signed-off-by: Or Mergi --- tests/network/vmi_multus.go | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/tests/network/vmi_multus.go b/tests/network/vmi_multus.go index c53af1067200..094175706f56 100644 --- a/tests/network/vmi_multus.go +++ b/tests/network/vmi_multus.go @@ -902,21 +902,21 @@ var _ = Describe("[Serial]SRIOV", func() { }) It("[test_id:3957]should connect to another machine with sriov interface over IPv6", func() { - cidrA := "fc00::1/64" - cidrB := "fc00::2/64" - ipA, err := cidrToIP(cidrA) + vmi1CIDR := "fc00::1/64" + vmi2CIDR := "fc00::2/64" + vmi1IP, err := cidrToIP(vmi1CIDR) Expect(err).ToNot(HaveOccurred()) - ipB, err := cidrToIP(cidrB) + vmi2IP, err := cidrToIP(vmi2CIDR) Expect(err).ToNot(HaveOccurred()) //create two vms on the same sriov network - vmi1, vmi2 := createSriovVMs(sriovnet3, sriovnet3, cidrA, cidrB) + vmi1, vmi2 := createSriovVMs(sriovnet3, sriovnet3, vmi1CIDR, vmi2CIDR) Eventually(func() error { - return libnet.PingFromVMConsole(vmi1, ipA) + return libnet.PingFromVMConsole(vmi1, vmi1IP) }, 15*time.Second, time.Second).Should(Succeed()) Eventually(func() error { - return libnet.PingFromVMConsole(vmi2, ipB) + return libnet.PingFromVMConsole(vmi2, vmi2IP) }, 15*time.Second, time.Second).Should(Succeed()) }) From 05b7f70ab1fc58e8a46e8d65d0d0e6ae0cb1ce70 Mon Sep 17 00:00:00 2001 From: Or Mergi Date: Sun, 4 Apr 2021 20:35:28 +0300 Subject: [PATCH 2/2] tests, sriov ipv6 ping: Fix test to ping the right address Currently each VM pinged itself. Signed-off-by: Or Mergi --- tests/network/vmi_multus.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/network/vmi_multus.go b/tests/network/vmi_multus.go index 094175706f56..011b70d48159 100644 --- a/tests/network/vmi_multus.go +++ b/tests/network/vmi_multus.go @@ -913,10 +913,10 @@ var _ = Describe("[Serial]SRIOV", func() { vmi1, vmi2 := createSriovVMs(sriovnet3, sriovnet3, vmi1CIDR, vmi2CIDR) Eventually(func() error { - return libnet.PingFromVMConsole(vmi1, vmi1IP) + return libnet.PingFromVMConsole(vmi1, vmi2IP) }, 15*time.Second, time.Second).Should(Succeed()) Eventually(func() error { - return libnet.PingFromVMConsole(vmi2, vmi2IP) + return libnet.PingFromVMConsole(vmi2, vmi1IP) }, 15*time.Second, time.Second).Should(Succeed()) })