Skip to content

Conversation

@rainest
Copy link

@rainest rainest commented Apr 15, 2022

What type of PR is this?

/kind feature
/kind api-change

What this PR does / why we need it:
Adds a RouteConditionReason for when a Gateway has no Listeners that can accept the route because none had allowedRoutes criteria that would accept it.

Does this PR introduce a user-facing change?:

adds RouteReasonNotAllowedByListeners RouteConditionReason constant

Add a RouteConditionReason constant, "NotAllowedByListeners", for use
when a route fails to match any of the allowedRoutes conditions on a
Gateway's Listeners.
@rainest
Copy link
Author

rainest commented Apr 18, 2022

Per kubernetes-sigs#1114 (comment) closing to reopen as a PR against kubernetes-sigs/gateway-api after that's merged.

@rainest rainest closed this Apr 18, 2022
mikemorris pushed a commit that referenced this pull request Apr 5, 2024
…Gateway to Backend...) Update with API details (kubernetes-sigs#2113)

* GEP-1897 Update with API details

* GEP-1897 Update with API details - suggested changes #1

* Update geps/gep-1897.md

Co-authored-by: Rob Scott <[email protected]>

* Update geps/gep-1897.md

Co-authored-by: Rob Scott <[email protected]>

* GEP-1897 Update with API details - suggested changes kubernetes-sigs#2

* GEP-1897 Update with API details - suggested changes kubernetes-sigs#3

* GEP-1897 Update with API details - suggested changes 4

* GEP-1897 Update with API details - suggested changes 5

* GEP-1897 Update with API details - suggested changes 7

* GEP-1897 Update with API details - suggested changes 8

* GEP-1897 Update with API details - suggested changes 9

* GEP-1897 Update with API details - suggested changes 10

* GEP-1897 Update with API details - suggested changes 11

---------

Co-authored-by: Rob Scott <[email protected]>
mikemorris pushed a commit that referenced this pull request Feb 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant