Fix cortex_scheduler_pipeline_step_shift_origin metric: use outidx

PhilippMatthes · PhilippMatthes · commit 1869812b668b · 2025-05-09T14:03:51.000+02:00
diff --git a/internal/scheduler/monitor.go b/internal/scheduler/monitor.go
@@ -218,7 +218,7 @@ func (s *StepMonitor) Run(request api.Request) (map[string]float64, error) {
 			// Observe how far it was moved.
 			originalIdx := slices.Index(hostsIn, hostsOut[idx])
 			if s.stepReorderingsObserver != nil {
-				o := s.stepReorderingsObserver.WithLabelValues(stepName, hostsOut[idx])
+				o := s.stepReorderingsObserver.WithLabelValues(stepName, strconv.Itoa(idx))
 				o.Observe(float64(originalIdx))
 			}
 			slog.Info(

Original file line number	Diff line number	Diff line change
`@@ -218,7 +218,7 @@ func (s *StepMonitor) Run(request api.Request) (map[string]float64, error) {`
`218`	`218`	`// Observe how far it was moved.`
`219`	`219`	`originalIdx := slices.Index(hostsIn, hostsOut[idx])`
`220`	`220`	`if s.stepReorderingsObserver != nil {`
`221`		`- o := s.stepReorderingsObserver.WithLabelValues(stepName, hostsOut[idx])`
	`221`	`+ o := s.stepReorderingsObserver.WithLabelValues(stepName, strconv.Itoa(idx))`
`222`	`222`	`o.Observe(float64(originalIdx))`
`223`	`223`	`}`
`224`	`224`	`slog.Info(`