Skip to content

Commit

Permalink
Merge pull request #2007 from robscott/remove-flawed-redirect-test
Browse files Browse the repository at this point in the history
Conformance: Removing status-only redirect test
  • Loading branch information
k8s-ci-robot authored May 9, 2023
2 parents 2d1f9c5 + 2dc7d97 commit 74fcabc
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 17 deletions.
9 changes: 0 additions & 9 deletions conformance/tests/httproute-redirect-host-and-status.go
Original file line number Diff line number Diff line change
Expand Up @@ -58,15 +58,6 @@ var HTTPRouteRedirectHostAndStatus = suite.ConformanceTest{
Host: "example.org",
},
Namespace: ns,
}, {
Request: http.Request{
Path: "/status-code-301",
UnfollowRedirect: true,
},
Response: http.Response{
StatusCode: 301,
},
Namespace: ns,
}, {
Request: http.Request{
Path: "/host-and-status",
Expand Down
8 changes: 0 additions & 8 deletions conformance/tests/httproute-redirect-host-and-status.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,14 +15,6 @@ spec:
- type: RequestRedirect
requestRedirect:
hostname: example.org
- matches:
- path:
type: PathPrefix
value: /status-code-301
filters:
- type: RequestRedirect
requestRedirect:
statusCode: 301
- matches:
- path:
type: PathPrefix
Expand Down

0 comments on commit 74fcabc

Please sign in to comment.