[pve-devel] [PATCH qemu-server 4/5] test: fix tests for changes when printing disk size suffix

Fiona Ebner f.ebner at proxmox.com
Mon May 12 14:41:28 CEST 2025


The 'disk-size' format in the JSON schema in pve-common was changed to
print more verbose, non-ambiguous disk size suffixes. This requires
adapting some tests here.

Bump the corresponding build-dependency in d/control.

Signed-off-by: Fiona Ebner <f.ebner at proxmox.com>
---
 debian/control                         |  2 +-
 test/restore-config-expected/139.conf  |  4 +-
 test/restore-config-expected/142.conf  |  2 +-
 test/restore-config-expected/1422.conf |  2 +-
 test/restore-config-expected/179.conf  |  8 +--
 test/run_qemu_migrate_tests.pl         | 86 +++++++++++++-------------
 6 files changed, 52 insertions(+), 52 deletions(-)

diff --git a/debian/control b/debian/control
index d6c20040..18f25d66 100644
--- a/debian/control
+++ b/debian/control
@@ -8,7 +8,7 @@ Build-Depends: debhelper-compat (= 13),
                libjson-c-dev,
                libpve-apiclient-perl,
                libpve-cluster-perl,
-               libpve-common-perl (>= 8.0.2),
+               libpve-common-perl (>= 8.3.2),
                libpve-guest-common-perl (>= 5.2.2),
                libpve-storage-perl (>= 8.2.10),
                libtest-mockmodule-perl,
diff --git a/test/restore-config-expected/139.conf b/test/restore-config-expected/139.conf
index 94425f7e..aeed26cc 100644
--- a/test/restore-config-expected/139.conf
+++ b/test/restore-config-expected/139.conf
@@ -2,14 +2,14 @@
 bios: ovmf
 boot: order=scsi0;ide2;net0
 cores: 1
-efidisk0: target:139/vm-139-disk-0.qcow2,size=128K
+efidisk0: target:139/vm-139-disk-0.qcow2,size=128KiB
 ide2: local:iso/debian-10.6.0-amd64-netinst.iso,media=cdrom
 memory: 2048
 name: eficloneclone
 net0: virtio=7A:6C:A5:8B:11:93,bridge=vmbr0,firewall=1
 numa: 0
 ostype: l26
-scsi0: target:139/vm-139-disk-1.raw,size=4G
+scsi0: target:139/vm-139-disk-1.raw,size=4GiB
 scsihw: virtio-scsi-pci
 smbios1: uuid=21a7e7bc-3cd2-4232-a009-a41f4ee992ae
 sockets: 1
diff --git a/test/restore-config-expected/142.conf b/test/restore-config-expected/142.conf
index ac2d2ad2..82e7b9fc 100644
--- a/test/restore-config-expected/142.conf
+++ b/test/restore-config-expected/142.conf
@@ -7,7 +7,7 @@ name: apache
 net0: virtio=92:38:11:FD:ED:87,bridge=vmbr0,firewall=1
 numa: 0
 ostype: l26
-scsi0: target:142/vm-142-disk-0.qcow2,size=4G
+scsi0: target:142/vm-142-disk-0.qcow2,size=4GiB
 scsihw: virtio-scsi-pci
 smbios1: uuid=ddf91b3f-a597-42be-9a7e-fb6421dcd5cd
 sockets: 1
diff --git a/test/restore-config-expected/1422.conf b/test/restore-config-expected/1422.conf
index 2d77a44a..b601d362 100644
--- a/test/restore-config-expected/1422.conf
+++ b/test/restore-config-expected/1422.conf
@@ -7,7 +7,7 @@ name: apache
 net0: virtio=92:38:11:FD:ED:87,bridge=vmbr0,firewall=1
 numa: 0
 ostype: l26
