DRA CEL: avoid redundant cel.CostTracking

pohly · pohly · commit 39f259286356 · 2024-10-23T21:24:45.000+02:00
It's already called by the base environment.
diff --git a/staging/src/k8s.io/dynamic-resource-allocation/cel/compile.go b/staging/src/k8s.io/dynamic-resource-allocation/cel/compile.go
@@ -136,7 +136,6 @@ func (c compiler) CompileCELExpression(expression string, envType environment.Ty
 		// This call here should override that. In practice it shouldn't
 		// matter because the limits are the same.
 		cel.CostLimit(resourceapi.CELSelectorExpressionMaxCost),
-		cel.CostTracking(estimator),
 		cel.InterruptCheckFrequency(celconfig.CheckFrequency),
 	)
 	if err != nil {

Original file line number	Diff line number	Diff line change
`@@ -136,7 +136,6 @@ func (c compiler) CompileCELExpression(expression string, envType environment.Ty`
`136`	`136`	`// This call here should override that. In practice it shouldn't`
`137`	`137`	`// matter because the limits are the same.`
`138`	`138`	`cel.CostLimit(resourceapi.CELSelectorExpressionMaxCost),`
`139`		`- cel.CostTracking(estimator),`
`140`	`139`	`cel.InterruptCheckFrequency(celconfig.CheckFrequency),`
`141`	`140`	`)`
`142`	`141`	`if err != nil {`