Skip to content

Comments

Re-enable systemd-nspawn test#9008

Merged
edolstra merged 1 commit intoNixOS:masterfrom
cole-h:re-enable-nspawn-test
Sep 20, 2023
Merged

Re-enable systemd-nspawn test#9008
edolstra merged 1 commit intoNixOS:masterfrom
cole-h:re-enable-nspawn-test

Conversation

@cole-h
Copy link
Member

@cole-h cole-h commented Sep 20, 2023

Motivation

More tests is better than less tests.

Context

It was disabled in c6953d1 because a recent Nixpkgs bump brought in a new systemd which changed how systemd-nspawn worked.

As far as I can tell, the issue was caused by this upstream systemd commit:
systemd/systemd@b71a019

Bind-mounting the host's /sys and /proc into the container's /run/host/{sys,proc} fixes the issue and allows the test to succeed.

Priorities

Add 👍 to pull requests you find important.

It was disabled in c6953d1 because
a recent Nixpkgs bump brought in a new systemd which changed how
systemd-nspawn worked.

As far as I can tell, the issue was caused by this upstream systemd
commit:
systemd/systemd@b71a019

Bind-mounting the host's `/sys` and `/proc` into the container's
`/run/host/{sys,proc}` fixes the issue and allows the test to succeed.
@cole-h cole-h requested a review from edolstra as a code owner September 20, 2023 16:13
@github-actions github-actions bot added the with-tests Issues related to testing. PRs with tests have some priority label Sep 20, 2023
@edolstra edolstra enabled auto-merge September 20, 2023 16:36
@edolstra edolstra merged commit 3ce793d into NixOS:master Sep 20, 2023
@cole-h cole-h deleted the re-enable-nspawn-test branch September 20, 2023 17:03
@github-actions
Copy link

Successfully created backport PR for 2.18-maintenance:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

with-tests Issues related to testing. PRs with tests have some priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants