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 committed May 2, 2023
1 parent f17f0fc commit 78ac353
Showing 1 changed file with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions connectivity/check/test.go
Original file line number Diff line number Diff line change
Expand Up @@ -558,13 +558,8 @@ 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 {
ipFams = []IPFamily{IPFamilyV4}
ipFams = []IPFamily{IPFamilyV4, IPFamilyV6}
}

for _, ipFam := range ipFams {
Expand Down

0 comments on commit 78ac353

Please sign in to comment.