Conformance Tests: HTTPRoute Filters

[robscott]

This issue is intended to track conformance test development for HTTPRoute filters. Comment below if you're interested in working on covering any of these areas.

Core Capabilities:

• Header modifier (@skriss)
  • Set
  • Add
  • Remove
• Request Redirect: (@LiorLieberman)
  • Scheme
  • Port Number

Experimental Capabilities:

• Request Redirect:
  • Path
• Request Rewrite: (@LiorLieberman )
  • Host
  • Path

Extended Capabilities:

• Request mirroring
• Combinations of the above with core capabilities (will be covered by Conformance: HTTPRoute Filter Combinations #2037)

[skriss]

I've submitted #1163 for the header modifier set/add/remove cases.

[k8s-triage-robot]

The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs.

This bot triages issues and PRs according to the following rules:

• After 90d of inactivity, lifecycle/stale is applied
• After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
• After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

• Mark this issue or PR as fresh with /remove-lifecycle stale
• Mark this issue or PR as rotten with /lifecycle rotten
• Close this issue or PR with /close
• Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

[mikemorris]

/remove-lifecycle stale

[sunjayBhatia]

I think this issue should be augmented (or another created) to add testing for HTTPRoute Filters per-BackendRef as well, since the existing tests only test per-RouteRule

Part of that will include adding a test helper that can conditionally check a response has the correct content, given it was routed to a particular backend

[robscott]

@LiorLieberman is working on conformance tests for HTTP Redirects.

[LiorLieberman]

@robscott is request conformance for request mirroring still wanted?

also proposing changes to the description

...

Extended Capabilities:
Request Redirect: (@LiorLieberman)
[x] Scheme
[x] Port Number

Experimental Capabilities:
Request Redirect:
[x] Path
Request Rewrite: (@LiorLieberman )
[x] host
[x] Path

...

[robscott]

Thanks @LiorLieberman! Updated the issue to match. We are still looking for tests covering request mirroring.

[shaneutt]

At this point we only have one item left, talked with @robscott he's gonna close this and spin off the last one.

/assign @robscott

[LiorLieberman]

What is the one item left? @shaneutt

[robscott]

@LiorLieberman added #2037 to track the last part of this (combining HTTPRoute filters together in tests).

Going to close this one out in favor of that more specific one.