Skip to content

Commit

Permalink
Merge pull request #16773 from hakman/fix-kr
Browse files Browse the repository at this point in the history
Fix tests skips for kube-router
  • Loading branch information
k8s-ci-robot authored Aug 25, 2024
2 parents c11270f + 17ebe3f commit 5721bef
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions tests/e2e/pkg/tester/skip_regex.go
Original file line number Diff line number Diff line change
Expand Up @@ -83,9 +83,9 @@ func (t *Tester) setSkipRegexFlag() error {
// https://github.com/kubernetes/kubernetes/blob/418ae605ec1b788d43bff7ac44af66d8b669b833/test/e2e/network/networking.go#L135
skipRegex += "|should.check.kube-proxy.urls"

if k8sVersion.Minor < 32 {
if k8sVersion.Minor < 33 {
// This seems to be specific to the kube-proxy replacement
// < 32 so we look at this again
// < 33 so we look at this again
skipRegex += "|Services.should.support.externalTrafficPolicy.Local.for.type.NodePort"
}

Expand Down Expand Up @@ -171,7 +171,8 @@ func (t *Tester) setSkipRegexFlag() error {
// ref: https://github.com/kubernetes/kops/issues/16349
// ref: https://github.com/kubernetes/kubernetes/issues/123255
// ref: https://github.com/kubernetes/kubernetes/issues/121018
if k8sVersion.Minor < 32 {
// < 33 so we look at this again
if k8sVersion.Minor < 33 {
skipRegex += "|Services.should.function.for.service.endpoints.using.hostNetwork"
}

Expand Down

0 comments on commit 5721bef

Please sign in to comment.