yolokube/country-geo-locations
yolokube/country-geo-locations
2
1
Fork 0
Code Issues 1 Pull requests Projects Releases Packages 1 Wiki Activity

fix(metrics): add cumulative count as histogram expects #57

Merged
tom merged 1 commit from tn-fix-histogram-quantile-monotonicity into main 2024-12-30 23:20:13 +01:00
Conversation 0 Commits 1 Files changed 1 +3 -1
1 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
internal/exporter/metrics.go
View file

@ -193,10 +193,12 @@ func (m *Metrics) collectReqeustDataMetrics(ch chan<- prometheus.Metric, queue *
sum += latency sum += latency
count++ count++
var cumulativeCount uint64
for _, bound := range bucketBounds { for _, bound := range bucketBounds {
if latency <= bound { if latency <= bound {
buckets[bound]++ cumulativeCount++
} }
buckets[bound] += cumulativeCount
} }
} }