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

Conversation

sjberman
Copy link
Contributor

@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
apis/v1alpha1/nginxproxy_types.go Outdated Show resolved Hide resolved
apis/v1alpha1/nginxproxy_types.go Show resolved Hide resolved
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.
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.

apis/v1alpha1/nginxproxy_types.go Outdated Show resolved Hide resolved
apis/v1alpha1/nginxproxy_types.go Show resolved Hide resolved
apis/v1alpha1/nginxproxy_types.go Outdated Show resolved Hide resolved
apis/v1alpha1/nginxproxy_types.go Show resolved Hide resolved
@sjberman sjberman enabled auto-merge (squash) April 17, 2024 14:08
@sjberman sjberman merged commit eeaa3f2 into nginxinc:main Apr 17, 2024
40 checks passed
@sjberman sjberman deleted the enh/gateway-settings-crd branch April 17, 2024 14:22
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
Archived in project
Development

Successfully merging this pull request may close these issues.

Add API for Gateway Settings
4 participants