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

Add NginxProxy CRD #1815

Merged
merged 7 commits into from
Apr 17, 2024
Merged

Add NginxProxy CRD #1815

merged 7 commits into from
Apr 17, 2024

Conversation

sjberman
Copy link
Collaborator

@sjberman sjberman commented Apr 8, 2024

Problem: Users want to be able to configure global Gateway settings, such as the Otel tracing exporter, for all Gateways in a Class.

Solution: Add the NginxProxy CRD, which provides a way to configure these settings.
Note: this PR contains the CRD only. A subsequent PR will add the implementation.

Testing: Verified that CRD is installed and can be created.

Closes #1800

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that all unit tests pass after adding my changes
  • I have updated necessary documentation
  • I have rebased my branch onto main
  • I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

Release notes

If this PR introduces a change that affects users and needs to be mentioned in the release notes,
please add a brief note that summarizes the change.

NONE

@sjberman sjberman requested a review from a team as a code owner April 8, 2024 15:09
@github-actions github-actions bot added the enhancement New feature or request label Apr 8, 2024
Problem: Users want to be able to configure global Gateway  settings, such as the Otel tracing exporter, for all Gateways in a Class.

Solution: Add the NginxProxy CRD, which provides a way to configure these settings.
Note: this PR contains the CRD only. A subsequent PR will add the implementation.
@sjberman sjberman force-pushed the enh/gateway-settings-crd branch from 0c6d8f4 to 9f285d0 Compare April 12, 2024 20:35
Copy link
Contributor

@pleshakov pleshakov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added a few comments, mostly about validation.
If the rules are not clear yet, I suggest converting them to FIXME, where the implementation story will resolve them.

@sjberman sjberman force-pushed the enh/gateway-settings-crd branch from 4240149 to e845bf0 Compare April 15, 2024 20:59
@sjberman sjberman enabled auto-merge (squash) April 17, 2024 14:08
@sjberman sjberman merged commit eeaa3f2 into nginx:main Apr 17, 2024
40 checks passed
@sjberman sjberman deleted the enh/gateway-settings-crd branch April 17, 2024 14:22
miledxz added a commit to miledxz/nginx-gateway-fabric that referenced this pull request Jan 14, 2025
Problem: Users want to be able to configure global Gateway  settings, such as the Otel tracing exporter, for all Gateways in a Class.

Solution: Add the NginxProxy CRD, which provides a way to configure these settings.
Note: this PR contains the CRD only. A subsequent PR will add the implementation.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add API for Gateway Settings
4 participants