Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add s6 backend #17

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Add s6 backend #17

wants to merge 3 commits into from

Conversation

cnt0
Copy link

@cnt0 cnt0 commented Aug 17, 2024

s6 homepage: https://skarnet.org/software/s6/

I've been using this backend for about a week and can confirm that it works

@skarnet might want to review that as well

@skarnet
Copy link

skarnet commented Aug 17, 2024

Nitpick: if you run s6-svscan on $2, then $2 shouldn't be named a service directory, but a scan directory (which contains [links to] service directories), which is where you host a full supervision tree. A service directory is one that is managed by s6-supervise and contains the information for one service only.

Apart from that, I don't know the details of the turnstile API so I can't tell, but it looks okay.

As with anything involving user services, however, note that there's a question with logging: here, whatever user services write to their stdout and stderr will fall through to s6-svscan's stdout/stderr, i.e. whatever turnstile runs the script with. If these streams end up in a generic root log, be aware that a user can flood these logs, which is probably not a good idea.

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.

2 participants