[Feature Request] integrate with kube-prometheus-stack

[ImSingee]

Currently, the chart only supports installing its own prometheus and grafana.

However, in most cases, user has already set up the kube-prometheus-stack.

Feature Request: add a way to just utilize existing promtheus and grafana for monitoring

[luiscosta18]

Hello

I believe that you mean is this: https://github.com/prometheus-community/helm-charts/tree/main/charts/kube-prometheus-stack

I've my own monitoring stack:

• prometheus or opentelemetry, doing service discovery based on annotations https://github.com/temporalio/helm-charts/blob/main/charts/temporal/templates/server-deployment.yaml#L26-L28
• grafana, with temporal dashboard https://github.com/temporalio/dashboards/tree/master

So, having prometheus + grafana as a dependency in Chart.yaml, would only be usefull to test the chart locally. @robholland are these dependencies really needed in Chart.yaml? (of course they can be disabed in values.yaml)

[robholland]

Done in https://github.com/temporalio/helm-charts/pull/534 and https://github.com/temporalio/helm-charts/pull/526, but these are on the v1 branch. There is still more work to do there before these will be released.