timescale / helm-charts

Configuration and Documentation to run TimescaleDB in your Kubernetes cluster
Apache License 2.0
264 stars 223 forks source link

helm lint (v3.10.3) fails on timescaledb-single chart #542

Closed davidandreoletti closed 1 year ago

davidandreoletti commented 1 year ago

What happened?

Linting the timescaledb-single chart with a helm 3.10.3 fails. Failure not checked with early helm version. https://github.com/helm/helm/issues/10149#issuecomment-980243695 concludes the error is not in helm (so far).

This PR provides a suitable fix to lint / generate a yaml valid pgbackrest.yaml file.

Did you expect to see something different?

No linting error.

How to reproduce it (as minimally and precisely as possible):

+ Install helm 3.10.3.

+ helm repo add timescale https://charts.timescale.com
"timescale" already exists with the same configuration, skipping

+ helm repo update
Hang tight while we grab the latest from your chart repositories...
...Successfully got an update from the "timescale" chart repository
Update Complete. ⎈Happy Helming!⎈

+ helm fetch timescale/timescaledb-single --untar

+ helm lint <extracted folder>
==> Linting timescaledb-single-original
[INFO] Chart.yaml: icon is recommended
[ERROR] templates/pgbackrest.yaml: unable to parse YAML: invalid Yaml document separator: apiVersion: batch/v1

Environment

Helm: 3.10.3 values.yaml: N/A K8s: N/A