From e2d71dc2503854f8da77069d439de6faf733c917 Mon Sep 17 00:00:00 2001 From: pedrofrxncx Date: Wed, 22 Jul 2026 05:01:50 -0300 Subject: [PATCH] fix(helm): add CPU limits to main API and nginx containers MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit API containers requested 750m CPU each with no ceiling, and nginx requested 500m with no CPU limit. This allows containers to burst unbounded and starve other workloads on the node. Add CPU limits aligned with best practices: 1 CPU per container (slightly above requests for headroom) matching the pattern already used for memory. The main pod now limits to ~2.5 CPU cores total (2× API + nginx) preventing resource hogging while accommodating LLM stream bursts. --- deploy/helm/studio/README.md | 4 +++- deploy/helm/studio/values.yaml | 2 ++ 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/deploy/helm/studio/README.md b/deploy/helm/studio/README.md index 9c1aaac47a..be6bb6b073 100644 --- a/deploy/helm/studio/README.md +++ b/deploy/helm/studio/README.md @@ -686,6 +686,7 @@ resources: memory: "1Gi" cpu: "750m" limits: + cpu: "1" memory: "1Gi" nginx: @@ -694,11 +695,12 @@ nginx: memory: "128Mi" cpu: "500m" limits: + cpu: "1" memory: "256Mi" ``` `resources` applies to each Bun API container. With two API containers plus -nginx, the main pod requests 2 CPU cores by default. +nginx, the main pod requests 2 CPU cores and limits to ~2.5 CPU cores total by default. ### Health Checks diff --git a/deploy/helm/studio/values.yaml b/deploy/helm/studio/values.yaml index 528b492ef2..8be8078740 100644 --- a/deploy/helm/studio/values.yaml +++ b/deploy/helm/studio/values.yaml @@ -41,6 +41,7 @@ resources: cpu: "750m" memory: "1Gi" limits: + cpu: "1" memory: "1Gi" database: @@ -100,6 +101,7 @@ nginx: cpu: "500m" memory: "128Mi" limits: + cpu: "1" memory: "256Mi" securityContext: runAsNonRoot: true