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): correct cumulative count calculation
All checks were successful
ci/woodpecker/push/lint Pipeline was successful
Details
ci/woodpecker/push/test Pipeline was successful
Details
ci/woodpecker/push/build Pipeline was successful
Details

Browse source
This commit is contained in:
Tom Neuber 2024-12-30 23:42:52 +01:00
parent 3a1e4fdc3c
commit 53954090a3
Signed by: tom
GPG key ID: F17EFE4272D89FF6
1 changed files with 7 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
internal/exporter/metrics.go
View file

@ -193,15 +193,19 @@ 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 {
cumulativeCount++ buckets[bound]++
} }
buckets[bound] += cumulativeCount
} }
} }
var cumulativeCount uint64
for _, bound := range bucketBounds {
cumulativeCount += buckets[bound]
buckets[bound] = cumulativeCount
}
m.counter += uint(len(data)) m.counter += uint(len(data))
ch <- prometheus.MustNewConstMetric( ch <- prometheus.MustNewConstMetric(
m.metricRequestsTotal, m.metricRequestsTotal,