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

helm: provisioning "gateway" support for nginx-gateway-fabric helm chart #1533

Open
arukiidou opened this issue Feb 2, 2024 · 5 comments · May be fixed by #1623
Open

helm: provisioning "gateway" support for nginx-gateway-fabric helm chart #1533

arukiidou opened this issue Feb 2, 2024 · 5 comments · May be fixed by #1623
Assignees
Labels
enhancement New feature or request helm-chart Relates to helm chart
Milestone

Comments

@arukiidou
Copy link
Contributor

arukiidou commented Feb 2, 2024

Is your enhancement request related to a problem? Please describe.

Currently, NGF does not seems to be provisioning Gateway

What would you like to be added:
provisioning "Gateway" resource.

Why this is needed:

After deploying the NGF, the cluster administrator must also deploy the gateway.
NGF and Gateway cannot be used at the same time, making Gitops more difficult
This is not sufficient given the gateway-api persona.

Additional context
some charts support Gateway

example: traefik chart supports this.

@arukiidou arukiidou added the enhancement New feature or request label Feb 2, 2024
@mpstefan
Copy link
Collaborator

mpstefan commented Feb 5, 2024

Talked about this issue in our community meeting. We think this makes sense, especially from the operator persona.

Looking at the Traefik example, we fairly sure we want something less opinionated in terms of how many ports, certificates, etc and should be able to provide a more flexible experience. But there has to be some limit...

What level of customization are you looking for in the Gateway resource? When looking at the API specification, what level of customization would you expect for each type?

Also, how are you planning on managing TLS certificates with respect to the Gateway?

@mpstefan mpstefan added the helm-chart Relates to helm chart label Feb 5, 2024
@arukiidou
Copy link
Contributor Author

arukiidou commented Feb 6, 2024

@mpstefan

For me, I generally hope to be able to reproduce the following.
In addition, annotations (required if using cert-manager)

Also, how are you planning on managing TLS certificates with respect to the Gateway?

For now, manual certificate management yet 😑.

@mpstefan
Copy link
Collaborator

mpstefan commented Feb 8, 2024

Thanks @arukiidou, we'll be discussing this item at our next community meeting on Monday.

@pleshakov
Copy link
Contributor

We discussed in our community meeting today.

We think it will be useful.

We're thinking about allowing cluster operators to fully customize annotations of the Gateway as well as Listeners. Do you think it will address your use case?

@pleshakov pleshakov added this to the v2.0.0 milestone Feb 12, 2024
@arukiidou
Copy link
Contributor Author

Thank you! Perhaps it will fit my use case

@ciarams87 ciarams87 self-assigned this Feb 27, 2024
@ciarams87 ciarams87 linked a pull request Feb 28, 2024 that will close this issue
6 tasks
@mpstefan mpstefan modified the milestones: v1.3.0, v1.4.0 May 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request helm-chart Relates to helm chart
Projects
Status: 👀 In Review
Development

Successfully merging a pull request may close this issue.

4 participants