-
Notifications
You must be signed in to change notification settings - Fork 493
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
GEP-1762 enforces resources to be created in the Gateway namespace #3366
Comments
discussed this issue in the community meeting today, here's a summary
|
The test expects generated Gateway resources to have the `gateway.networking.k8s.io/gateway` label set. Mark the test as provisional until these statements have moved into the API docs. Relates to kubernetes-sigs#3366 (comment) Signed-off-by: Arko Dasgupta <[email protected]>
The test expects generated Gateway resources to have the `gateway.networking.k8s.io/gateway` label set. Mark the test as provisional until these statements have moved into the API docs. Relates to #3366 (comment) Signed-off-by: Arko Dasgupta <[email protected]>
I'm curious what in this model a user and an infrastructure namespace entails. In my mind with the design goal of gateway api and it's personas a gateway resource would never live in a 'user' namespace, this would always live in some kind of infrastructure namespace (depending on definitions of these types of namespaces). |
@rouke-broersma, the user here is the platform engineer/admin. The point this GH issue is trying to make is that both the deployment models are valid, some implementations support both these models, but will default to 1. or 2.
so the |
What happened:
The Automated Deployments section with GEP-1762 mentions the following
What you expected to happen:
My suggestion is to modify the
MUST
toSHOULD
. Some implementations such as Envoy Gateway dont have write access into user namespaces and can write into an infrastructure namespaceHow to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
The text was updated successfully, but these errors were encountered: