helm-charts icon indicating copy to clipboard operation
helm-charts copied to clipboard

Published 20 hours ago • verified publisher icon temporalio

Remove grafana dependency

Open BrianKopp opened this issue 7 months ago • 1 comments

What was changed

Removed Grafana dependency and related config.

Why?

Configuration & installation of monitoring tools is a cluster-level consideration, and shouldn't be specified in an application such as this.

Checklist

  1. Closes #518

  2. How was this tested: Diff between helm template main and this branch has no differences.

Note - if running helm from the directory or source, and if you don't run helm dependencies update, then helm will want to install grafana since it's enabled by default.

  1. Any docs updates needed?

README is updated.

This will be a breaking change for folks who have grafana enabled, since it will delete their temporal-installed-grafana.

BrianKopp avatar Jul 05 '24 13:07 BrianKopp