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

checks: disable nixGarbageCollector for multi-user #822

Closed
wants to merge 1 commit into from

Conversation

zowoq
Copy link
Contributor

@zowoq zowoq commented Dec 4, 2023

Currently if you run activation as root you need to set nix.gc.user = "root"; or nix.gc.user = "";.

nixGarbageCollector = ''
if test -O /nix/store; then
echo "�[1;31merror: A single-user install can't run gc as root, aborting activation�[0m" >&2
echo "Configure the garbage collector to run as the current user:" >&2
echo >&2
echo " nix.gc.user = \"$USER\";" >&2
echo >&2
exit 2
fi

@zowoq
Copy link
Contributor Author

zowoq commented May 18, 2024

Rebased to fix merge conflict.

@emilazy
Copy link
Collaborator

emilazy commented Jun 13, 2024

I believe checks currently run as part of user activation; I’m not sure we support running that as root. I would like to move all activation to be run as root, but as it stands I’m worried that this might just mask other issues with that configuration.

@zowoq
Copy link
Contributor Author

zowoq commented Jul 27, 2024

I’m not sure we support running that as root.

No, not officially.

move all activation to be run as root

I'll close the PR as this is the better option.

@zowoq zowoq closed this Jul 27, 2024
@zowoq zowoq deleted the gc-user branch July 27, 2024 03:43
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