helm
helm copied to clipboard
helm
Proposal: disable values schema validation for charts including a values.schema.json file
Even when a chart designer/maintainer has decided to include a values.schema.json file in the Helm chart he/she releases it could be that the user of that chart (using it directly or as a dependency in an integration/umbrella Helm chart) cannot assure to fulfill the defined schema for the deployment specific values passed to that subchart (e.g. he/she passes a deployment specific values.yaml file to the helm install/upgrade command applied on that chart, where some "old" Helm parameter removed from the values.schema.json of a newer version of that (sub)chart still remains).
I think it would be useful to support a new option in helm install, helm upgrade, helm lint and helm template allowing to disable the values schema validation (in case a values.schema.json file is packaged in the target Helm chart or some listed subchart)
Running into this issue now. A helm chart im using has a values.schema.json which points to a git repo for some of the values, which unfortunately means I cant helm install the chart without an internet connection unless I untar the helm chart, remove the schema file, and then install. Would be interested in a flag like --skip-validation where it does the equivalent of not having the schema file in the chart. Does a flag like this exist?
This issue has been marked as stale because it has been open for 90 days with no activity. This thread will be automatically closed in 30 days if no further activity occurs.
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
This issue has been marked as stale because it has been open for 90 days with no activity. This thread will be automatically closed in 30 days if no further activity occurs.
As a developer of charts with linked values schemas I would also want to not prevent my users from failing to install the script when not having internet connection. :+1: for this feature request.
Hello, I made an attempt to solve this problem. This is my first time contributing so, please let me know if anything is wrong so I can fix.
This issue has been marked as stale because it has been open for 90 days with no activity. This thread will be automatically closed in 30 days if no further activity occurs.
is there any progress on this? In air-gapped evironments this can be crucial.
We've made an in-CI workaround by fetching the chart to our CI agent, inflating it, renaming the json schema file, and then running the helm upgrade with the local chart path rather than the remote. Its ugly and I hate it.
I have created a merge request (https://github.com/helm/helm/pull/12743) that solves this issue, including tests and doc updates. However, it seems to be really hard to get hold of a maintainer to look into it. I have mentioned it on the #helm-dev slack channel, but no luck. I will try again, but maybe you can vote on the PR if you want to see this fixed.
@anessi it looks like your PR got approved! I would love to have this merged in (bumped up against this myself earlier). Do you need any help rebasing or anything?
@anessi it looks like your PR got approved! I would love to have this merged in (bumped up against this myself earlier). Do you need any help rebasing or anything?
@tthornton3-chwy : I'd love to merge, but I don't have write permissions, so depend on the maintainers to merge it. I will leave a comment on the merge request.