Repository navigation
Add exec, config, resource limits and CLI conveniences - #8
Merged
Merged
Conversation
- The project folder is optional: `systemd-compose up` works in the current folder. `systemd-compose <folder> <command>` still works. - -h/--help/help print a full usage summary and exit 0. - logs: follows by default; extra options go straight to journalctl (`logs -n 200`, `logs --since today -f`). - down keeps the venv; `down --volumes` (or -v) also deletes it. Unknown options are rejected before anything is removed. - exec: run a one-off command like the service runs the app (venv first on PATH, .env variables, inside APP_DIR), e.g. `exec python manage.py migrate`. Returns the command's exit code. - config: print the service file `run` would write and say whether the installed one is out of date. Changes nothing. - MEMORY_MAX / CPU_QUOTA in config.env set MemoryMax=/CPUQuota=. Values are validated (systemd silently ignores invalid ones) and written without %-escaping, since these settings take "%" literally. `run` warns when the kernel can't enforce them; status shows the limit. - Commands that take no options now reject extra arguments. - 19 new tests (68 total), including systemd-analyze checks that the limits parse. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KdsNJV9e2ktZ9gF5kmczEu
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR 2 of 3 in this round, stacked on the tests PR (#7). Merge that one first; GitHub then retargets this PR to
main.Changes
.systemd-compose upworks inside a project folder.systemd-compose <folder> <command>still works. A folder whose name matches a command needs a path:./logs up.-h/--helplogsoptionslogs -n 200,logs --since today -f.downkeeps the venvdownstops the app and removes the service.down --volumes(-v) also deletes the venv, like Docker. Unknown options are rejected before anything is removed.execsystemd-compose exec python manage.py migrateruns a command the way the service runs the app: venv first onPATH,.envvariables, insideAPP_DIR. It returns the command's exit code.configrunwould write and says whether the installed one is out of date. Changes nothing.MEMORY_MAX="300M"/CPU_QUOTA="50%"inconfig.envbecomeMemoryMax=/CPUQuota=.statusshows the memory limit.stop --allfails instead of being ignored.Behaviour change:
downno longer deletes the venv. Usedown --volumesfor the old behaviour. The README is updated.Resource limit details
While testing I found that
MemoryMax=andCPUQuota=take%literally. The%→%%escaping used for the other settings would have producedCPUQuota=50%%, which systemd silently ignores. Two things handle this:systemd-analyze verifyon the generated file and assertsCPUQuotaPerSecSec: 500ms.systemd also silently ignores invalid limits, which is another reason for validating them up front.
runwarns when the user manager doesn't have the memory/cpu cgroup controllers, because the limit wouldn't be enforced there.Testing
%escaping back on the limit values makes 2 tests fail.🤖 Generated with Claude Code
https://claude.ai/code/session_01KdsNJV9e2ktZ9gF5kmczEu
Generated by Claude Code