Skip to content
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

If another Gateway API implementation removes NGF from statuses of resources, NGF will not immediately restore it #1813

Open
pleshakov opened this issue Apr 5, 2024 · 2 comments
Labels
backlog Currently unprioritized work. May change with user feedback or as the product progresses.

Comments

@pleshakov
Copy link
Contributor

If another controllers changes the status of the Gateway/HTTPRoute resource so that the information set by our
Gateway is removed, our Gateway will not restore the status until the EventLoop invokes the StatusUpdater as a
result of processing some other new change to a resource(s).

Corresponds to the FIXME in the StatusUpdater

@mpstefan mpstefan added the backlog Currently unprioritized work. May change with user feedback or as the product progresses. label Apr 18, 2024
@mpstefan
Copy link
Collaborator

Is this within our realm of responsibility? If the user wants to install two Gateway API control planes in the same scope, I would expect this kind of behavior.

@pleshakov
Copy link
Contributor Author

Statuses of Gateway API resources are designed so that multiple controllers can update them independently and do not overwrite each other updates. For example, for HTTPRoute, each Controller will create its own entry and set the ControllerName to its name https://github.com/kubernetes-sigs/gateway-api/blob/6b93e6effdcbf48a4e8424e20d1152094eb4e3eb/apis/v1/shared_types.go#L454

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Currently unprioritized work. May change with user feedback or as the product progresses.
Projects
Status: 🆕 New
Development

No branches or pull requests

2 participants