Introduce contour-crds and contour-gateway-provisioner charts

[pschichtel]

As suggested in #3 a PR that introduces a chart for the gateway provisioner. I needed to build this anyway, because we standardized on deployments using helm. As a preparation I've also added a chart for the CRDs, which could be pulled by the main contour chart as well.

What I considered, but didn't do in this PR:

• change the contour chart to pull contour-crds
• pull the gateway-api CRDs via oci://docker.io/envoyproxy/gateway-crds-helm

[pschichtel]

I bootstrapped the contour-gateway-provisioner chart using helm create and then reduced that to what's needed and merged in the non-crd resources from the official manifest file.

[pschichtel]

I have an internal fork deployed to a cluster in together with the contour chart (with manageCRDs: false)

[deepy]

I need to freshen up a bit more on subcharts and take a second pass, but it looks good and I think this is something we absolutely want to have

[tsaarni]

I’m curious about the community consensus on splitting CRDs from the Contour chart. I’m not an experienced Helm user, so I’m not familiar on the best practices. My assumption is that this would be a breaking change, so it should be done before the Contour Helm chart is set to a stable version.

@pschichtel Would it be possible to separate the contour-gateway-provisioner changes from this PR, so that they can be handled independently of the CRD discussion?

[pschichtel]

@tsaarni This PR is not a breaking change, it just introduces a chart that carries nothing but the CRDs in order to possibly share it with the contour chart in the future. So I don't understand the issue. Also: I have long moved away from contour for technical reasons in my infrastructure, so I won't be investing time here when things are unclear. Anyone else is fine to take over in the meantime.

[tsaarni]

Thanks, @pschichtel, for your response!

This PR is not a breaking change, it just introduces a chart that carries nothing but the CRDs in order to possibly share it with the contour chart in the future. So I don't understand the issue.

While this PR doesn’t remove CRDs from main chart, I think we should avoid duplicating CRDs across multiple charts and instead have them exclusively in the separate CRD chart. If we removed the CRDs from the main chart, that change could be breaking (if done after stable release).

Also: I have long moved away from contour for technical reasons in my infrastructure, so I won't be investing time here when things are unclear.

Understood, and thanks for your contributions so far!

[Sefiris]

While this PR doesn’t remove CRDs from main chart, I think we should avoid duplicating CRDs across multiple charts and instead have them exclusively in the separate CRD chart. If we removed the CRDs from the main chart, that change could be breaking (if done after stable release).

Better to get this over with as soon as possible IMHO, so maybe let's remove this from the chart? since @pschichtel mentioned he would not want to invest more time into it are you okay with me taking over this PR @tsaarni?

[tsaarni]

are you okay with me taking over this PR

Yes absolutely, I created new issue specifically for the CRD chart #21, Thank you @Sefiris