Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature Request: helm lint should verify that values used in charts are present in values.yaml #12982

Open
dgn opened this issue Apr 24, 2024 · 1 comment

Comments

@dgn
Copy link

dgn commented Apr 24, 2024

Hi! I'd like to see helm lint perform some checks against the values.yaml and the templates, namely that any usage of .Values.someField is cross-checked against the values.yaml file to make sure that it indeed contains someField.

Now, I know that this is sort of complex due to the nature of templating, but in a first iteration, I think a check that just verifies that any direct reference of a value (using .Values.someField) is backed up by a real value.

If there's interest in a feature like this, I would like to submit a PR to implement it

@gjenkins8
Copy link
Contributor

It is frequently expected that the default values provided with a chart are incomplete. And that the user (chart installer) is expected to provide additional values. For this reason I don't think it really works for lint to verify that the default values are complete.

Better I think would be to provide a jsonschema with the chart. Which verifies values at install time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants