timescale / helm-charts

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

Restore-or-init script migration and shellcheck cleanup #521

Closed paulfantom closed 1 year ago

paulfantom commented 1 year ago

What this PR does / why we need it

Moving and refactoring restore_or_init.sh script out of being embedded in a ConfigMap template to a separate directory to allow easier code maintenance. Before script could be moved, following changes had to be applied (as seen in PR):

This PR is also moving towards shellchecking scripts BEFORE they are mangled by helm template. This should allow easier development and improve code readability.

This the last PR of an effort listed in https://github.com/timescale/helm-charts/pull/505

PR is blocked by https://github.com/timescale/helm-charts/pull/515, https://github.com/timescale/helm-charts/pull/516, https://github.com/timescale/helm-charts/pull/517, https://github.com/timescale/helm-charts/pull/518, https://github.com/timescale/helm-charts/pull/519, and #520

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

Checklist