-scsi0: target:1422/vm-1422-disk-0.qcow2,size=4G
+scsi0: target:1422/vm-1422-disk-0.qcow2,size=4GiB
 scsihw: virtio-scsi-pci
 smbios1: uuid=ddf91b3f-a597-42be-9a7e-fb6421dcd5cd
 sockets: 1
diff --git a/test/restore-config-expected/179.conf b/test/restore-config-expected/179.conf
index 4444efb5..255318fb 100644
--- a/test/restore-config-expected/179.conf
+++ b/test/restore-config-expected/179.conf
@@ -6,10 +6,10 @@ memory: 2048
 net0: virtio=26:15:5B:73:3F:7C,bridge=vmbr0,firewall=1
 numa: 0
 ostype: l26
-scsi0: target:179/vm-179-disk-0.qcow2,cache=none,discard=on,size=32G,ssd=1
-scsi1: target:179/vm-179-disk-1.qcow2,cache=writethrough,size=32G
-scsi2: target:179/vm-179-disk-2.qcow2,mbps_rd=7,mbps_wr=7,replicate=0,size=32G
-scsi3: target:179/vm-179-disk-3.vmdk,size=32G
+scsi0: target:179/vm-179-disk-0.qcow2,cache=none,discard=on,size=32GiB,ssd=1
+scsi1: target:179/vm-179-disk-1.qcow2,cache=writethrough,size=32GiB
+scsi2: target:179/vm-179-disk-2.qcow2,mbps_rd=7,mbps_wr=7,replicate=0,size=32GiB
+scsi3: target:179/vm-179-disk-3.vmdk,size=32GiB
 #scsi4: myfs:179/vm-179-disk-1.qcow2,backup=0,size=32G
 scsihw: virtio-scsi-pci
 smbios1: uuid=1819ead7-a55d-4544-8d38-29ca94869a9c
