Skip to content

Comments

Make recursive-nix work even when not privileged#5364

Merged
edolstra merged 1 commit intoNixOS:masterfrom
L-as:recursive-nix-fix
Jan 10, 2022
Merged

Make recursive-nix work even when not privileged#5364
edolstra merged 1 commit intoNixOS:masterfrom
L-as:recursive-nix-fix

Conversation

@L-as
Copy link
Member

@L-as L-as commented Oct 8, 2021

Before this, setns would fail when switching to the mount namespace,
since we did not have the privileges to do so when not root.

Closes #5360

@L-as L-as force-pushed the recursive-nix-fix branch from a7ac388 to 4e0f851 Compare October 8, 2021 23:59
@L-as
Copy link
Member Author

L-as commented Oct 9, 2021

Note: The sudo part of the issue isn't resolved, but it's a nicher use case.

@Ericson2314 Ericson2314 mentioned this pull request Oct 12, 2021
4 tasks
@edolstra edolstra self-assigned this Oct 13, 2021
@L-as
Copy link
Member Author

L-as commented Nov 21, 2021

bump

Before this, `setns` would fail when switching to the mount namespace,
since we did not have the privileges to do so when not root.

Closes NixOS#5360
@L-as L-as force-pushed the recursive-nix-fix branch from 4e0f851 to 8388d2c Compare November 25, 2021 11:15
@Ericson2314
Copy link
Member

@edolstra any questions with this one? I thought it was a pretty straight-forward fix.

@L-as
Copy link
Member Author

L-as commented Jan 9, 2022

I think this patch is still needed.

@edolstra edolstra merged commit df11e75 into NixOS:master Jan 10, 2022
@edolstra
Copy link
Member

Thanks!

@L-as L-as deleted the recursive-nix-fix branch October 22, 2022 15:11
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.

Recursive Nix is broken with --store

4 participants