Skip to content

Disable read timeout for exec#179

Open
mendral-app[bot] wants to merge 2 commits into
mainfrom
mendral/disable-exec-read-timeout
Open

Disable read timeout for exec#179
mendral-app[bot] wants to merge 2 commits into
mainfrom
mendral/disable-exec-read-timeout

Conversation

@mendral-app

@mendral-app mendral-app Bot commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Pass timeout=None to the client.post("/process", ...) call in exec() to prevent ReadTimeout on long-running sandbox processes, matching the pattern used for the streaming endpoint (PR Disable read timeout for streaming #169).

Process execution time is inherently unbounded — any command could take longer than 5 minutes — so no fixed timeout is appropriate for this request.

Insight: Sandbox exec ReadTimeout on long-running processes


Note

Created by Mendral. Tag @mendral-app with feedback or questions.

@mendral-app mendral-app Bot requested a review from a team June 26, 2026 07:32
@mendral-app mendral-app Bot marked this pull request as ready for review June 26, 2026 07:46
@cploujoux

Copy link
Copy Markdown
Contributor

@mendral-app how about the sync method ?

@mendral-app

mendral-app Bot commented Jun 26, 2026

Copy link
Copy Markdown
Contributor Author

Good catch! I missed the sync method. I've pushed a fix — timeout=None is now also applied to the sync exec() in src/blaxel/core/sandbox/sync/process.py:207.

The integration test failure is unrelated (503 from httpbin.org and network timeouts in proxy tests).

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