Skip to content

Use prlimit#1462

Closed
josephjclark wants to merge 5 commits into
mainfrom
rlimit
Closed

Use prlimit#1462
josephjclark wants to merge 5 commits into
mainfrom
rlimit

Conversation

@josephjclark

Copy link
Copy Markdown
Collaborator

Short Description

Implementation Details

Spun out of #1370

AI Usage

Please disclose whether you've used AI anywhere in this PR (it's cool, we just
want to know!):

  • I have used Claude Code
  • I have used another model
  • I have not used AI

You can read more details in our
Responsible AI Policy

@github-project-automation github-project-automation Bot moved this to New Issues in Core Jun 23, 2026
@josephjclark josephjclark changed the title Use Rlimit Use prlimit Jun 23, 2026
@josephjclark

Copy link
Copy Markdown
Collaborator Author

Fantastic. Wasted a whole day on getting this ready and it turns out it's just not going to work.

This won't help us. The address space limit we set has to be siginificantly larger than the space required by a run. I'll post more detai in #1370.

tl;dr: If I set AS to 1gb using rlimit, thenthe node process won't even run. We need to let it allocate way more memory (the original PR uses 10x the limit). That means kubernetes will kill the pod way before the rlimit kicks in.

But turns out this completely won't work
@github-project-automation github-project-automation Bot moved this from New Issues to Done in Core Jun 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants