Skip to content

systemd-sysupdate: init#243995

Merged
RaitoBezarius merged 4 commits intoNixOS:masterfrom
nikstur:systemd-sysupdate
Jul 26, 2023
Merged

systemd-sysupdate: init#243995
RaitoBezarius merged 4 commits intoNixOS:masterfrom
nikstur:systemd-sysupdate

Conversation

@nikstur
Copy link
Contributor

@nikstur nikstur commented Jul 17, 2023

Description of changes

Introduces systemd-sysupdate. Among other things, this enables partition based A/B updates for NixOS appliances that do not rely on the traditional NixOS generations mechanism.

The first commit factors the generation of a gpg-keyring out of nixos/tests/systemd-nspawn into nixos/tests/common/gpg-keyring.nix so it can be re-used for testing systemd-sysupdate.

Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 23.11 Release Notes (or backporting 23.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

@github-actions github-actions bot added 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 8.has: module (update) This PR changes an existing module in `nixos/` 6.topic: systemd Software suite that provides an array of system components for Linux operating systems. labels Jul 17, 2023
@nikstur nikstur changed the title Systemd sysupdate Draft: systemd-sysupdate: init Jul 17, 2023
@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels Jul 17, 2023
@nikstur nikstur force-pushed the systemd-sysupdate branch from 1990627 to 858767f Compare July 18, 2023 10:17
@github-actions github-actions bot added 8.has: documentation This PR adds or changes documentation 8.has: changelog This PR adds or changes release notes labels Jul 18, 2023
@nikstur nikstur changed the title Draft: systemd-sysupdate: init systemd-sysupdate: init Jul 18, 2023
@nikstur nikstur marked this pull request as ready for review July 18, 2023 10:17
@nikstur nikstur requested a review from a team as a code owner July 18, 2023 10:17
@nikstur nikstur force-pushed the systemd-sysupdate branch from 858767f to 39709e3 Compare July 18, 2023 11:00
@nikstur nikstur force-pushed the systemd-sysupdate branch 2 times, most recently from 5dbab49 to 048c987 Compare July 19, 2023 08:41
@nikstur nikstur requested review from arianvp and flokli July 19, 2023 08:52
@nikstur nikstur force-pushed the systemd-sysupdate branch 3 times, most recently from 321a6b1 to e0e4b5c Compare July 24, 2023 14:39
@nikstur
Copy link
Contributor Author

nikstur commented Jul 24, 2023

@arianvp is this good to go now?

@nikstur nikstur added the 12.approvals: 1 This PR was reviewed and approved by one person. label Jul 25, 2023
Copy link
Member

@RaitoBezarius RaitoBezarius left a comment

Choose a reason for hiding this comment

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

This would greatly benefit from:

(a) adding yourself as a maintainer of the module.
(b) a Markdown documentation of this module for operational usage inside nixpkgs.

It's not a blocker for this PR.

@nikstur nikstur force-pushed the systemd-sysupdate branch from e0e4b5c to 200c4a2 Compare July 26, 2023 18:30
@nikstur
Copy link
Contributor Author

nikstur commented Jul 26, 2023

(a) adding yourself as a maintainer of the module.

Ah I definitely meant to do this. Somehow missed it. Thank you for pointing it out.

I'll write some docs in a separate PR. Ideally when I have collected some more insights on how to use it well with NixOS.

@nikstur nikstur force-pushed the systemd-sysupdate branch from 200c4a2 to e6862fa Compare July 26, 2023 18:34
@nikstur
Copy link
Contributor Author

nikstur commented Jul 26, 2023

@ofborg test systemd-sysupdate

@RaitoBezarius RaitoBezarius merged commit 96f7ceb into NixOS:master Jul 26, 2023
@nikstur nikstur mentioned this pull request Oct 26, 2023
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 6.topic: systemd Software suite that provides an array of system components for Linux operating systems. 8.has: changelog This PR adds or changes release notes 8.has: documentation This PR adds or changes documentation 8.has: module (update) This PR changes an existing module in `nixos/` 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 12.approvals: 1 This PR was reviewed and approved by one person.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants