-
Notifications
You must be signed in to change notification settings - Fork 297
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
Add comment support when parsing YAML files #602
Comments
Hi @fredgate. This comment in the Helm output is a Helm-specific thing, and is not a part of the YAML specification so it is not parsed by conftest. I will take a look into what can be done here to make this clearer in the output when testing helm files. |
@fredgate The YAML package we use [1] does not return us any comments, so adding this would be a significant refactor. Time allowing we may still add this, but I wanted to let you know it probably won't be soon. That said, I'd welcome a PR if you have the time! |
I have an identical issue. I want to test yamls where they have fields and these fields have values as a template standards:
This kind of value is a standard from ytt templates. When I generated the test I got a null value. |
I have a Helm chart that I test for Kubernetes API deprecations using deprek8ion policies.
Here is the command and output I have :
In the result, the filename is missing.
My chart I have one file by resource (one file for deployment, another file for service, yet another file for ingress...), but the
helm template
command render one output with a multi document YAML like thatEach resource is prefixed with a comment indicating the file containing the resource definition. But this information is not reported in
conftest
output.How to proceed ?
The text was updated successfully, but these errors were encountered: