service: the systemd unit needs the same descriptor ceiling - #79
Merged
Merged
Conversation
#78 gave the launchd plist a `NumberOfFiles` because the Mac daemon had died of `Too many open files`. The systemd unit has the same omission and I only fixed one side, which is how a defect keeps a door open. `LimitNOFILE` appeared nowhere in `render_systemd_user_unit`, so a Linux agent inherits whatever the user manager defaults to. MEASURED ON BOTH PEERS RATHER THAN ASSUMED. Linux is not in the same trouble the Mac was, and that is worth stating plainly: hetz soft 1024, hard 524288, 53 descriptors in use droppy soft 1024, hard 524288, 27 descriptors in use 1024 is four times launchd's default and neither peer has come close to it. So this is not a fire. It is the same inherited-number problem, and 1024 is still a ceiling nobody picked for a daemon that holds a QUIC endpoint, a connection per peer, a control socket and a dial socket per tunnel. 8192 matches the plist, and it is not a measurement there either. Both platforms now state the ceiling instead of inheriting one. Agent: Silber.fabric
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
1 participant
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.
#78 gave the launchd plist a
NumberOfFilesbecause the Mac daemon had died ofToo many open files. The systemd unit has the same omission and I only fixed one side — which is how a defect keeps a door open.LimitNOFILEappeared nowhere inrender_systemd_user_unit, so a Linux agent inherits whatever the user manager defaults to.Measured on both peers rather than assumed
Linux is not in the same trouble the Mac was, and that is worth stating plainly:
1024 is four times launchd's default and neither peer has come close to it. So this is not a fire. It is the same inherited-number problem: 1024 is still a ceiling nobody picked for a daemon that holds a QUIC endpoint, a connection per peer, a control socket, and a dial socket per tunnel.
8192matches the plist, and it is not a measurement there either. Both platforms now state the ceiling instead of inheriting one.Agent: Silber.fabric