From 40e723d58ce7300f24e657bcf45843f6560e2ff7 Mon Sep 17 00:00:00 2001 From: Bolek Kulbabinski <1416262+bolekk@users.noreply.github.com> Date: Fri, 2 Jan 2026 10:26:48 -0800 Subject: [PATCH] [CRE] Adjust default limits 1. Bump capability concurrency up to 30. Relaying on concurrency limit can cause problems with OCR when different nodes throttle different requests. Let's rely on per-capability limits instead. 2. Reduce consensus call limit to something sane. 3. Reduce trigger event queus size. Usually queueing a lot of events makes them cross the expiration threshold anyway. --- pkg/settings/cresettings/defaults.json | 6 +++--- pkg/settings/cresettings/defaults.toml | 6 +++--- pkg/settings/cresettings/settings.go | 6 +++--- 3 files changed, 9 insertions(+), 9 deletions(-) diff --git a/pkg/settings/cresettings/defaults.json b/pkg/settings/cresettings/defaults.json index da605124b..62bbfc34c 100644 --- a/pkg/settings/cresettings/defaults.json +++ b/pkg/settings/cresettings/defaults.json @@ -19,9 +19,9 @@ "TriggerRegistrationsTimeout": "10s", "TriggerSubscriptionTimeout": "15s", "TriggerSubscriptionLimit": "10", - "TriggerEventQueueLimit": "1000", + "TriggerEventQueueLimit": "50", "TriggerEventQueueTimeout": "10m0s", - "CapabilityConcurrencyLimit": "3", + "CapabilityConcurrencyLimit": "30", "CapabilityCallTimeout": "3m0s", "SecretsConcurrencyLimit": "5", "ExecutionConcurrencyLimit": "5", @@ -67,7 +67,7 @@ }, "Consensus": { "ObservationSizeLimit": "100kb", - "CallLimit": "2000" + "CallLimit": "20" }, "HTTPAction": { "CallLimit": "5", diff --git a/pkg/settings/cresettings/defaults.toml b/pkg/settings/cresettings/defaults.toml index 195f3977d..013475840 100644 --- a/pkg/settings/cresettings/defaults.toml +++ b/pkg/settings/cresettings/defaults.toml @@ -19,9 +19,9 @@ VaultSecretsLimit = '100' TriggerRegistrationsTimeout = '10s' TriggerSubscriptionTimeout = '15s' TriggerSubscriptionLimit = '10' -TriggerEventQueueLimit = '1000' +TriggerEventQueueLimit = '50' TriggerEventQueueTimeout = '10m0s' -CapabilityConcurrencyLimit = '3' +CapabilityConcurrencyLimit = '30' CapabilityCallTimeout = '3m0s' SecretsConcurrencyLimit = '5' ExecutionConcurrencyLimit = '5' @@ -68,7 +68,7 @@ PayloadSizeLimit = '5kb' [PerWorkflow.Consensus] ObservationSizeLimit = '100kb' -CallLimit = '2000' +CallLimit = '20' [PerWorkflow.HTTPAction] CallLimit = '5' diff --git a/pkg/settings/cresettings/settings.go b/pkg/settings/cresettings/settings.go index 6a7c4978c..2fd096419 100644 --- a/pkg/settings/cresettings/settings.go +++ b/pkg/settings/cresettings/settings.go @@ -81,11 +81,11 @@ var Default = Schema{ }, PerWorkflow: Workflows{ TriggerRegistrationsTimeout: Duration(10 * time.Second), - TriggerEventQueueLimit: Int(1_000), + TriggerEventQueueLimit: Int(50), TriggerEventQueueTimeout: Duration(10 * time.Minute), TriggerSubscriptionTimeout: Duration(15 * time.Second), TriggerSubscriptionLimit: Int(10), - CapabilityConcurrencyLimit: Int(3), + CapabilityConcurrencyLimit: Int(30), // we should rely on per-capability execution limits instead of concurrency limit CapabilityCallTimeout: Duration(3 * time.Minute), SecretsConcurrencyLimit: Int(5), ExecutionConcurrencyLimit: Int(5), @@ -132,7 +132,7 @@ var Default = Schema{ }, Consensus: consensus{ ObservationSizeLimit: Size(100 * config.KByte), - CallLimit: Int(2000), + CallLimit: Int(20), }, HTTPAction: httpAction{ CallLimit: Int(5),