fix sub-second precision metric reporting (#16358)

dprotaso · web-flow · commit 4004491765ef · 2026-01-26T16:45:31.000Z
diff --git a/pkg/autoscaler/metrics/stats_scraper.go b/pkg/autoscaler/metrics/stats_scraper.go
@@ -226,7 +226,7 @@ func (s *serviceScraper) Scrape(window time.Duration) (stat Stat, err error) {
 			return
 		}
 		scrapeTime := s.clock.Since(startTime)
-		scrapeTimeSec := float64(scrapeTime / time.Second)
+		scrapeTimeSec := scrapeTime.Seconds()
 		s.duration.Record(context.Background(), scrapeTimeSec, metric.WithAttributeSet(s.attrs))
 	}()
 
diff --git a/pkg/queue/request_metric.go b/pkg/queue/request_metric.go
@@ -18,7 +18,6 @@ package queue
 
 import (
 	"net/http"
-	"time"
 
 	"go.opentelemetry.io/otel/metric"
 	"k8s.io/utils/clock"
@@ -103,7 +102,7 @@ func (h *appRequestMetricsHandler) ServeHTTP(w http.ResponseWriter, r *http.Requ
 			status = http.StatusInternalServerError
 		}
 
-		elapsedTime := float64(latency / time.Second)
+		elapsedTime := latency.Seconds()
 		h.duration.Record(r.Context(), elapsedTime,
 			metric.WithAttributes(semconv.HTTPResponseStatusCode(status)),
 		)

Original file line number	Diff line number	Diff line change
`@@ -226,7 +226,7 @@ func (s *serviceScraper) Scrape(window time.Duration) (stat Stat, err error) {`
`226`	`226`	`return`
`227`	`227`	`}`
`228`	`228`	`scrapeTime := s.clock.Since(startTime)`
`229`		`- scrapeTimeSec := float64(scrapeTime / time.Second)`
	`229`	`+ scrapeTimeSec := scrapeTime.Seconds()`
`230`	`230`	`s.duration.Record(context.Background(), scrapeTimeSec, metric.WithAttributeSet(s.attrs))`
`231`	`231`	`}()`
`232`	`232`
Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,6 @@ package queue`
`18`	`18`
`19`	`19`	`import (`
`20`	`20`	`"net/http"`
`21`		`- "time"`
`22`	`21`
`23`	`22`	`"go.opentelemetry.io/otel/metric"`
`24`	`23`	`"k8s.io/utils/clock"`
`@@ -103,7 +102,7 @@ func (h appRequestMetricsHandler) ServeHTTP(w http.ResponseWriter, r http.Requ`
`103`	`102`	`status = http.StatusInternalServerError`
`104`	`103`	`}`
`105`	`104`
`106`		`- elapsedTime := float64(latency / time.Second)`
	`105`	`+ elapsedTime := latency.Seconds()`
`107`	`106`	`h.duration.Record(r.Context(), elapsedTime,`
`108`	`107`	`metric.WithAttributes(semconv.HTTPResponseStatusCode(status)),`
`109`	`108`	`)`