fork of https://github.com/sourcegraph/zoekt
Expand upper range on indexing delay metric (#847)
When adding this metric, I got the calculation wrong. So the maximum bucket was capped at ~1.3 days, which we easily saturate at the 90% percentile.
This PR updates it to ~ 5.5 days to be safe:
```
>>> math.pow(2, 14 - 1) / 60 / 24
5.688888888888889
```
Note the `- 1`, which is the key piece I missed before.
author
Julie Tibshirani
committer
GitHub
date
(Oct 23, 2024, 9:23 AM -0700)
commit
d64e9313
d64e9313e20bcb61c38e5999305f15b8a995c175
parent
cc190492
cc190492c8d1e3ece352fc56ff6a6d0ba951f855
+1
-1
+1
-1
cmd/zoekt-sourcegraph-indexserver/main.go
+1
-1
cmd/zoekt-sourcegraph-indexserver/main.go
···
92
92
metricIndexingDelay = promauto.NewHistogramVec(prometheus.HistogramOpts{
93
93
Name: "index_indexing_delay_seconds",
94
94
Help: "A histogram of durations from when an index job is added to the queue, to the time it completes.",
95
-
Buckets: prometheus.ExponentialBuckets(60, 2, 12), // 1m -> ~3 days
95
+
Buckets: prometheus.ExponentialBuckets(60, 2, 14), // 1 minute -> 5.5 days
96
96
}, []string{
97
97
"state", // state is an indexState
98
98
"name", // the name of the repository that was indexed