Skip to content

Commit 925a1cc

Browse files
committed
vdi: only write a small data amount in large volumes for faster tests
Signed-off-by: Gaëtan Lehmann <gaetan.lehmann@vates.tech>
1 parent fc481ce commit 925a1cc

1 file changed

Lines changed: 5 additions & 13 deletions

File tree

tests/storage/storage.py

Lines changed: 5 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -307,16 +307,8 @@ def vdi_export_import(vm: VM, sr: SR, image_format: ImageFormat, temp_large_dir:
307307
defer(lambda: vm.disconnect_vdi(vdi_src) if vdi_src is not None and vdi_src.uuid in vm.vdis else None)
308308
dev = f'/dev/{vbd.param_get("device")}'
309309

310-
# the stream is 1/5 of the full one, truncated to a multiple of 32KiB, in order to
311-
# be validable in a single command
312-
stream_size = (config.volume_size // 5 // (32 * KiB)) * (32 * KiB)
313-
stream_position = (config.volume_size // 2)
314-
315-
checksum1 = randstream(vm, f'generate --size {stream_size} {dev}')
316-
# use a different seed to not write the same data (default seed is 0)
317-
checksum2 = randstream(vm, f'generate --seed 1 --position {stream_position} --size {stream_size} {dev}')
318-
randstream(vm, f'validate --size {stream_size} --expected-checksum {checksum1} {dev}')
319-
randstream(vm, f'validate --position {stream_position} --size {stream_size} --expected-checksum {checksum2} {dev}')
310+
checksums = partially_populate_device(vm, dev, config.volume_size)
311+
validate_partially_populated_device(vm, dev, config.volume_size, checksums)
320312
vm.disconnect_vdi(vdi_src)
321313

322314
image_path = f'{temp_large_dir}/{vdi_src.uuid}.{image_format}'
@@ -327,7 +319,8 @@ def vdi_export_import(vm: VM, sr: SR, image_format: ImageFormat, temp_large_dir:
327319

328320
# check that the zero blocks are not part of the result
329321
size_mb = int(vm.host.ssh(f'du -sm --apparent-size {image_path}').split()[0])
330-
assert stream_size // MiB * 2 < size_mb < stream_size // MiB * 2.1, f"unexpected image size: {size_mb}"
322+
stream_size = partial_stream_size(config.volume_size)
323+
assert stream_size // MiB * 3 < size_mb < stream_size // MiB * 3.1, f"unexpected image size: {size_mb}"
331324
vdi_dest = sr.create_vdi(image_format=image_format, virtual_size=config.volume_size)
332325
defer(lambda: vdi_dest.destroy())
333326

@@ -336,5 +329,4 @@ def vdi_export_import(vm: VM, sr: SR, image_format: ImageFormat, temp_large_dir:
336329
defer(lambda: vm.disconnect_vdi(vdi_dest))
337330
dev = f'/dev/{vbd.param_get("device")}'
338331

339-
randstream(vm, f'validate --size {stream_size} --expected-checksum {checksum1} {dev}')
340-
randstream(vm, f'validate --position {stream_position} --size {stream_size} --expected-checksum {checksum2} {dev}')
332+
validate_partially_populated_device(vm, dev, config.volume_size, checksums)

0 commit comments

Comments
 (0)