Skip to content

modular-services: add user creation/spec support - #1

Closed
eveeifyeve wants to merge 1 commit into
masterfrom
modular-services-user
Closed

eveeifyeve wants to merge 1 commit into
masterfrom
modular-services-user

Conversation

@eveeifyeve

@eveeifyeve eveeifyeve commented Oct 16, 2026 •

Copy link
Copy Markdown
Member

Caution

This pull request is still being cleaned up and is not fully production ready, use at your own risk.

This pull request adds user creation/specification support to modular services! This allows you to introduce services built with specific users, userid, group and group id and adds further support for future API's such as runtimeDirectory, cacheDirectory stateDirectory, logDirectory in future.

Things done

  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.
  • Follows the automation/AI policy.
  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • aarch64-darwin
  • Tested, as applicable:
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Validated with the following use cases:
    • simple, no users
    • services with same-user constraints but no names or ids. Names can be derived from the service attribute path. Ids use existing solutions like NixOS id-less users (mutable-users or otherwise).
    • services with user names
    • services with all information pinned down
    • services with ids pinned down but not using default names

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for contributing to the documentation.

Follow the documentation style guide, especially:

  • Show, don’t tell. Start with a minimal working example. Explain after the code.
  • Avoid meta-commentary. Don’t write "This section explains how to…".
  • Use present tense, imperative mood, and active voice. For example: "Run the command," not "The user should run the following command."
  • Avoid vague references like "this" when the referent is unclear. Write "This command creates a folder," not "This will create a folder."
  • Be direct. Avoid "should", "might", "typically", and "usually".
  • Cut filler words like "simply", "just", "easily", and "basically". Use "to" instead of "in order to".

For larger changes, such as adding or removing whole sections, ask the NixOS documentation team for review.

Comment thread lib/services/identity.nix Outdated
Comment thread nixos/modules/system/service/README.md Outdated
@eveeifyeve

eveeifyeve commented Oct 16, 2026 •

Copy link
Copy Markdown
Member Author

CC @aanderse for review.

@eveeifyeve
eveeifyeve force-pushed the modular-services-user branch 2 times, most recently from c06a856 to 38bab53 Compare October 16, 2026 19:44
@eveeifyeve
eveeifyeve requested a review from roberth October 16, 2026 19:45

@roberth roberth left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a partial review

Main things left to do:

  • Review the prose critically
  • Validate the interface by applying it in diverse scenarios

Comment thread lib/services/identity.nix Outdated
Comment thread lib/services/identity.nix
Comment thread lib/services/identity.nix Outdated
@eveeifyeve
eveeifyeve force-pushed the modular-services-user branch from 38bab53 to ae1fe02 Compare October 17, 2026 23:35
@eveeifyeve
eveeifyeve requested a review from roberth October 18, 2026 03:47
@eveeifyeve
eveeifyeve force-pushed the modular-services-user branch 2 times, most recently from 7068b7d to c5d1298 Compare October 22, 2026 04:31
@eveeifyeve eveeifyeve changed the title WIP: modular-services: add user creation/spec support modular-services: add user creation/spec support Oct 22, 2026
@eveeifyeve
eveeifyeve marked this pull request as ready for review October 22, 2026 04:32
@eveeifyeve

Copy link
Copy Markdown
Member Author

Moving to nixpkgs, as I consider this ready for a pr to nixpkgs.

@eveeifyeve eveeifyeve closed this Oct 27, 2026
@eveeifyeve

Copy link
Copy Markdown
Member Author

NixOS#579760

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.

2 participants