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.

Sign up for GitHub

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

Jump to bottom

New configuration format #1247

Closed
pastequo opened this issue Jan 25, 2024 · 2 comments · Fixed by #1249
Closed

New configuration format #1247

pastequo opened this issue Jan 25, 2024 · 2 comments · Fixed by #1249

Comments

@pastequo
Copy link
Contributor

This Issue comes from this design document: https://docs.google.com/document/d/12X1-9nhjAhpf-Wlt3RbdoMbvx31zFBva/edit#heading=h.gjdgxs

The proposal is to make the configuration more flexible, more extensible. This issue focus only on the new configuration format, without any filtering capabilities (that will come in another issue)

See the design document for more information
This was referenced Jan 25, 2024
Closed
Merged
pastequo added a commit to pastequo/net-istio that referenced this issue Jan 25, 2024
@pastequo
feat: New istio config format (knative-extensions#1247)
22c45ce
pastequo added a commit to pastequo/net-istio that referenced this issue Jan 29, 2024
@pastequo
feat: New istio config format (knative-extensions#1247)
4befabe
pastequo added a commit to pastequo/net-istio that referenced this issue Jan 30, 2024
@pastequo
feat: New istio config format (knative-extensions#1247)
c29c707
pastequo added a commit to pastequo/net-istio that referenced this issue Feb 5, 2024
@pastequo
feat: New istio config format (knative-extensions#1247)
fc6e93f
pastequo added a commit to pastequo/net-istio that referenced this issue Feb 6, 2024
@pastequo
feat: New istio config format (knative-extensions#1247)
5ead169
@dprotaso dprotaso mentioned this issue Feb 6, 2024
Open
knative-prow bot pushed a commit that referenced this issue Feb 6, 2024
@pastequo
feat: New istio config format (#1247) (#1249)
708e6d3
pastequo added a commit to pastequo/net-istio that referenced this issue Feb 8, 2024
@pastequo
docs: Add documentation about the new config format (knative-extensio…
b161e47 
…ns#1247)
pastequo added a commit to pastequo/net-istio that referenced this issue Feb 8, 2024
@pastequo
docs: Add documentation about the new config format (knative-extensio…
52f2359 
…ns#1247)
pastequo added a commit to pastequo/net-istio that referenced this issue Feb 8, 2024
@pastequo
docs: Add documentation about the new config format (knative-extensio…
b488336 
…ns#1247)
pastequo added a commit to pastequo/net-istio that referenced this issue Feb 8, 2024
@pastequo
docs: Add documentation about the new config format (knative-extensio…
aa8d5dd 
…ns#1247)
knative-prow bot pushed a commit that referenced this issue Feb 9, 2024
@pastequo
docs: Add documentation about the new config format (#1247) (#1257)
271819f 
* docs: Add documentation about the new config format (#1247)

* chore: Doc formatting

* chore: Remove default values
@pastequo pastequo mentioned this issue Feb 9, 2024
Merged
pastequo added a commit to pastequo/net-istio that referenced this issue Feb 16, 2024
@pastequo
feat: New istio config format (knative-extensions#1247)
0d8cbcf
@nguyenthai0107
Copy link

Hello @pastequo .
Is there a way to implement with Nginx Ingress ?.
Im believe that a lot of company which was built and using technology and system base on Nginx Ingress ( without supported network layer such as Contour, Kourier, Istio). It will more options, more flexible if this was implemented in configmap.
Thank you.

@dprotaso
Copy link
Contributor

dprotaso commented Apr 3, 2024

@nguyenthai0107 nginx support would come through our use of the Gateway API - we have a plugin here but we haven't done any testing with nginx.

https://github.com/knative-extensions/net-gateway-api

Curious do you require the need to select different gateways?

@dprotaso dprotaso mentioned this issue Apr 22, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: New istio config format pastequo/net-istio
3 participants
@dprotaso @pastequo @nguyenthai0107