Skip to content

Commit

Permalink
connectivity: Disable IPv6 with EP feat when netpols exist
Browse files Browse the repository at this point in the history
The per-endpoint routes feature is broken with IPv6 when there are any
netpols installed (tracked in
cilium/cilium#23852 and
cilium/cilium#23910). Once both issues are
resolved, we can start testing IPv6 with netpols.

Signed-off-by: Martynas Pumputis <m@lambda.lt>
  • Loading branch information
brb committed Feb 21, 2023
1 parent a46a2ae commit 505058c
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions connectivity/check/test.go
Original file line number Diff line number Diff line change
Expand Up @@ -347,6 +347,15 @@ 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}
}

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

0 comments on commit 505058c

Please sign in to comment.