diff --git a/test/run_qemu_migrate_tests.pl b/test/run_qemu_migrate_tests.pl
index 4373a380..1a60c29c 100755
--- a/test/run_qemu_migrate_tests.pl
+++ b/test/run_qemu_migrate_tests.pl
@@ -113,7 +113,7 @@ my $vm_configs = {
      105 => {
 	'bootdisk' => 'scsi0',
 	'cores' => 1,
-	'ide0' => 'local-zfs:vm-105-disk-1,size=103M',
+	'ide0' => 'local-zfs:vm-105-disk-1,size=103MiB',
 	'ide2' => 'none,media=cdrom',
 	'memory' => 512,
 	'name' => 'Copy-of-VM-newapache',
@@ -122,7 +122,7 @@ my $vm_configs = {
 	'ostype' => 'l26',
 	'parent' => 'ohsnap',
 	'pending' => {},
-	'scsi0' => 'local-zfs:vm-105-disk-0,size=4G',
+	'scsi0' => 'local-zfs:vm-105-disk-0,size=4GiB',
 	'scsihw' => 'virtio-scsi-pci',
 	'smbios1' => 'uuid=1ddfe18b-77e0-47f6-a4bd-f1761bf6d763',
 	'snapshots' => {
@@ -135,7 +135,7 @@ my $vm_configs = {
 		'net0' => 'virtio=4A:A3:E4:4C:CF:F0,bridge=vmbr0,firewall=1',
 		'numa' => 0,
 		'ostype' => 'l26',
-		'scsi0' => 'local-zfs:vm-105-disk-0,size=4G',
+		'scsi0' => 'local-zfs:vm-105-disk-0,size=4GiB',
 		'scsihw' => 'virtio-scsi-pci',
 		'smbios1' => 'uuid=1ddfe18b-77e0-47f6-a4bd-f1761bf6d763',
 		'snaptime' => 1580976924,
@@ -151,7 +151,7 @@ my $vm_configs = {
     111 => {
 	'bootdisk' => 'scsi0',
 	'cores' => 1,
-	'ide0' => 'local-lvm:vm-111-disk-0,size=4096M',
+	'ide0' => 'local-lvm:vm-111-disk-0,size=4096MiB',
 	'ide2' => 'none,media=cdrom',
 	'memory' => 512,
 	'name' => 'pending-test',
@@ -159,7 +159,7 @@ my $vm_configs = {
 	'numa' => 0,
 	'ostype' => 'l26',
 	'pending' => {
-		'scsi0' => 'local-zfs:vm-111-disk-0,size=103M',
+		'scsi0' => 'local-zfs:vm-111-disk-0,size=103MiB',
 	},
 	'scsihw' => 'virtio-scsi-pci',
 	'snapshots' => {},
@@ -170,8 +170,8 @@ my $vm_configs = {
     123 => {
 	'bootdisk' => 'scsi0',
 	'cores' => 1,
-	'scsi0' => 'zfs-alias-1:vm-123-disk-0,size=4096M',
-	'scsi1' => 'zfs-alias-2:vm-123-disk-0,size=4096M',
+	'scsi0' => 'zfs-alias-1:vm-123-disk-0,size=4096MiB',
+	'scsi1' => 'zfs-alias-2:vm-123-disk-0,size=4096MiB',
 	'ide2' => 'none,media=cdrom',
 	'memory' => 512,
 	'name' => 'alias-test',
@@ -196,8 +196,8 @@ my $vm_configs = {
 	'net0' => 'virtio=52:5D:7E:62:85:97,bridge=vmbr1',
 	'numa' => 1,
 	'ostype' => 'l26',
-	'scsi0' => 'local-lvm:vm-149-disk-0,format=raw,size=4G',
-	'scsi1' => 'local-dir:149/vm-149-disk-0.qcow2,format=qcow2,size=1G',
+	'scsi0' => 'local-lvm:vm-149-disk-0,format=raw,size=4GiB',
+	'scsi1' => 'local-dir:149/vm-149-disk-0.qcow2,format=qcow2,size=1GiB',
 	'scsihw' => 'virtio-scsi-pci',
 	'snapshots' => {},
 	'smbios1' => 'uuid=e980bd43-a405-42e2-b5f4-31efe6517460',
@@ -217,7 +217,7 @@ my $vm_configs = {
 	'net0' => 'virtio=4E:F1:82:6D:D7:4B,bridge=vmbr0,firewall=1,rate=10',
 	'numa' => 0,
 	'ostype' => 'l26',
-	'scsi0' => 'rbd-store:vm-341-disk-0,size=1G',
+	'scsi0' => 'rbd-store:vm-341-disk-0,size=1GiB',
 	'scsihw' => 'virtio-scsi-pci',
 	'snapshots' => {},
 	'smbios1' => 'uuid=e01e4c73-46f1-47c8-af79-288fdf6b7462',
@@ -227,7 +227,7 @@ my $vm_configs = {
     1033 => {
 	'bootdisk' => 'scsi0',
 	'cores' => 1,
-	'ide0' => 'rbd-store:vm-1033-cloudinit,media=cdrom,size=4M',
+	'ide0' => 'rbd-store:vm-1033-cloudinit,media=cdrom,size=4MiB',
 	'ide2' => 'none,media=cdrom',
 	'ipconfig0' => 'ip=103.214.69.10/25,gw=103.214.69.1',
 	'memory' => 4096,
@@ -235,7 +235,7 @@ my $vm_configs = {
 	'net0' => 'virtio=4E:F1:82:6D:D7:4B,bridge=vmbr0,firewall=1,rate=10',
 	'numa' => 0,
 	'ostype' => 'l26',
-	'scsi0' => 'rbd-store:vm-1033-disk-1,size=1G',
+	'scsi0' => 'rbd-store:vm-1033-disk-1,size=1GiB',
 	'scsihw' => 'virtio-scsi-pci',
 	'snapshots' => {},
 	'smbios1' => 'uuid=e01e4c73-46f1-47c8-af79-288fdf6b7462',
@@ -253,7 +253,7 @@ my $vm_configs = {
 	'ostype' => 'l26',
 	'parent' => 'snap1',
 	'pending' => {},
-	'scsi0' => 'local-dir:4567/vm-4567-disk-0.qcow2,size=4G',
+	'scsi0' => 'local-dir:4567/vm-4567-disk-0.qcow2,size=4GiB',
 	'scsihw' => 'virtio-scsi-pci',
 	'smbios1' => 'uuid=2925fdec-a066-4228-b46b-eef8662f5e74',
 	'snapshots' => {
@@ -268,7 +268,7 @@ my $vm_configs = {
 		'ostype' => 'l26',
 		'runningcpu' => 'kvm64,enforce,+kvm_pv_eoi,+kvm_pv_unhalt,+lahf_lm,+sep',
 		'runningmachine' => 'pc-i440fx-5.0+pve0',
-		'scsi0' => 'local-dir:4567/vm-4567-disk-0.qcow2,size=4G',
+		'scsi0' => 'local-dir:4567/vm-4567-disk-0.qcow2,size=4GiB',
 		'scsihw' => 'virtio-scsi-pci',
 		'smbios1' => 'uuid=2925fdec-a066-4228-b46b-eef8662f5e74',
 		'snaptime' => 1595928799,
@@ -289,8 +289,8 @@ my $vm_configs = {
 		'parent' => 'snap1',
 		'runningcpu' => 'kvm64,enforce,+kvm_pv_eoi,+kvm_pv_unhalt,+lahf_lm,+sep',
 		'runningmachine' => 'pc-i440fx-5.0+pve0',
-		'scsi0' => 'local-dir:4567/vm-4567-disk-0.qcow2,size=4G',
-		'scsi1' => 'local-zfs:vm-4567-disk-0,size=1G',
+		'scsi0' => 'local-dir:4567/vm-4567-disk-0.qcow2,size=4GiB',
+		'scsi1' => 'local-zfs:vm-4567-disk-0,size=1GiB',
 		'scsihw' => 'virtio-scsi-pci',
 		'smbios1' => 'uuid=2925fdec-a066-4228-b46b-eef8662f5e74',
 		'snaptime' => 1595928871,
@@ -543,7 +543,7 @@ my $tests = [
 		'local-dir:341/vm-341-disk-10.raw' => 1,
 	    },
 	    vm_config => get_patched_config(341, {
-		efidisk0 => 'local-dir:341/vm-341-disk-10.raw,format=raw,size=128K',
+		efidisk0 => 'local-dir:341/vm-341-disk-10.raw,format=raw,size=128KiB',
 	    }),
 	    vm_status => {
 		running => 1,
@@ -571,7 +571,7 @@ my $tests = [
 		'local-lvm:vm-341-disk-10' => 1,
 	    },
 	    vm_config => get_patched_config(341, {
-		efidisk0 => 'local-lvm:vm-341-disk-10,format=raw,size=128K',
+		efidisk0 => 'local-lvm:vm-341-disk-10,format=raw,size=128KiB',
 	    }),
 	    vm_status => {
 		running => 1,
@@ -735,15 +735,15 @@ my $tests = [
 		'local-zfs:vm-4567-disk-0' => 1,
 	    },
 	    vm_config => get_patched_config(4567, {
-		'scsi0' => 'other-dir:4567/vm-4567-disk-0.qcow2,size=4G',
+		'scsi0' => 'other-dir:4567/vm-4567-disk-0.qcow2,size=4GiB',
 		snapshots => {
 		    snap1 => {
-			'scsi0' => 'other-dir:4567/vm-4567-disk-0.qcow2,size=4G',
+			'scsi0' => 'other-dir:4567/vm-4567-disk-0.qcow2,size=4GiB',
 			'vmstate' => 'other-dir:4567/vm-4567-state-snap1.raw',
 		    },
 		    snap2 => {
-			'scsi0' => 'other-dir:4567/vm-4567-disk-0.qcow2,size=4G',
-			'scsi1' => 'local-zfs:vm-4567-disk-0,size=1G',
+			'scsi0' => 'other-dir:4567/vm-4567-disk-0.qcow2,size=4GiB',
+			'scsi1' => 'local-zfs:vm-4567-disk-0,size=1GiB',
 			'vmstate' => 'other-dir:4567/vm-4567-state-snap2.raw',
 		    },
 		},
@@ -822,7 +822,7 @@ my $tests = [
 		'local-zfs:vm-149-disk-10' => 1,
 	    },
 	    vm_config => get_patched_config(149, {
-		scsi0 => 'local-zfs:vm-149-disk-10,format=raw,size=4G',
+		scsi0 => 'local-zfs:vm-149-disk-10,format=raw,size=4GiB',
 		scsi1 => undef,
 	    }),
 	    vm_status => {
@@ -858,7 +858,7 @@ my $tests = [
 		'local-lvm:vm-149-disk-10' => 1,
 	    },
 	    vm_config => get_patched_config(149, {
-		scsi0 => 'local-lvm:vm-149-disk-10,format=raw,size=4G',
+		scsi0 => 'local-lvm:vm-149-disk-10,format=raw,size=4GiB',
 		scsi1 => undef,
 	    }),
 	    vm_status => {
@@ -894,8 +894,8 @@ my $tests = [
 		'local-lvm:vm-149-disk-0' => 1,
 	    },
 	    vm_config => get_patched_config(149, {
-		scsi0 => 'local-zfs:vm-149-disk-0,format=raw,size=4G',
-		scsi1 => 'local-lvm:vm-149-disk-0,format=qcow2,size=1G',
+		scsi0 => 'local-zfs:vm-149-disk-0,format=raw,size=4GiB',
+		scsi1 => 'local-lvm:vm-149-disk-0,format=qcow2,size=1GiB',
 	    }),
 	    vm_status => {
 		running => 0,
@@ -927,8 +927,8 @@ my $tests = [
 		'local-zfs:vm-149-disk-0' => 1,
 	    },
 	    vm_config => get_patched_config(149, {
-		scsi0 => 'local-dir:149/vm-149-disk-0.raw,format=raw,size=4G',
-		scsi1 => 'local-zfs:vm-149-disk-0,format=qcow2,size=1G',
+		scsi0 => 'local-dir:149/vm-149-disk-0.raw,format=raw,size=4GiB',
+		scsi1 => 'local-zfs:vm-149-disk-0,format=qcow2,size=1GiB',
 	    }),
 	    vm_status => {
 		running => 0,
@@ -956,8 +956,8 @@ my $tests = [
 		'local-lvm:vm-149-disk-11' => 1,
 	    },
 	    vm_config => get_patched_config(149, {
-		scsi0 => 'local-zfs:vm-149-disk-10,format=raw,size=4G',
-		scsi1 => 'local-lvm:vm-149-disk-11,format=raw,size=1G',
+		scsi0 => 'local-zfs:vm-149-disk-10,format=raw,size=4GiB',
+		scsi1 => 'local-lvm:vm-149-disk-11,format=raw,size=1GiB',
 	    }),
 	    vm_status => {
 		running => 1,
@@ -986,8 +986,8 @@ my $tests = [
 		'local-dir:149/vm-149-disk-11.qcow2' => 1,
 	    },
 	    vm_config => get_patched_config(149, {
-		scsi0 => 'local-zfs:vm-149-disk-10,format=raw,size=4G',
-		scsi1 => 'local-dir:149/vm-149-disk-11.qcow2,format=qcow2,size=1G',
+		scsi0 => 'local-zfs:vm-149-disk-10,format=raw,size=4GiB',
+		scsi1 => 'local-dir:149/vm-149-disk-11.qcow2,format=qcow2,size=1GiB',
 	    }),
 	    vm_status => {
 		running => 1,
@@ -1016,8 +1016,8 @@ my $tests = [
 		'local-zfs:vm-149-disk-11' => 1,
 	    },
 	    vm_config => get_patched_config(149, {
-		scsi0 => 'local-zfs:vm-149-disk-10,format=raw,size=4G',
-		scsi1 => 'local-zfs:vm-149-disk-11,format=raw,size=1G',
+		scsi0 => 'local-zfs:vm-149-disk-10,format=raw,size=4GiB',
+		scsi1 => 'local-zfs:vm-149-disk-11,format=raw,size=1GiB',
 	    }),
 	    vm_status => {
 		running => 1,
@@ -1038,7 +1038,7 @@ my $tests = [
 	    'with-local-disks' => 1,
 	},
 	config_patch => {
-	    scsi0 => 'local-lvm:vm-149-disk-0,size=123T',
+	    scsi0 => 'local-lvm:vm-149-disk-0,size=123TiB',
 	},
 	expected_calls => $default_expected_calls_online,
 	expected => {
@@ -1048,8 +1048,8 @@ my $tests = [
 		'local-dir:149/vm-149-disk-11.qcow2' => 1,
 	    },
 	    vm_config => get_patched_config(149, {
-		scsi0 => 'local-lvm:vm-149-disk-10,format=raw,size=4G',
-		scsi1 => 'local-dir:149/vm-149-disk-11.qcow2,format=qcow2,size=1G',
+		scsi0 => 'local-lvm:vm-149-disk-10,format=raw,size=4GiB',
+		scsi1 => 'local-dir:149/vm-149-disk-11.qcow2,format=qcow2,size=1GiB',
 	    }),
 	    vm_status => {
 		running => 1,
@@ -1080,8 +1080,8 @@ my $tests = [
 		'local-dir:149/vm-149-disk-11.qcow2' => 1,
 	    },
 	    vm_config => get_patched_config(149, {
-		scsi0 => 'local-lvm:vm-149-disk-10,format=raw,size=4G',
-		scsi1 => 'local-dir:149/vm-149-disk-11.qcow2,format=qcow2,size=1G',
+		scsi0 => 'local-lvm:vm-149-disk-10,format=raw,size=4GiB',
+		scsi1 => 'local-dir:149/vm-149-disk-11.qcow2,format=qcow2,size=1GiB',
 	    }),
 	    vm_status => {
 		running => 1,
@@ -1500,8 +1500,8 @@ my $tests = [
 		'local-dir:149/vm-149-disk-11.qcow2' => 1,
 	    },
 	    vm_config => get_patched_config(149, {
-		scsi0 => 'local-lvm:vm-149-disk-10,format=raw,size=4G',
-		scsi1 => 'local-dir:149/vm-149-disk-11.qcow2,format=qcow2,size=1G',
+		scsi0 => 'local-lvm:vm-149-disk-10,format=raw,size=4GiB',
+		scsi1 => 'local-dir:149/vm-149-disk-11.qcow2,format=qcow2,size=1GiB',
 	    }),
 	    vm_status => {
 		running => 1,
@@ -1635,9 +1635,9 @@ my $tests = [
 		'local-lvm:vm-111-disk-10' => 1,
 	    },
 	    vm_config => get_patched_config(111, {
-		ide0 => 'local-lvm:vm-111-disk-10,format=raw,size=4G',
+		ide0 => 'local-lvm:vm-111-disk-10,format=raw,size=4GiB',
 		pending => {
-		    scsi0 => 'local-zfs:vm-111-disk-0,size=103M',
+		    scsi0 => 'local-zfs:vm-111-disk-0,size=103MiB',
 		},
 	    }),
 	    vm_status => {
-- 
2.39.5





More information about the pve-devel mailing list