Reduce the list of skipped e2e test for Cilium

[aojea]

kops/tests/e2e/pkg/tester/skip_regex.go

Lines 60 to 85 in f2fe044

	if networking.Cilium != nil {
	// https://github.com/cilium/cilium/issues/10002
	skipRegex += "|TCP.CLOSE_WAIT"
	// https://github.com/cilium/cilium/issues/15361
	skipRegex += "|external.IP.is.not.assigned.to.a.node"
	// https://github.com/cilium/cilium/issues/14287
	skipRegex += "|same.port.number.but.different.protocols|same.hostPort.but.different.hostIP.and.protocol"
	// https://github.com/cilium/cilium/issues/9207
	skipRegex += "|serve.endpoints.on.same.port.and.different.protocols"
	// This may be fixed in Cilium 1.13 but skipping for now
	skipRegex += "|Service.with.multiple.ports.specified.in.multiple.EndpointSlices"
	if k8sVersion.Minor >= 22 {
	// ref:
	// https://github.com/kubernetes/kubernetes/issues/96717
	// https://github.com/cilium/cilium/issues/5719
	skipRegex += "|should.create.a.Pod.with.SCTP.HostPort"
	}
	// https://github.com/cilium/cilium/issues/18241
	skipRegex += "|Services.should.create.endpoints.for.unready.pods"
	skipRegex += "|Services.should.be.able.to.connect.to.terminating.and.unready.endpoints.if.PublishNotReadyAddresses.is.true"
	} else if networking.KubeRouter != nil {
	skipRegex += "|load-balancer|hairpin|affinity\\stimeout|service\\.kubernetes\\.io|CLOSE_WAIT"
	skipRegex += "|EndpointSlice.should.support.a.Service.with.multiple"
	} else if networking.Kubenet != nil {
	skipRegex += "|Services.*affinity"
	}

Some of them got fixed in
cilium/cilium#24202
cilium/cilium#24174

you can compare against the regex in cilium repo
https://github.com/cilium/cilium/pull/25258/files#diff-01b72d17856efb38bd2914aec50791eef6b9bc8238b6975d12f87b39cb24b008R214

Feel free to tag me to review the PR

[aojea]

/assign @justinsb @hakman

[hakman]

Thanks @aojea. I checked in #15449 and, for now, there's not much we can do with Cilium 1.12.x.

[aojea]

yeah, these changes are more recent

[johngmyers]

Office hours: after we upgrade Cilium

[johngmyers]

Now we've upgraded to Cilium 1.13, I've submitted kubernetes/test-infra#30300 to see which skips we can remove.

We are still seeing failures for "Services should fallback to local terminating endpoints when there are no ready endpoints with externalTrafficPolicy=Local": https://testgrid.k8s.io/kops-network-plugins#kops-aws-cni-cilium-k8s-ci

[aojea]

is this aws specific or do we have any other jobs we can compare with?

cilium seems to have a lot of knobs per platform ...

[johngmyers]

I'm adding a GCP variant of the test in kubernetes/test-infra#30324

[johngmyers]

Filed cilium/cilium#27358