From 39f259286356204ebf0527026256e988b22278c0 Mon Sep 17 00:00:00 2001 From: Patrick Ohly Date: Thu, 17 Oct 2024 17:50:41 +0200 Subject: [PATCH] DRA CEL: avoid redundant cel.CostTracking It's already called by the base environment. --- staging/src/k8s.io/dynamic-resource-allocation/cel/compile.go | 1 - 1 file changed, 1 deletion(-) diff --git a/staging/src/k8s.io/dynamic-resource-allocation/cel/compile.go b/staging/src/k8s.io/dynamic-resource-allocation/cel/compile.go index 375b2717eef..53418314a47 100644 --- 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 {