Skip to content

Commit

Permalink
Merge pull request #16770 from justinsb/skip_cilium_kubeproxy_metrics
Browse files Browse the repository at this point in the history
tests: skip kube-proxy metric tests with cilium
  • Loading branch information
k8s-ci-robot authored Aug 25, 2024
2 parents b8e9ba5 + 9645d82 commit c11270f
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions tests/e2e/pkg/tester/skip_regex.go
Original file line number Diff line number Diff line change
Expand Up @@ -103,6 +103,14 @@ func (t *Tester) setSkipRegexFlag() error {
// Ref: https://github.com/kubernetes/kubernetes/pull/113335
skipRegex += "|should.create.a.Pod.with.SCTP.HostPort"
}

if k8sVersion.Minor < 34 {
// < 34 so we revisit this in future
// This test checks for kube-proxy on port 10249 (`127.0.0.1:10249/proxyMode`)
// It appears that the cilium kube-proxy replacement does not implement this.
// Ref: https://github.com/kubernetes/kubernetes/issues/126903
skipRegex += "|KubeProxy.should.update.metric.for.tracking.accepted.packets.destined.for.localhost.nodeports"
}
} else if networking.KubeRouter != nil {
skipRegex += "|should set TCP CLOSE_WAIT timeout|should check kube-proxy urls"
} else if networking.Kubenet != nil {
Expand Down

0 comments on commit c11270f

Please sign in to comment.