Skip to content

fix(runtime): retry transient sandbox startup failures#768

Merged
bearsyankees merged 2 commits into
mainfrom
devin/1784084629-sandbox-start-retry
Jul 15, 2026
Merged

fix(runtime): retry transient sandbox startup failures#768
bearsyankees merged 2 commits into
mainfrom
devin/1784084629-sandbox-start-retry

Conversation

@bearsyankees

Copy link
Copy Markdown
Collaborator

No description provided.

@greptile-apps

greptile-apps Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR adds retries for transient sandbox startup failures. The main changes are:

  • Configurable startup attempts with exponential backoff.
  • Transient transport error detection across exception chains.
  • Cleanup of failed sessions before each retry.
  • Immediate retry cancellation when cleanup fails.
  • Tests for retries, cleanup failures, and staged directory lifetime.

Confidence Score: 5/5

This looks safe to merge.

  • A failed teardown now stops the retry loop before another sandbox is created.
  • The new test confirms that the original startup error is raised and only one session is created when teardown fails.
  • No blocking issues were found in the updated code.

Important Files Changed

Filename Overview
strix/runtime/backends.py Adds sandbox startup retries and stops creating sessions when cleanup of a failed attempt does not complete.
tests/test_backend_retries.py Covers transient retries, non-transient failures, failed cleanup, repeated attempts, and staged directory cleanup.

Reviews (2): Last reviewed commit: "fix(runtime): fail closed when sandbox t..." | Re-trigger Greptile

Comment thread strix/runtime/backends.py Outdated
@devin-ai-integration

Copy link
Copy Markdown
Contributor

@greptile

@bearsyankees
bearsyankees merged commit 40f4e67 into main Jul 15, 2026
1 check passed
0xallam pushed a commit that referenced this pull request Jul 16, 2026
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.

1 participant