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

Enable extra system calls in systemd service #1309

Merged
merged 3 commits into from
Jul 12, 2023
Merged

Conversation

runningnoodle
Copy link
Contributor

@runningnoodle runningnoodle commented May 7, 2023

Proposed fix for #1310.

Since v2.4.0, it looks like Listmonk is making system calls for changing resource limits, memory or scheduling parameters. This PR allows those system calls in the systemd service configuration. This allows Listmonk to run as a systemd service.

The downside of this change is that it weakens the sandbox around the Listmonk process when run as a systemd service. I don't fully understand how systemd works, but I don't expect any serious consequence.

@knadh
Copy link
Owner

knadh commented May 8, 2023

@runningnoodle this is curious. There have been no changes in v2.4.0 that should do this. No library that should also exhibit this behaviour. Would like to figure out this behaviour somehow before we merge this.

@knadh knadh added the enhancement New feature or request label May 8, 2023
@runningnoodle runningnoodle deleted the branch knadh:master May 9, 2023 21:30
@runningnoodle runningnoodle deleted the master branch May 9, 2023 21:30
@runningnoodle runningnoodle restored the master branch May 9, 2023 21:31
@runningnoodle
Copy link
Contributor Author

I closed by accident. The last two commits were also accidents. Sorry about the mess I'm creating.

@runningnoodle runningnoodle reopened this May 9, 2023
Copy link
Collaborator

@rhnvrm rhnvrm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Based on the investigation at #1310 and NixOS/nixpkgs#197443

I think it might be okay to remove @resources

@knadh knadh merged commit d69b766 into knadh:master Jul 12, 2023
@ozgurerdogan
Copy link

When trying to start service, I am getting;
init.go:798: error starting HTTP server: listen tcp 0.0.0.0:392: bind: permission denied

@runningnoodle
Copy link
Contributor Author

runningnoodle commented Sep 12, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants