bug: Missing CRDs for 3.4.2 Helm chart release

[marcofranssen]

Describe the issue

We are fetching the crds from https://github.com/fluent/fluent-operator/releases/download/v${var.chart_version}/setup.yaml

As the Helm chart release is done only on the charts repository https://github.com/fluent/helm-charts/releases/tag/fluent-operator-3.4.2 there is currently no matching setup.yaml.

Options to fix this:

1. Publish a release on operator repo.
2. Attach the setup.yaml on the chart release.
3. Having another way (helm chart, e.g. fluent-operator-crds) to publish the crds.

To Reproduce

N.A.

Expected behavior

The CRDs for 3.4.2 or newer releases are published in some form.

Your Environment

N.A.

How did you install fluent operator?

We have a piece of Terraform which installs both CRDs as shown above, as well the Chart by taking a version variable.

Additional context

We are in the process of improving the release process, so we need to think about how we want to handle this.

[joshuabaird]

I like the idea of decoupling fluent-operator and Helm releases so that we can release Helm charts out of band of fluent-operator releases.

[joshuabaird]

Also -- posting some messages from slack:

I think the way we handle the sub-CRD charts is also broken.

I'm not sure we should be referencing a local directory here.

I don't think it's possible to even install the CRDs right now from a remote repository - could be wrong.

I think we need to think about how to handle the CRD charts in a different way. I tend to like publishing entirely chart with the CRD(s) in a templates directory like Karpenter does.