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

systemd,launchd match nixos open files limit #6645

Merged
merged 1 commit into from
Dec 9, 2022

Conversation

Artturin
Copy link
Member

it was bumped in NixOS/nixpkgs#176558

@edolstra
Copy link
Member

1 million file descriptors sounds rather excessive. If Nix really needs that many, it's clearly a bug.

@Artturin
Copy link
Member Author

I just chose some random value i found in a stackoverflow post and didn't see a reason to make it lower

@roberth
Copy link
Member

roberth commented Jun 22, 2022

1 million file descriptors sounds rather excessive. If Nix really needs that many, it's clearly a bug.

Nix only needs like 3.
The question is what does a user's derivation need? A million is only excessive until it is not, but probably not enough to cause issues on the system.

clearly a bug

Or a highly concurrent test suite that's heavy on fds.

@lovesegfault
Copy link
Member

@edolstra Can you take another look at this?

I can't build many derivations on macOS without manually patching the plist like this; otherwise it complains about too many open files.

@mstone
Copy link

mstone commented Dec 3, 2022

I also recently ran into (another) derivation that needed a higher limit than was provided by default [ IIRC, testing out a mass-rebuild-causing change that had been merged into staging ] and so would appreciate a fix along these lines ~regardless of what number is ultimately chosen.

@github-actions
Copy link

github-actions bot commented Dec 9, 2022

Successfully created backport PR #7432 for 2.11-maintenance.

@Artturin
Copy link
Member Author

Artturin commented Dec 9, 2022

Thanks thufschmitt & edolstra

@Artturin Artturin deleted the limitinfinity branch December 10, 2022 21:51
@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/2022-12-09-nix-team-meeting-minutes-15/23951/1

@github-actions
Copy link

Backport failed for 2.11-maintenance, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally.

git fetch origin 2.11-maintenance
git worktree add -d .worktree/backport-6645-to-2.11-maintenance origin/2.11-maintenance
cd .worktree/backport-6645-to-2.11-maintenance
git checkout -b backport-6645-to-2.11-maintenance
ancref=$(git merge-base 499e99d099ec513478a2d3120b2af3a16d9ae49d 2320a2f93e0aa61e8d45dcfb3fb536361a1fcb6f)
git cherry-pick -x $ancref..2320a2f93e0aa61e8d45dcfb3fb536361a1fcb6f

@github-actions
Copy link

Successfully created backport PR #7618 for 2.12-maintenance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

7 participants