From 2616fac240c4fd58bebfdf88d2a05cb41314f126 Mon Sep 17 00:00:00 2001 From: Martin Pitt Date: Tue, 27 Aug 2024 11:00:16 +0200 Subject: [PATCH] test: Clean up "supports external snapshot" OS list The ubuntu-stable image has been 24.04 for some time, so drop the dynamic special case. We stopped releasing to Fedora 39, so drop that special case. --- test/check-machines-snapshots | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/test/check-machines-snapshots b/test/check-machines-snapshots index 751f166c4..5b7272ab1 100755 --- a/test/check-machines-snapshots +++ b/test/check-machines-snapshots @@ -243,12 +243,7 @@ class TestMachinesSnapshots(machineslib.VirtualMachinesCase): b.wait_visible("#vm-subVmTest1-add-snapshot-button") # external memory snapshots introduced in libvirt 9.9.0 - supports_external = not ( - m.image.startswith("rhel-8") or - m.image in ["debian-stable", "ubuntu-2204", "ubuntu-stable", "fedora-39"]) - # Transitional code while we move ubuntu-stable from 23.10 mantic to 24.04 noble - if m.image == "ubuntu-stable" and m.execute(". /etc/os-release; echo $VERSION_ID").strip() == "24.04": - supports_external = True + supports_external = not (m.image.startswith("rhel-8") or m.image in ["debian-stable", "ubuntu-2204"]) # HACK: deleting external snapshots for non-running VMs is broken https://bugs.debian.org/bug=1061725 # Work around that by temporarily disabling libvirtd's AppArmor profile. AppArmor isn't installed by