-
Notifications
You must be signed in to change notification settings - Fork 679
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
Update defaults for guidelines on Configuring Envoy as an Edge Proxy #4717
Comments
The Contour project currently lacks enough contributors to adequately respond to all Issues. This bot triages Issues according to the following rules:
You can:
Please send feedback to the #contour channel in the Kubernetes Slack |
Seems still valuable to do, unstaling |
The Contour project currently lacks enough contributors to adequately respond to all Issues. This bot triages Issues according to the following rules:
You can:
Please send feedback to the #contour channel in the Kubernetes Slack |
Can you provide a way to make these changes? I am trying to find out where to make these changes.. |
This will require code changes to Contour to configure Envoy with some of these values |
Is this not what contour is, making envoy an edge proxy not a service mesh? Why would these values not be exposed by default? |
Contour does not aim to make every envoy configuration knob available to users to configure, this issue and the linked Envoy guidelines are for sensible defaults that we should apply, which involve code changes to Contour so the xDS server can send them to Envoy. Some values may additionally be exposed as user configurable, but that again involves code changes to Contour to expose those in an API and to send that configuration to Envoy. |
Envoy provides documentation guidelines on Configuring Envoy as an edge proxy
Contour has implemented some of these guidelines, but we should do another pass to make sure we have covered it all, for proxying TLS/TCP, HTTP/1.1, HTTP/2, etc.
There may be duplicates of this issue that can be linked, closed, etc. as needed
Tasks:
The text was updated successfully, but these errors were encountered: