-
Notifications
You must be signed in to change notification settings - Fork 192
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
connectivity: Skip IPv6 requests in north-south-loadbalancing-with-l7-policy when running on < 1.14.0 Cilium #1629
Conversation
2000b3d
to
39fab92
Compare
Ran |
connectivity/tests/service.go
Outdated
// Skip IPv6 requests when running on <1.14.0 Cilium with CNPs | ||
if check.GetIPFamily(addr.Address) == check.IPFamilyV6 && | ||
versioncheck.MustCompile("<1.14.0")(t.Context().CiliumVersion) && | ||
len(t.CiliumNetworkPolicies()) > 0 { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please don't do my mistakes - please check t.knps
too (#1587) 😅
39fab92
to
af19916
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
connectivity/check/context.go
Outdated
@@ -705,6 +705,7 @@ func (ct *ConnectivityTest) CurlCommand(peer TestPeer, ipFam IPFamily, opts ...s | |||
peer.Scheme(), | |||
net.JoinHostPort(peer.Address(ipFam), fmt.Sprint(peer.Port())), | |||
peer.Path())) | |||
println(strings.Join(cmd, " ")) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: do we need this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry - - I should have checked my PR
…-policy when running on < 1.14.0 Cilium cilium/cilium#21954 for the IPv6 path was resolved only for v1.14, but not for v1.13. In order to be able to run the latest connectivity tests on v1.13, we need to skip curl requests to the IPv6 addresses in that particular test. Fixes: cilium#1627 Signed-off-by: Zhichuan Liang <gray.liang@isovalent.com>
af19916
to
329f3d0
Compare
#1629 initially limited the IPv6 test to v1.14, as v1.13 was missing some functionality. But now that v1.13 has cilium/cilium#31161, the IPv6 path should actually work. So let's extend the test coverage. Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
cilium/cilium#21954 for the IPv6 path was resolved only for v1.14, but not for v1.13. In order to be able to run the latest connectivity tests on v1.13, we need to skip curl requests to the IPv6 addresses in that particular test.
Fixes: #1627
Signed-off-by: Zhichuan Liang <gray.isovalent.com>