Currently, all helm-install checks are failing. This addresses the issue
by changing the branch of the chart-testing-action to dev-v2 which
provides support for Helm 2. Helm 3, which is the new default,
introduces breaking changes.
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)
fixes #318
Special notes for your reviewer:
Alternatively, we could add the unit to the config here, but we'd be pulling in Helm 3 with several breaking changes.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]
monotek
commented
4 years ago
What this PR does / why we need it:
Currently, all helm-install checks are failing. This addresses the issue by changing the branch of the chart-testing-action to
dev-v2which provides support for Helm 2. Helm 3, which is the new default, introduces breaking changes.Which issue this PR fixes
(optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)format, will close that issue when PR gets merged)Special notes for your reviewer:
Alternatively, we could add the unit to the config here, but we'd be pulling in Helm 3 with several breaking changes.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]
[fluentd-elasticsearch])