-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Support user accounts managed by external identity management systems #6640
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
base: master
Are you sure you want to change the base?
Conversation
23cf1dc to
08cbdd8
Compare
That is, support entries in `home-manager.users` that do not have a corresponding entry in `users.users`.
to permit enabling this feature for users specified in `home-manager.users` that have a corresponding entry in `users.users` while still supporting users that do not.
08cbdd8 to
1213f40
Compare
|
this changeset is pretty much exactly what I’m looking for; is anyone available to review or comment on why this may not be a particularly good approach? |
|
I have been testing this PR locally (after merging it with master to resolve the conflicts) and it seems to be working pretty effectively. The main downside is I didn't realize One issue that I am unsure is related to this or something else in my configuration (I am pretty sure this is related to my identity provider wigging out on me so I think it is unrelated) but I get the following error sometimes
Oddly enough when it does occur it is transient since manually running the service works (To be clear not looking for help or anything just figured I would share the weird thing that happened when testing) |
|
@Guvante -- I have hit this same issue. I tried adding |
Description
This changeset introduces support in the NixOS module for entries in
home-manager.usersthat do not have corresponding entries inusers.users. The goal is to support Home Manager configurations for users provided by external identity management systems, as described in #5244, without resorting to hacks like re-specifying those users' details (UIDs, etc.) inusers.users.Checklist
Change is backwards compatible. NOTE: this changeset alters the semantics of
home-manager.useUserPackages: that option now provides the default value of the per-user optionhome-manager.users.<name>.home.useUserPackages, rather than globally instituting (or not) the use ofusers.users.<name>.packages. I believe this change is backward-compatible, given that no Home Manager NixOS module configuration will have usedhome-manager.users.<name>.home.useUserPackagesbefore now, and so existing configurations will continue to globally use (or not)users.users.<name>.packages.Code formatted with
./format.Code tested through
nix-shell --pure tests -A run.allor
nix build --reference-lock-file flake.lock ./tests#test-allusing Flakes.Test cases updated/added. See example.
Commit messages are formatted like
See CONTRIBUTING for more information and recent commit messages for examples.
If this PR adds a new module[ ] Added myself as module maintainer. See example.Maintainer CC