It turns out that checking for the last mount time of an ext4 file
system isn't a very reliable way to check whether the file system was
properly unmounted.
When creating that test in the first place (88530e02b6
),
I was reluctant to inspect the file system when the VM is down and was
searching for a way to check for a clean unmount *after* the file system
was mounted again to make sure we don't need to create a 512 MB raw
image on the host.
Fortunately however, when converting from qcow2, qemu-img actually
writes a sparse file, so for most file systems (that is, file systems
supporting sparse files) this shouldn't waste a lot of disk space.
So when investigating the flakiness, I found that whenever the test is
failing, the unmount of /test-x-initrd-mount was done *before* the final
step during which systemd remounts+unmounts all the remaining file
systems.
I haven't investigated why this is the case, but the test is a
regression test for https://github.com/NixOS/nixpkgs/issues/35268, which
actually didn't unmount the file system *at* *all*, so really all we
need to take care here is whether the unmount has happened and not
*how*.
To make sure that checking the filesystem state is enough for this, I
temporarily replaced the $machine->shutdown call with $machine->crash
and verified that the file system state is "not clean".
Signed-off-by: aszlig <aszlig@nix.build>
Fixes: https://github.com/NixOS/nixpkgs/issues/67555
103 lines
3.5 KiB
Nix
103 lines
3.5 KiB
Nix
import ./make-test.nix ({ pkgs, ... }: {
|
|
name = "systemd";
|
|
|
|
machine = { lib, ... }: {
|
|
imports = [ common/user-account.nix common/x11.nix ];
|
|
|
|
virtualisation.emptyDiskImages = [ 512 ];
|
|
|
|
fileSystems = lib.mkVMOverride {
|
|
"/test-x-initrd-mount" = {
|
|
device = "/dev/vdb";
|
|
fsType = "ext2";
|
|
autoFormat = true;
|
|
noCheck = true;
|
|
options = [ "x-initrd.mount" ];
|
|
};
|
|
};
|
|
|
|
systemd.extraConfig = "DefaultEnvironment=\"XXX_SYSTEM=foo\"";
|
|
systemd.user.extraConfig = "DefaultEnvironment=\"XXX_USER=bar\"";
|
|
services.journald.extraConfig = "Storage=volatile";
|
|
services.xserver.displayManager.auto.user = "alice";
|
|
|
|
systemd.shutdown.test = pkgs.writeScript "test.shutdown" ''
|
|
#!${pkgs.stdenv.shell}
|
|
PATH=${lib.makeBinPath (with pkgs; [ utillinux coreutils ])}
|
|
mount -t 9p shared -o trans=virtio,version=9p2000.L /tmp/shared
|
|
touch /tmp/shared/shutdown-test
|
|
umount /tmp/shared
|
|
'';
|
|
|
|
systemd.services.testservice1 = {
|
|
description = "Test Service 1";
|
|
wantedBy = [ "multi-user.target" ];
|
|
serviceConfig.Type = "oneshot";
|
|
script = ''
|
|
if [ "$XXX_SYSTEM" = foo ]; then
|
|
touch /system_conf_read
|
|
fi
|
|
'';
|
|
};
|
|
|
|
systemd.user.services.testservice2 = {
|
|
description = "Test Service 2";
|
|
wantedBy = [ "default.target" ];
|
|
serviceConfig.Type = "oneshot";
|
|
script = ''
|
|
if [ "$XXX_USER" = bar ]; then
|
|
touch "$HOME/user_conf_read"
|
|
fi
|
|
'';
|
|
};
|
|
};
|
|
|
|
testScript = ''
|
|
$machine->waitForX;
|
|
# wait for user services
|
|
$machine->waitForUnit("default.target","alice");
|
|
|
|
# Regression test for https://github.com/NixOS/nixpkgs/issues/35415
|
|
subtest "configuration files are recognized by systemd", sub {
|
|
$machine->succeed('test -e /system_conf_read');
|
|
$machine->succeed('test -e /home/alice/user_conf_read');
|
|
$machine->succeed('test -z $(ls -1 /var/log/journal)');
|
|
};
|
|
|
|
# Regression test for https://github.com/NixOS/nixpkgs/issues/50273
|
|
subtest "DynamicUser actually allocates a user", sub {
|
|
$machine->succeed('systemd-run --pty --property=Type=oneshot --property=DynamicUser=yes --property=User=iamatest whoami | grep iamatest');
|
|
};
|
|
|
|
# Regression test for https://github.com/NixOS/nixpkgs/issues/35268
|
|
subtest "file system with x-initrd.mount is not unmounted", sub {
|
|
$machine->succeed('mountpoint -q /test-x-initrd-mount');
|
|
$machine->shutdown;
|
|
system('qemu-img', 'convert', '-O', 'raw',
|
|
'vm-state-machine/empty2.qcow2', 'x-initrd-mount.raw');
|
|
my $extinfo = `${pkgs.e2fsprogs}/bin/dumpe2fs x-initrd-mount.raw`;
|
|
die "File system was not cleanly unmounted: $extinfo"
|
|
unless $extinfo =~ /^Filesystem state: *clean$/m;
|
|
};
|
|
|
|
subtest "systemd-shutdown works", sub {
|
|
$machine->shutdown;
|
|
$machine->waitForUnit('multi-user.target');
|
|
$machine->succeed('test -e /tmp/shared/shutdown-test');
|
|
};
|
|
|
|
# Test settings from /etc/sysctl.d/50-default.conf are applied
|
|
subtest "systemd sysctl settings are applied", sub {
|
|
$machine->waitForUnit('multi-user.target');
|
|
$machine->succeed('sysctl net.core.default_qdisc | grep -q "fq_codel"');
|
|
};
|
|
|
|
# Test cgroup accounting is enabled
|
|
subtest "systemd cgroup accounting is enabled", sub {
|
|
$machine->waitForUnit('multi-user.target');
|
|
$machine->succeed('systemctl show testservice1.service -p IOAccounting | grep -q "yes"');
|
|
$machine->succeed('systemctl status testservice1.service | grep -q "CPU:"');
|
|
};
|
|
'';
|
|
})
|