https://garden.io logo
#🌱|help-and-getting-started
helm modules and crds?
# 🌱|help-and-getting-started
t

tall-greece-98310

03/01/2023, 11:24 AM
Hi, I'm trying to use a helm chart that has a
crds
directory alongside the typical templates directory. It doesn't seem garden propagates this directory into the
.garden/build/
hierarchy, so the crds are hence not installed. Is this intended or a bug? If intended how are you supposed to get CRDs installed? Thanks!
Looking at it it seems like the logic that collects things into
.garden/build
only copies files and the templates dir, but no other dirs. The
crds
dir is the standard way in helm 3 to install CRDs, so this seems a bug to me: https://helm.sh/docs/chart_best_practices/custom_resource_definitions/
b

bright-policeman-43626

03/01/2023, 2:25 PM
Hello @tall-greece-98310 🌸! Thanks for reporting this, Would you mind sharing the Garden configuration you are trying to deploy with those CRD's? Is this a public chart that you are trying to install or it's something you can't share with me to try to replicate the issue?
t

tall-greece-98310

03/01/2023, 2:26 PM
it's not a public chart, but any chart that has a top level looking like
/Chart.yaml
file with a
/crds
and
/templates
the crds dir does not make it into the
.garden/build
dir for the module
my reading of helm docs is that helm v3 prefers to install crds in this way
b

bright-policeman-43626

03/01/2023, 2:35 PM
I can try to reproduce the issue with that one that seems like replicates the chart structure you are mentioning, yes. Having a separate /crds folder is the preferred way to install those CRD's with Helm3. I will try to get that Kong installed with the CRD's!
t

tall-greece-98310

03/01/2023, 5:31 PM
yes, that's the structure my chart has
the
crds
dir shwos in the
.garden/sources
dir but not in the
.garden/build
dir and hence are not applied. I worked around it using an explicit kubernetes module referencing the crd files (via a repositoryUrl to the git repo of the chart).
b

bright-policeman-43626

03/02/2023, 9:09 AM
Hey @tall-greece-98310 thanks a lot for reporting this, I was able to reproduce this issue and I created a Github Issue with the details! https://github.com/garden-io/garden/issues/3864 I think this is a bug but it might also be a Feature request. We will see whenever one of our developers takes a look into this one! Thanks again!
t

tall-greece-98310

03/02/2023, 9:10 AM
cheers - thanks for picking it up. Was going to file an issue, just wanted to ask first in case it was pilot error by me.
b

bright-policeman-43626

03/02/2023, 9:15 AM
Thank you, it seems like a bug, but perhaps we might convert it later to a feature request in case it's identified as that later on. I will be on the loop and will update you once we get some friction in the issue! Thanks for reporting ❤️
f

freezing-pharmacist-34446

03/03/2023, 12:05 PM
Hi @tall-greece-98310 , I thought we fixed that issue last summer 🤔 https://github.com/garden-io/garden/pull/3004/files# I have a couple of helm charts with crds that I install with a garden helm module and I can see the crds directory in the .garden/build.
Would love to see the config of either your helm chart / garden module or where you reproduced it @bright-policeman-43626 🙂
t

tall-greece-98310

03/03/2023, 1:03 PM
I don't have anything public. It is a helm chart module using a git repo for the chart sources via repositoryURL
I can share a screen shot later of structure.
b

bright-policeman-43626

03/03/2023, 2:01 PM
Sure! Here you can find where I reproduced this issue with Kong public chart https://github.com/ShankyJS/garden-charts-crds
Hopefully that one helps @freezing-pharmacist-34446!🌸
f

freezing-pharmacist-34446

03/03/2023, 3:40 PM
Thanks!
h

hallowed-apartment-22484

03/03/2023, 4:39 PM
Hi! Chiming in to say I’ve seen this too with scylladb’s helm charts https://operator.docs.scylladb.com/stable/helm.html https://scylla-operator-charts.storage.googleapis.com/latest (Don’t do stable - they apparently deleted stable xD not that stable after all)
b

bright-policeman-43626

03/03/2023, 4:55 PM
Thanks for sharing that @hallowed-apartment-22484 that help us to confirm that this behavior is still happening after the fix implemented on pr-3004!
h

hallowed-apartment-22484

03/03/2023, 5:08 PM
Ofc!
14 Views