Metrics using tally are not compatible with prometheus
mohakkataria opened this issue · 0 comments
Describe the bug
Currently the tally version used if v3.3.5 (https://github.com/uber-go/cadence-client/blob/v0.18.5/go.mod#L16) uses a different prometheus client(https://github.com/uber-go/tally/blob/v3.3.5/prometheus/reporter.go#L31) than https://github.com/prometheus/client_golang. This means that if there any other dependency using the official prometheus client's defaultRegistry, those metrics will not be reported on tally since they do not share the common default registry
To Reproduce
Is the issue reproducible?
- Yes
Steps to reproduce the behavior:
Just need to add a metric using https://github.com/prometheus/client_golang in the code and check the metrics endpoint. The metric registered with this client will be missing
Expected behavior
All metrics registered anyhow using prometheus library should be reported on metrics endpoint even if it's tally which responds to metrics instead
Screenshots
None :(
Additional context
Solution would be to update the tally version to v4.1.0 https://github.com/uber-go/tally/blob/v4.1.0/prometheus/reporter.go#L30