Skip to content

Conversation

@zmarty
Copy link

@zmarty zmarty commented Jun 25, 2025

  • Add rlimits field to sandbox configuration and CLI arguments
  • Support resource limits in format RLIMIT_NAME=soft:hard (e.g., RLIMIT_NOFILE=16384:65536)
  • Update API documentation with rlimits examples
  • Add rlimits support to MCP server
  • Extend server handler to process rlimits configuration
  • Update all relevant CLI binaries and configuration builders

WARNING: I am not a Rust developer and this is vibe coded. It works, but it needs review and it needs to be closed out by someone more familiar with the codebase.

Fixes #284

- Add rlimits field to sandbox configuration and CLI arguments
- Support resource limits in format RLIMIT_NAME=soft:hard (e.g., RLIMIT_NOFILE=16384:65536)
- Update API documentation with rlimits examples
- Add rlimits support to MCP server
- Extend server handler to process rlimits configuration
- Update all relevant CLI binaries and configuration builders

WARNING: I am not a Rust developer and this is vibe coded. It works, but it needs review.
@appcypher
Copy link
Member

Hey @zmarty, thanks for the contribution.
I'm going to review it and give you feedback, it might just take a couple days to get to it.

@appcypher
Copy link
Member

I have reviewed this PR btw. I’m holding off on merging for now because there are other features and fixes I’ll be batching together, and some of the changes are related to this one.

@appcypher appcypher changed the title Feat: add rlimits (resource limits) support for sandboxes feat: add rlimits (resource limits) support for sandboxes Jul 20, 2025
@chemeris
Copy link

Hi, @appcypher, any chance this could be merged?
I'm also facing heavy "too many open files" issues. Basically, I can't run anything because I'm getting "too many open files" as soon as I try.

@appcypher
Copy link
Member

@chemeris I'm making ground up changes to microsandbox but I haven't gotten to it yet.
This pr handles some part of it but it is not complete. Until then, I can't merge this pr.
Also can't make a commitment because I just don't know when I will have time to fix these issues.

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.

Too many open files - 1024 and 4096 soft and hard open file limit

3 participants