Merge pull request #8577 from abdelrahman882/cb-fix-main

k8s-ci-robot · web-flow · commit 94637a20b3dc · 2025-09-26T05:48:21.000-07:00
Fix capacity buffers injector order in pod list processor
diff --git a/cluster-autoscaler/main.go b/cluster-autoscaler/main.go
@@ -193,7 +193,7 @@ func buildAutoscaler(context ctx.Context, debuggingSnapshotter debuggingsnapshot
 		}
 		if capacitybufferClientError == nil && capacitybufferClient != nil {
 			bufferPodInjector := cbprocessor.NewCapacityBufferPodListProcessor(capacitybufferClient, []string{common.ActiveProvisioningStrategy})
-			podListProcessor.AddProcessor(bufferPodInjector)
+			podListProcessor = pods.NewCombinedPodListProcessor([]pods.PodListProcessor{bufferPodInjector, podListProcessor})
 			opts.Processors.ScaleUpStatusProcessor = status.NewCombinedScaleUpStatusProcessor([]status.ScaleUpStatusProcessor{cbprocessor.NewFakePodsScaleUpStatusProcessor(), opts.Processors.ScaleUpStatusProcessor})
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -193,7 +193,7 @@ func buildAutoscaler(context ctx.Context, debuggingSnapshotter debuggingsnapshot`
`193`	`193`	`}`
`194`	`194`	`if capacitybufferClientError == nil && capacitybufferClient != nil {`
`195`	`195`	`bufferPodInjector := cbprocessor.NewCapacityBufferPodListProcessor(capacitybufferClient, []string{common.ActiveProvisioningStrategy})`
`196`		`- podListProcessor.AddProcessor(bufferPodInjector)`
	`196`	`+ podListProcessor = pods.NewCombinedPodListProcessor([]pods.PodListProcessor{bufferPodInjector, podListProcessor})`
`197`	`197`	`opts.Processors.ScaleUpStatusProcessor = status.NewCombinedScaleUpStatusProcessor([]status.ScaleUpStatusProcessor{cbprocessor.NewFakePodsScaleUpStatusProcessor(), opts.Processors.ScaleUpStatusProcessor})`
`198`	`198`	`}`
`199`	`199`	`}`