Skip to content

Commit

Permalink
Merge pull request #17172 from hakman/skip-cilium-test
Browse files Browse the repository at this point in the history
Skip test `Services should implement NodePort and HealthCheckNodePort…
  • Loading branch information
k8s-ci-robot authored Jan 6, 2025
2 parents 4d47cba + c4a3f0b commit 84948bb
Showing 1 changed file with 3 additions and 6 deletions.
9 changes: 3 additions & 6 deletions tests/e2e/pkg/tester/skip_regex.go
Original file line number Diff line number Diff line change
Expand Up @@ -83,12 +83,9 @@ func (t *Tester) setSkipRegexFlag() error {
// This seems to be specific to the kube-proxy replacement
// < 33 so we look at this again
skipRegex += "|Services.should.support.externalTrafficPolicy.Local.for.type.NodePort"

if cluster.Spec.LegacyCloudProvider == "gce" {
// https://github.com/kubernetes/kubernetes/issues/129221
// < 33 so we look at this again
skipRegex += "|Services.should.implement.NodePort.and.HealthCheckNodePort.correctly.when.ExternalTrafficPolicy.changes"
}
// https://github.com/kubernetes/kubernetes/issues/129221
// < 33 so we look at this again
skipRegex += "|Services.should.implement.NodePort.and.HealthCheckNodePort.correctly.when.ExternalTrafficPolicy.changes"
}

if isPre28 {
Expand Down

0 comments on commit 84948bb

Please sign in to comment.