Added systemd oom handling and tests#227
Conversation
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
jakubno
left a comment
There was a problem hiding this comment.
We should probably set StartLimitBurst
Systemd's default rate limit is 5 starts in 10 seconds. A crash loop exhausts this and puts the unit in failed state permanently with no self-recovery.
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
jakubno
left a comment
There was a problem hiding this comment.
one nit JUPYTER_CONFIG_PATH can be defined only for Docker in template.py

Note
Medium Risk
Changes the template’s startup/process supervision to rely on
systemdfor restarting Jupyter and theuvicornserver, which can affect sandbox boot and recovery behavior. Adds kill/restart integration tests that may be timing-sensitive in CI.Overview
Switches the non-docker template to run Jupyter and the code-interpreter API as
systemdservices withRestart=on-failure, service dependency wiring, and a shared Jupyter readiness pre-check via the newjupyter-healthcheck.sh.Updates
template.pyto install the new unit files and start viasystemctl(while keeping the docker path usingstart-up.sh), and adds JS + Python (sync/async) tests thatkill -9Jupyter oruvicornand assert the/healthendpoint and code execution recover after restart.Written by Cursor Bugbot for commit dc0d06d. This will update automatically on new commits. Configure here.