<@U08MR0PSATF> <@U066HU6NCL9> re: the Helm chart.....
# fluent-operatorj
Josh Baird
06/05/2025, 1:20 PM@Marco Franssen @Chengwei Guo re: the Helm chart.. do you find it confusing that we maintain a "local" copy of the chart in the fluent-operator repo and then have to manually sync to helm-charts to release?
👀 1
m
Marco Franssen
06/05/2025, 1:41 PMPro: CRDs are in same repo
Con: Indeed weird to having to copy it over.
Maybe better to copy CRDs when required and do all helm stuff in the helm repo
p
Pat
06/05/2025, 1:43 PMThe other advantage is it's not released officially until it's in the fluent repo so you can test things on main before you release to the wild
Pat
06/05/2025, 1:44 PMI've followed this approach a lot with private repos doing the actual chart dev and it's trivial to set up CI to sync over on release.
j
Josh Baird
06/05/2025, 1:45 PMRight - that makes sense, but I do think it's confusing for contributors in it's current state because folks are opening PR's in helm-charts, etc.
👍 1
Josh Baird
06/05/2025, 1:45 PMThis could probably be solved with some documentation updates.
p
Pat
06/05/2025, 2:03 PMYeah you need to decide on what you want and enforce it. We can automatically close PRs in the wrong place or remove the current location or other things.
Either way it should be enforced automatically, docs are ok but people may not see them.
👍 2
Pat
06/05/2025, 2:11 PMIf you're going to copy crds then we should try to make this obvious so people don't change them for example. The less effort required to manually review things the better, particularly as it's a multi chart repo