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

Make static mode compatible with provisioner #657

Merged

Conversation

pleshakov
Copy link
Contributor

This PR adds two features for the static mode:

  • Support configuring a single Gateway resource to watch.
  • Support not reporting the GatewayClass status, so that it doesn't conflict with the status updates done by the provisioner.

Needed by #634

@pleshakov pleshakov requested a review from a team as a code owner May 22, 2023 17:12
@github-actions github-actions bot added documentation Improvements or additions to documentation enhancement New feature or request labels May 22, 2023
cmd/gateway/validation_test.go Outdated Show resolved Hide resolved
internal/manager/filter/filter.go Show resolved Hide resolved
internal/status/updater_test.go Outdated Show resolved Hide resolved
@pleshakov pleshakov requested a review from sjberman May 22, 2023 20:41
cmd/gateway/commands.go Outdated Show resolved Hide resolved
cmd/gateway/commands.go Show resolved Hide resolved
internal/manager/filter/filter_test.go Outdated Show resolved Hide resolved
This commit adds two features for the static mode:
- Support configuring a single Gateway resource to watch.
- Support not reporting the GatewayClass status,
so that it doesn't conflict with the status updates done by
the provisioner.

Needed by nginxinc#634
@pleshakov pleshakov force-pushed the feature/static-mode-for-provisioner branch from 128ef94 to 8230e48 Compare May 23, 2023 20:49
@pleshakov pleshakov merged commit 717da40 into nginxinc:main May 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants