From 31677520f38fc07425f4fdca6c41b5debcf29ac3 Mon Sep 17 00:00:00 2001 From: cici37 Date: Tue, 8 Mar 2022 12:03:25 -0800 Subject: [PATCH] Update cost budget --- .../pkg/apiserver/schema/cel/compilation.go | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/staging/src/k8s.io/apiextensions-apiserver/pkg/apiserver/schema/cel/compilation.go b/staging/src/k8s.io/apiextensions-apiserver/pkg/apiserver/schema/cel/compilation.go index 5ca78b043a6..a3804387f20 100644 --- a/staging/src/k8s.io/apiextensions-apiserver/pkg/apiserver/schema/cel/compilation.go +++ b/staging/src/k8s.io/apiextensions-apiserver/pkg/apiserver/schema/cel/compilation.go @@ -18,7 +18,6 @@ package cel import ( "fmt" - "math" "strings" "time" @@ -43,12 +42,12 @@ const ( OldScopedVarName = "oldSelf" // PerCallLimit specify the actual cost limit per CEL validation call - //TODO: pick the number for PerCallLimit - PerCallLimit = uint64(math.MaxInt64) + // current PerCallLimit gives roughly 0.1 second for each expression validation call + PerCallLimit = 2000000 // RuntimeCELCostBudget is the overall cost budget for runtime CEL validation cost per CustomResource - //TODO: pick the RuntimeCELCostBudget - RuntimeCELCostBudget = math.MaxInt64 + // current RuntimeCELCostBudget gives roughly 1 seconds for CR validation + RuntimeCELCostBudget = 20000000 ) // CompilationResult represents the cel compilation result for one rule