Open
Conversation
jfroche
force-pushed
the
system-manager-base
branch
from
4d4ff65 to
293c344
Compare
This comment has been minimized.
This comment has been minimized.
jfroche
force-pushed
the
system-manager-base
branch
3 times, most recently
from
e9e7d85 to
ee5fe34
Compare
picnoir
force-pushed
the
system-manager-base
branch
from
6e9dd1f to
ee5fe34
Compare
* Create default system manager configuration * chores: add nix run .#check-system-module to github actions workflows * feat: replace Docker-based system-manager tests with container test framework Switch from building Docker images and running pytest+testinfra externally to using system-manager's built-in makeContainerTest API backed by systemd-nspawn. The test is now a Nix check derivation that runs inside the build sandbox. It requires auto-allocating UIDs in the ephemeral Nix installation, which is now enabled by default in the GitHub Action. Rebased from #2010 Co-authored-by: Yvan Sraka <yvan@sraka.xyz>
system-manager's userborn service rewrites /etc/passwd entries: - root shell: /bin/bash -> /run/system-manager/sw/bin/bash - nobody shell: /usr/sbin/nologin -> /run/system-manager/sw/bin/nologin
The check should be part of the regular nix-build workflow
Enabling the nginx service in the system configuration was a good start, but it had implications for the test suite verifying that the AMI was correctly configured. We change the configuration to set up a basic ssh config file that matches the expected configuration for the AMI, and update the tests to verify that the file is created with the correct content and permissions.
samrose
force-pushed
the
system-manager-base
branch
from
fa57ba5 to
9fb8bdf
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Create default system manager configuration
chores: add nix run .#check-system-module to github actions workflows
feat: replace Docker-based system-manager tests with container test framework
Switch from building Docker images and running pytest+testinfra externally to using system-manager's built-in makeContainerTest API backed by systemd-nspawn. The test is now a Nix check derivation that runs inside the build sandbox.
It requires auto-allocating UIDs in the ephemeral Nix installation, which is now enabled by default in the GitHub Action.
Splited from #1882
Rebased from #2010
Testing: The test uses system-manager's makeContainerTest API which spins up a lightweight systemd-nspawn container (not a full VM). Inside the container it waits for the system to reach multi-user
target, then activates the system-manager configuration and waits for system-manager.target to come up. Once active, it verifies that the /etc/system-manager-genesis file was created by
checking that it exists, has mode 0644, and is owned by root:root. This proves the full system-manager pipeline works end to end, from Nix module definition through to file deployment inside a
running system.