Hi all! I'm trying to deploy a helm chart with garden 0.13.7 and it's failing with the following error:

Error: INSTALLATION FAILED: values don't meet the specifications of the schema(s) in the following chart(s):
jupyterhub:
- (root): Additional property .garden is not allowed

So, the error itself is pretty obvious. The jupyterhub helm chart is invalidating any additional properties not included in the chart's schema. The only solution to this that I'm currently coming up with is to fork their helm-chart and edit the schema. That is certainly not an ideal solution and would require a lot of extra work on my end, and I don't have any need to make further edits to that helm chart schema other than to allow additional properties at the root. I've installed 7-8 other charts with Garden and have had no issues. Absolutely in love with it. My question: Is there a way to disable the

--set \.garden.mode=default

argument that is added to the command? The command and output that's being run by garden is:

Command "/home/jessica/.garden/tools/helm/b736014484fd8f0f/linux-amd64/helm --kube-context jess-epx-platform --kubeconfig /home/jessica/GolandProjects/EpxPlatform-BaseDockerfile/garden/terraform/kubeconfig.yaml --namespace jupyterhub install jupyterhub jupyterhub --version 1.2.0 --repo https://jupyterhub.github.io/helm-chart/ --namespace jupyterhub --timeout 300s --values /home/jessica/GolandProjects/EpxPlatform-BaseDockerfile/garden/actions/deploy/platform-hub/helm/base.yaml --values /tmp/8336045834d1f61022757a21faa86dee --set \.garden.mode=default" failed with code 1:

This can be reproduced with the following deploy action:

kind: Deploy
type: helm
name: platformhub-helm
spec:
  releaseName: jupyterhub
  namespace: jupyterhub
  chart:
    repo: https://jupyterhub.github.io/helm-chart/
    name: jupyterhub
    version: "1.2.0"

If anyone has any insight to this, please help! I really really don't want to start maintaining another helm chart.

Hi @clean-kite-99549, welcome to the community and thank you so much for the kind words! I will start by reproducing your error then get right back to you 🕐

@quaint-dress-831 Thanks so much for the quick turnaround on this. Great support and thanks for confirming it. I don't envy the core team for trying to solve this without breaking the existing implementation.

Comment added. Feel free to add your own thoughts. We'll get to this Monday!

Hey @quaint-dress-831. Thanks for the update. It looks like that could be one of those PR's and issues that never end up getting merged/resolved but still referenced every few months(we all know those 😆 ). The vacation message was posted a month ago, and that person has committed within the last week to the repository. With that being said though, I understand my issue is an edge case and has a viable workaround, so fixing it on your side is of low value. I'll move forward with forking the helm chart. Really appreciate y'all looking at and discussing this issue though. It's great to see that level of responsiveness and does nothing but bolster my confidence in the product moving forward. Great work!