Feature request: skip `values.yaml` yamllint when it is absent

[thiagowfx]

Currently, ct lint fails whenever values.yaml is missing in a chart.

Sometimes there is simply no need to provide values.yaml (e.g. when installing manifests that don't need to be configurable).

Expected: ct lint should not fail when values.yaml is absent.

Observed:

% ct lint
[...]
[Errno 2] No such file or directory: 'helm/mychart/values.yaml'

------------------------------------------------------------------------------------------------------------------------
 ✖︎ mychart => (version: "1.2.2", path: "helm/mychart") > failed waiting for process: exit status 255
------------------------------------------------------------------------------------------------------------------------
Error: failed linting charts: failed processing charts
failed linting charts: failed processing charts

When running with debug: true, I see:

>>> yamllint --config-file .yamllint.yaml helm/mychart/Chart.yaml
>>> yamllint --config-file .yamllint.yaml helm/mychart/values.yaml
[Errno 2] No such file or directory: 'helm/mychart/values.yaml'

So the problem is that it attempts to run yamllint in the non-existing values.yaml.

I have validate-yaml: true in my ct config.

The current workaround is to create an empty values.yaml file in the chart, just for the sake of pleasing ct lint.

[thiagowfx]

The fix would be a matter of checking for the existence of values.yaml here, before linting.

[github-actions]

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

[github-actions]

This issue was closed because it has been stalled for 5 days with no activity.

[github-actions]

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

[thiagowfx]

!stayalive

Note that a PR has been sent to close this issue