timescale / tobs

tobs - The Observability Stack for Kubernetes. Easy install of a full observability stack into a k8s cluster with Helm charts.
Apache License 2.0
563 stars 60 forks source link

chart: improve make targets and add cleanup target #588

Closed paulfantom closed 2 years ago

paulfantom commented 2 years ago

What this PR does / why we need it

Adding checking of shell scripts. Since our development scripts from scripts/ are less important that ones from chart/scripts/ I decided to use different severities when linting those scripts, hence running shellcheck twice.

This PR is building on top of https://github.com/timescale/tobs/pull/574 and will need rebase after menitioned PR is merged.

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