Add conformance test for PartiallyInvalid Route #2458
Labels
area/conformance
lifecycle/stale
Denotes an issue or PR has remained open with no activity and has become stale.
needs-triage
Indicates an issue or PR lacks a `triage/foo` label and requires one.
What would you like to be added:
As suggested by @sunjayBhatia, we should figure out how to add conformance test coverage for #2429.
Before we do this we need to determine some kind of value that is considered invalid by implementations but passes CEL validation. Unclear what that will look like, and this will definitely require broader discussion.
Once that's in place (copied from #2429 (review)):
SupportsPartiallyInvalidRoutes
(name tbd) and test that makes sure the status is set to true when there is a partially invalid route, likely ignoring reason/messageSupportsPartiallyInvalidRoutes[DropRule|Fallback]
Implementations would opt-in to the test they support
Why this is needed:
The spec change in #2429 introduces a new requirement for all implementations. Although it won't be trivial, it would be good to have conformance test coverage for this.
The text was updated successfully, but these errors were encountered: