Skip to content

Commit

Permalink
connectivity: enable IPv6 test for per-endpoint routing
Browse files Browse the repository at this point in the history
As cilium/cilium#2385 and
cilium/cilium#23910 has been resolved, we
enable IPv6 test for per-endpoint routing.

Signed-off-by: Zhichuan Liang <gray.liang@isovalent.com>
  • Loading branch information
jschwinger233 authored and tklauser committed Jul 7, 2023
1 parent a1c8a1b commit 6c20d0d
Showing 1 changed file with 0 additions and 10 deletions.
10 changes: 0 additions & 10 deletions connectivity/check/test.go
Original file line number Diff line number Diff line change
Expand Up @@ -725,16 +725,6 @@ func (t *Test) collectSysdump() {

func (t *Test) ForEachIPFamily(do func(IPFamily)) {
ipFams := []IPFamily{IPFamilyV4, IPFamilyV6}

// TODO(brb):
// The per-endpoint routes feature is broken with IPv6 when there are any
// netpols installed (tracked in https://github.com/cilium/cilium/issues/23852
// and https://github.com/cilium/cilium/issues/23910). Once both issues
// are resolved, we can start testing IPv6 with netpols.
if f, ok := t.Context().Feature(FeatureEndpointRoutes); ok && f.Enabled && (len(t.cnps) > 0 || len(t.knps) > 0) {
ipFams = []IPFamily{IPFamilyV4}
}

for _, ipFam := range ipFams {
switch ipFam {
case IPFamilyV4:
Expand Down

0 comments on commit 6c20d0d

Please sign in to comment.