-
Notifications
You must be signed in to change notification settings - Fork 689
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
Producing knative-compatible yamls in a sustainable way #2254
Comments
Exposing a parts of the Contour YAML as a kustomize base can help with this. xref #2088 |
Hey @mattmoor, in a perfect world, what would you want here? Some Kustomize yaml? A helm chart? |
I don't have a strong preference, but what I ultimately did was to produce two yamls with This was much simpler than the crazy stuff I was doing previously, but still harder than it should be to find everything (e.g. |
Sorry if this isn't the best place for this, but what is the reason that Contour can't support knative natively? Why the need for the separate project? |
This sort of ties into: #2809 We use a separate installation of Contour for each "class"-equivalent in the Knative model. If a single install of Contour could satisfy Knative's needs, then we could maybe discuss it, but I'm still not sure it makes sense given costs/benefits (e.g. vendoring knative?) and the longer term prospects (e.g. ingress v2). |
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 |
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 |
Please describe the problem you have
knative.dev/net-contour requires a slightly odd configuration of Contour, where we effectively use two installs:
ExternalIP
visibility, where the Envoys have atype: LoadBalancer
service, and a Contour, which programs them with a specified class (e.g.contour-external
)ClusterLocal
visibility, where the Envoys have atype: ClusterIP
service, and a Contour, which programs them with a specified class (e.g.contour-internal
)The "class" and service name are configured for each visibility through our
config-contour
configmap here.To bootstrap things, I produced artisan hand-crafted yamls to achieve this, and I'll probably do this a few more times before we sort out how to do this sustainably, but I wanted to open this issue to seek guidance on how we might do this most sustainably moving forward.
The text was updated successfully, but these errors were encountered: