Skip to content

Commit

Permalink
{ids,checks}: update for new builder UID/GID values
Browse files Browse the repository at this point in the history
  • Loading branch information
emilazy committed Sep 10, 2024
1 parent f0acf79 commit 9f273e2
Show file tree
Hide file tree
Showing 7 changed files with 82 additions and 8 deletions.
14 changes: 14 additions & 0 deletions CHANGELOG
Original file line number Diff line number Diff line change
@@ -1,3 +1,17 @@
2024-09-10
- The default Nix build user group ID is now set to 350 when
`system.stateVersion` ≥ 5, to reflect the default for new Nix
installations. This only affects installations that enable
`nix.configureBuildUsers`, and any divergence will be detected on
system activation. To use `nix.configureBuildUsers` with a higher
`system.stateVersion` on installations using the old group ID, set:

ids.gids.nixbld = 30000;

We do not recommend trying to change the group ID with macOS user
management tools without a complete uninstallation and reinstallation
of Nix.

2024-06-15
- SECURITY NOTICE: The previous implementation of the
`users.users.<name>.openssh.authorizedKeys.*` options would not delete
Expand Down
2 changes: 1 addition & 1 deletion modules/examples/flake/flake.nix
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@

# Used for backwards compatibility, please read the changelog before changing.
# $ darwin-rebuild changelog
system.stateVersion = 4;
system.stateVersion = 5;

# The platform the configuration will be used on.
nixpkgs.hostPlatform = "x86_64-darwin";
Expand Down
2 changes: 1 addition & 1 deletion modules/examples/ofborg.nix
Original file line number Diff line number Diff line change
Expand Up @@ -25,5 +25,5 @@ with lib;

# Used for backwards compatibility, please read the changelog before changing.
# $ darwin-rebuild changelog
system.stateVersion = 4;
system.stateVersion = 5;
}
2 changes: 1 addition & 1 deletion modules/examples/simple.nix
Original file line number Diff line number Diff line change
Expand Up @@ -21,5 +21,5 @@

# Used for backwards compatibility, please read the changelog before changing.
# $ darwin-rebuild changelog
system.stateVersion = 4;
system.stateVersion = 5;
}
7 changes: 3 additions & 4 deletions modules/misc/ids.nix
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
# to change uids/gids on service start, in example a service with a lot of
# files.

{ lib, ... }:
{ lib, config, ... }:

let
inherit (lib) types;
Expand All @@ -34,15 +34,14 @@ in

};


config = {

ids.uids = {
nixbld = 300;
nixbld = lib.mkDefault 350;
};

ids.gids = {
nixbld = 30000;
nixbld = lib.mkDefault (if config.system.stateVersion < 5 then 30000 else 350);
};

};
Expand Down
61 changes: 61 additions & 0 deletions modules/system/checks.nix
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,41 @@ let
echo " system.checks.verifyBuildUsers = false;" >&2
echo >&2
exit 2
fi
'';

preSequoiaBuildUsers = ''
${lib.optionalString config.nix.configureBuildUsers ''
# Don’t complain when we’re about to migrate old‐style build users…
if ! dscl . -list /Users | grep -q '^nixbld'; then
''}
firstBuildUserID=$(dscl . -read /Users/_nixbld1 UniqueID | awk '{print $2}')
if [[ $firstBuildUserID != ${toString (config.ids.uids.nixbld + 1)} ]]; then
printf >&2 '\e[1;31merror: Build users have unexpected UIDs, aborting activation\e[0m\n'
printf >&2 'The default Nix build user ID range has been adjusted for\n'
printf >&2 'compatibility with macOS Sequoia 15. Your _nixbld1 user currently has\n'
printf >&2 'UID %d (usually 301) rather than the new default of 351.\n' \
"$firstBuildUserID"
printf >&2 '\n'
printf >&2 'You can automatically migrate your users using the following script\n'
printf >&2 'from the Nix repository:\n'
printf >&2 '\n'
printf >&2 ' https://github.com/NixOS/nix/raw/master/scripts/sequoia-nixbld-user-migration.sh\n'
printf >&2 '\n'
printf >&2 'This should work even if you installed Nix with the Determinate\n'
printf >&2 'Systems installer or are using Lix. If you are comfortable using the\n'
printf >&2 'script without review, you can run:\n'
printf >&2 '\n'
printf >&2 " curl --proto '=https' --tlsv1.2 -sSf -L https://github.com/NixOS/nix/raw/master/scripts/sequoia-nixbld-user-migration.sh | bash -\n"
printf >&2 '\n'
printf >&2 'If you have no intention of upgrading to macOS Sequoia 15, you can\n'
printf >&2 'disable this check by setting:\n'
printf >&2 '\n'
printf >&2 ' ids.uids.nixbld = %d;\n' "$((firstBuildUserID - 1))"
printf >&2 '\n'
exit 2
fi
${lib.optionalString config.nix.configureBuildUsers "fi"}
'';

buildUsers = ''
Expand All @@ -75,6 +109,31 @@ let
fi
'';

buildGroupID = ''
buildGroupID=$(dscl . -read /Groups/nixbld PrimaryGroupID | awk '{print $2}')
expectedBuildGroupID=${toString config.ids.gids.nixbld}
if [[ $buildGroupID != ${toString config.ids.gids.nixbld}) ]]; then
printf >&2 '\e[1;31merror: Build user group has mismatching GID, aborting activation\e[0m\n'
printf >&2 'The default Nix build user group ID was changed from 30000 to 350.\n'
printf >&2 'You are currently managing Nix build users with nix-darwin, but your\n'
printf >&2 'nixbld group has ID %d, whereas we expected %d.\n' \
"$buildGroupID" "$expectedBuildGroupID"
printf >&2 '\n'
printf >&2 'Possible causes include setting up a new Nix installation with an\n'
printf >&2 'existing nix-darwin configuration, or increasing your\n'
printf >&2 '`system.stateVersion` setting.'
printf >&2 '\n'
printf >&2 'You can either set the group ID to match the actual value:\n'
printf >&2 '\n'
printf >&2 ' ids.gids.nixbld = %d;\n' "$buildGroupID"
printf >&2 '\n'
printf >&2 'or disable `nix.configureBuildUsers`. We do not recommend trying to\n'
printf >&2 'change the group ID with macOS user management tools without a\n'
printf >&2 'complete uninstallation and reinstallation of Nix.\n'
exit 2
fi
'';

singleUser = ''
if grep -q 'build-users-group =' /etc/nix/nix.conf; then
echo "error: The daemon is not enabled but this is a multi-user install, aborting activation" >&2
Expand Down Expand Up @@ -266,7 +325,9 @@ in
darwinChanges
runLink
(mkIf (cfg.verifyBuildUsers && !config.nix.configureBuildUsers) oldBuildUsers)
(mkIf cfg.verifyBuildUsers preSequoiaBuildUsers)
(mkIf cfg.verifyBuildUsers buildUsers)
(mkIf config.nix.configureBuildUsers buildGroupID)
(mkIf (!config.nix.useDaemon) singleUser)
nixStore
(mkIf (config.nix.gc.automatic && config.nix.gc.user == null) nixGarbageCollector)
Expand Down
2 changes: 1 addition & 1 deletion modules/system/version.nix
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ in
options = {
system.stateVersion = mkOption {
type = types.int;
default = 4;
default = 5;
description = ''
Every once in a while, a new NixOS release may change
configuration defaults in a way incompatible with stateful
Expand Down

0 comments on commit 9f273e2

Please sign in to comment.