Skip to content

Commit 2836632

Browse files
Merge pull request #29858 from qinqon/ovn-virt-secondaries-skip-ipv6
NO-JIRA: ovn, virt, skip ipv6 for secondaries
2 parents 19aa300 + d5c1d3c commit 2836632

File tree

1 file changed

+11
-2
lines changed

1 file changed

+11
-2
lines changed

test/extended/networking/livemigration.go

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -85,8 +85,16 @@ var _ = Describe("[sig-network][OCPFeatureGate:PersistentIPsForVirtualization][F
8585

8686
f.Namespace = ns
8787
netConfig.namespace = f.Namespace.Name
88-
// correctCIDRFamily makes use of the ginkgo framework so it needs to be in the testcase
89-
netConfig.cidr = correctCIDRFamily(oc, cidrIPv4, cidrIPv6)
88+
89+
// IPv6 only supported for primaries yet, the
90+
// kubevirt bridge binding dhcp server do not
91+
// support it.
92+
if netConfig.role == "primary" {
93+
// correctCIDRFamily makes use of the ginkgo framework so it needs to be in the testcase
94+
netConfig.cidr = correctCIDRFamily(oc, cidrIPv4, cidrIPv6)
95+
} else {
96+
netConfig.cidr = cidrIPv4
97+
}
9098
workerNodes, err := getWorkerNodesOrdered(cs)
9199
Expect(err).NotTo(HaveOccurred())
92100
Expect(len(workerNodes)).To(BeNumerically(">=", 2))
@@ -113,6 +121,7 @@ var _ = Describe("[sig-network][OCPFeatureGate:PersistentIPsForVirtualization][F
113121
VMNamespace: f.Namespace.Name,
114122
FedoraContainterDiskImage: image.LocationFor("quay.io/kubevirt/fedora-with-test-tooling-container-disk:20241024_891122a6fc"),
115123
}
124+
116125
if netConfig.role == "primary" {
117126
vmCreationParams.NetBindingName = bindingName
118127
} else {

0 commit comments

Comments
 (0)