From 7783e6254cda42cbfaa44e9b54e2087c76221c0e Mon Sep 17 00:00:00 2001 From: Ciprian Hacman Date: Thu, 17 Aug 2023 07:54:26 +0300 Subject: [PATCH] Skip failing ProxyTerminatingEndpoints test --- tests/e2e/pkg/tester/skip_regex.go | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/tests/e2e/pkg/tester/skip_regex.go b/tests/e2e/pkg/tester/skip_regex.go index 754f8f0f3b..05542cf415 100644 --- a/tests/e2e/pkg/tester/skip_regex.go +++ b/tests/e2e/pkg/tester/skip_regex.go @@ -58,6 +58,11 @@ func (t *Tester) setSkipRegexFlag() error { // TODO: aojea // https://github.com/kubernetes/kubernetes/issues/113964 skipRegex += "|LoadBalancers.should.be.able.to.preserve.UDP.traffic" + } else { + // K8s 1.28 promoted ProxyTerminatingEndpoints to GA, but it has limited CNI support + // https://github.com/kubernetes/kubernetes/pull/117718 + // https://github.com/cilium/cilium/issues/27358 + skipRegex += "|fallback.to.local.terminating.endpoints.when.there.are.no.ready.endpoints.with.externalTrafficPolicy.Local" } networking := cluster.Spec.LegacyNetworking @@ -76,10 +81,6 @@ func (t *Tester) setSkipRegexFlag() error { // https://github.com/cilium/cilium/issues/9207 skipRegex += "|serve.endpoints.on.same.port.and.different.protocols" - if cluster.Spec.LegacyCloudProvider == "gce" { - skipRegex += "|fallback.to.local.terminating.endpoints.when.there.are.no.ready.endpoints.with.externalTrafficPolicy.Local" - } - if isPre28 { // These may be fixed in Cilium 1.13 but skipping for now skipRegex += "|Service.with.multiple.ports.specified.in.multiple.EndpointSlices"