Skip to content

fix(cloud-agent-next): increase tool cgroup reserve to 2GB - #5084

Merged
eshurakov merged 1 commit into
mainfrom
session/agent_47ea6473-1293-45e9-98b8-80df9de0cb3d
Aug 6, 2026
Merged

eshurakov merged 1 commit into
mainfrom
session/agent_47ea6473-1293-45e9-98b8-80df9de0cb3d

Conversation

@eshurakov

Copy link
Copy Markdown
Contributor

Summary

  • Bump TOOL_CGROUP_RESERVE_MB from 1024 to 2048 in cloud-agent-next wrangler config
  • Leaves more memory reserved outside the tool cgroup under enforce mode

Test plan

  • Confirm wrangler vars show TOOL_CGROUP_RESERVE_MB=2048 after deploy
  • Spot-check a session with tool cgroup enabled still starts and enforces limits

Raise TOOL_CGROUP_RESERVE_MB from 1024 to 2048 so more memory is reserved for the server/runtime slice under tool cgroup enforcement.

Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
@kilo-code-bot

kilo-code-bot Bot commented Aug 6, 2026 •

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

One-line config bump of TOOL_CGROUP_RESERVE_MB from 1024 to 2048 in cloud-agent-next. Verified against the wrapper's cgroup logic: in enforce mode the tool slice cap is MemTotal − reserve, so this reserves more memory for the server/runtime slice as intended. Even the smallest sandbox instance type (4096 MiB) keeps a ~2 GiB tool cap, safely above the 1 GiB MIN_CAP_BYTES floor, so enforcement stays active everywhere. High confidence.

Files Reviewed (1 files)
  • services/cloud-agent-next/wrangler.jsonc

Reviewed by kimi-k3 · Input: 53.7K · Output: 7.1K · Cached: 309.2K

Review guidance: REVIEW.md from base branch main

@eshurakov
eshurakov merged commit 0e24f1d into main Aug 6, 2026
16 checks passed
@eshurakov
eshurakov deleted the session/agent_47ea6473-1293-45e9-98b8-80df9de0cb3d branch August 6, 2026 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants