search

timescale / helm-charts

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

Update README.md #540

Closed alejandrodnm closed 1 year ago

alejandrodnm commented 1 year ago

Signed-off-by: alejandrodnm alejandrodnm@gmail.com

What this PR does / why we need it

Which issue this PR fixes

Special notes for your reviewer

Checklist

onprem commented 1 year ago

You'd need to bump the chart version though, for how our CI works.

alejandrodnm commented 1 year ago

@onprem do I need to bump the chart version just for this change?

onprem commented 1 year ago

@alejandrodnm Yes, we have CD setup to release a new version of chart with every PR, if there any changes in it's files. Since the README file is under the chart directory, we need to bump the version. Here we can just do a patch release.

alejandrodnm commented 1 year ago

@onprem the errors are related to a fix for the alerts that was made in this PR https://github.com/timescale/promscale/pull/1794 Is there something else I need to do?

alejandrodnm commented 1 year ago

I'll leave this open until the alerts are fixed on another PR

onprem commented 1 year ago

I have created #541 for the alerts. And these checks are unrelated and not required for this PR so I am going ahead and merging. Thanks!