-
Notifications
You must be signed in to change notification settings - Fork 472
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Define Default Gateway/Route Association Behavior #84
Comments
This question is closely related to #12 (using label selectors to associate routes with gateways). Label selectors feel natural and appropriate but may be incompatible with extremely fine-grained, explicit access control (how do you restrict who can apply a label to a resource?). |
I think we need both really. Union type of explicit reference OR selector. The real phase transition in complexity is to allow for some sort of selection at all. |
If there are multiple gateways, how would you know which routes go with which gateway?
Hugely +1 on this. Association with a gateway could mean getting traffic from the internets or getting a service provider bill. |
The route's status indicates with which gateways it is associated.
That is an interesting point. Does the API need a way to specify quotas? Which persona should be limits? Could this fall under traffic splitting or route rules? This seems like a new use-case. |
But how do you know which gateway it should go with? Would all gateways just pick up all routes? If a route was already bound to gateway A, should gateway B pick it up implicitly?
No, I don't think so. That's what GatewayClass is for. GatewayClass can mean "the internet-facing one", "the expensive one with guaranteed capacity", or whatever is needed. |
I think general take is that Gateways should not pick up routes unless it was explicitly configured it to do so to avoid wide open defaults. Also, need to note that there are several levels of ACLs here:
|
/close This is now documented in the route selection behavior in the code. It may still change due to discussion, but is now explicitly specified: |
@bowei: Closing this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
…cs-yamls fixes yaml error
What would you like to be added:
Document gateway/route association. Should the default approach be open or closed?
Why is this needed:
Define the
route
field of thegateway
resource. Should routes a) be associated to a gateway by default and usegateway.spec.routes
as a filtering mechanism (default open) or b) not be associated to a gateway unless specified bygateway.spec.routes
(default closed)? Default open provides ease-of-use, but may cause inadvertent route association. Default closed is more secure but requires additional coordination between cluster operator and developers./cc @bowei @jpeach @ironcladlou @Miciah
xref: #71
The text was updated successfully, but these errors were encountered: