Update to use seconds

telemetry/prometheus/config.go

@@ -16,7 +16,7 @@ type Config struct {
 	Subsystem            string // optional
 	HistogramBucketCount int    // Number of buckets for histogram, default to 1000
 	// Number of buckets for time buckets, default to 1000.
-	// The bucket size is 10ms, so the maximum covered time range is 10ms * TimeBucketCount.
+	// The bucket size is 0.01s(10ms), so the maximum covered time range is 10ms * TimeBucketCount.
 	TimeBucketCount int
 }
 

telemetry/prometheus/metrics.go

@@ -198,15 +198,15 @@ func (p *metrics) Time(name string, value time.Duration, tags []string) {
 			Namespace: p.cfg.Namespace,
 			Subsystem: p.cfg.Subsystem,
 			Help:      name + " timing histogram",
-			// Given bucket=10ms, the maximum covered time range is 10ms * TimeBucketCount
-			Buckets: prometheus.LinearBuckets(0, 10, p.cfg.TimeBucketCount),
+			// Given bucket=0.01s(10ms), the maximum covered time range is 10ms * TimeBucketCount
+			Buckets: prometheus.LinearBuckets(0, 0.01, p.cfg.TimeBucketCount),
 		}, labels)
 		prometheus.MustRegister(histogramVec)
 		p.histogramVecs[name] = histogramVec
 	}
 
 	// Convert time.Duration to seconds since Prometheus prefers base units
-	histogramVec.WithLabelValues(labelValues...).Observe(float64(value.Milliseconds()))
+	histogramVec.WithLabelValues(labelValues...).Observe(float64(value.Seconds()))
 }
 
 // Latency is a helper function to measure the latency of a routine.