Skip to content

fix(agents): say which quota stopped a session from starting - #133

Merged
tee8z merged 1 commit into
mainfrom
fix/sandbox-quota-error
Nov 5, 2026
Merged

tee8z merged 1 commit into
mainfrom
fix/sandbox-quota-error

Conversation

@tee8z

@tee8z tee8z commented Nov 5, 2026

Copy link
Copy Markdown
Collaborator

Problem

A direct message to Heimdall today failed with:

Could not finish: Sandbox controller has not created its Pod; inspect the Sandbox and retry

The real cause was in the Sandbox's Ready condition all along:

pods "buzz-agent-a53ce7308b33-571514d7bbb4" is forbidden: exceeded quota: primary-sessions, requested: pods=1, used: pods=5, limited: pods=5

Fix

On the deploy deadline, bind re-reads the Sandbox and translates its Ready condition:

  • primary-sessions → "All of your agent sessions are in use. End an idle session, or wait for one to finish, then retry"
  • any other quota → names it, for example "Your agent namespace quota agent-capacity is full…"
  • anything else → the existing message

Validation

  • New a_pod_the_quota_refused_names_the_full_limit uses the controller's exact staging message, plus another quota, an unrelated failure, and a Ready=True condition.
  • buzz-backend-kubernetes: 225 pass; clippy -D warnings is clean.

Ships in the Sandbox manager image.

🤖 Generated with Claude Code

When a developer's five primary sessions were in use, a new session's Pod was
refused by the namespace quota, and the app reported "Sandbox controller has
not created its Pod; inspect the Sandbox and retry". The controller already
records the cause in the Sandbox's Ready condition.

On the deploy deadline, read that condition: a primary-sessions refusal now
says all agent sessions are in use, and any other quota is named. Other
causes keep the existing message.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Tee8z <tee8z@protonmail.com>
@coderabbitai

coderabbitai Bot commented Nov 5, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

  • Run on-demand review

This review includes 2 billable files and costs up to $0.50.

  • Ask an admin to make reviews automatic

Open in CodeRabbit

Reviews can continue after your included limit without a manual trigger. An admin must approve usage-based billing.

Or wait 34 minutes for your next included review.

Check out review usage here.

View limit details

Limit details: You’ve used all 2 included reviews currently available. Your 51 included PR review attempts over the past 7 days set your current allowance at 2 reviews per hour.

Learn how review limits work.

Review configuration:

Run configuration

Commits

Reviewing files that changed from the base of the PR and between dbe955d and 3126ee2.

Files selected for processing: 2

  • crates/buzz-backend-kubernetes/src/sandbox.rs
  • crates/buzz-backend-kubernetes/src/sandbox_tests.rs
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

@tee8z
tee8z marked this pull request as ready for review November 5, 2026 17:07
@tee8z
tee8z merged commit da55379 into main Nov 5, 2026
173 of 176 checks passed
@tee8z
tee8z deleted the fix/sandbox-quota-error branch November 5, 2026 17:15
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