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

nixos/doc: document that services defined with systemd.users aren't restarted by nixos-rebuild #66561

Merged
merged 1 commit into from Aug 13, 2019

Conversation

Ma27
Copy link
Member

@Ma27 Ma27 commented Aug 13, 2019

Motivation for this change

Issues like #66187 prove that most newcomers expect nixos-rebuild to properly start/restart/reload user services which isn't the case at the moment. This patch mentions that in the nixos-rebuild manpage and in an additional warning block in the NixOS manual.

cc @l0b0

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.

@Ma27 Ma27 force-pushed the document-user-services-on-rebuild branch from 4baf514 to 551230b Compare August 13, 2019 14:26
@WilliButz
Copy link
Member

Thank you for documenting this 👍

@GrahamcOfBorg build manual manpages

@WilliButz WilliButz merged commit 7a29431 into NixOS:master Aug 13, 2019
@Ma27 Ma27 deleted the document-user-services-on-rebuild branch August 13, 2019 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants