New qemu upstream versions report block info like this: (monitor humanmonitor1) Sending command 'info block' (monitor humanmonitor1) Response to 'info block' (monitor humanmonitor1) virtio0: removable=0 file=/tmp/kvm_autotest_root/images/rhel6-64.qcow2 ro=0 drv=qcow2 encrypted=0 (monitor humanmonitor1) ide1-cd0: removable=1 locked=0 [not inserted] (monitor humanmonitor1) floppy0: removable=1 locked=0 [not inserted] (monitor humanmonitor1) sd0: removable=1 locked=0 [not inserted] (monitor humanmonitor1) Sending command 'info block' (monitor humanmonitor1) Response to 'info block' (monitor humanmonitor1) virtio0: type=hd removable=0 file=/tmp/kvm_autotest_root/images/rhel6-64.qcow2 ro=0 drv=qcow2 encrypted=0 (monitor humanmonitor1) ide1-cd0: type=cdrom removable=1 locked=0 [not inserted] (monitor humanmonitor1) floppy0: type=floppy removable=1 locked=0 [not inserted] (monitor humanmonitor1) sd0: type=floppy removable=1 locked=0 [not inserted] So the types are missing. Adapting the test to take into account those differences. Signed-off-by: Lucas Meneghel Rodrigues <lmr@xxxxxxxxxx> --- client/tests/kvm/tests/physical_resources_check.py | 8 ++++++-- 1 files changed, 6 insertions(+), 2 deletions(-) diff --git a/client/tests/kvm/tests/physical_resources_check.py b/client/tests/kvm/tests/physical_resources_check.py index 53a23d1..9d93c1c 100644 --- a/client/tests/kvm/tests/physical_resources_check.py +++ b/client/tests/kvm/tests/physical_resources_check.py @@ -29,6 +29,9 @@ def run_physical_resources_check(test, params, env): # resources to know which checks passed and which ones failed n_fail = 0 + # We will check HDs with the image name + image_name = virt_vm.get_image_filename(params, test.bindir) + # Check cpu count logging.info("CPU count check") expected_cpu_nr = int(params.get("smp")) @@ -70,7 +73,7 @@ def run_physical_resources_check(test, params, env): return expected_num, f_fail logging.info("Hard drive count check") - n_fail += check_num("images", "block", "type=hd")[1] + n_fail += check_num("images", "block", image_name)[1] logging.info("NIC count check") n_fail += check_num("nics", "network", "model=")[1] @@ -111,7 +114,8 @@ def run_physical_resources_check(test, params, env): n_fail += f_fail logging.info("Drive format check") - f_fail = chk_fmt_model("images", "drive_format", "block", "(.*)\: type=hd") + f_fail = chk_fmt_model("images", "drive_format", + "block", "(.*)\: .*%s" % image_name) n_fail += f_fail logging.info("Network card MAC check") -- 1.7.5 -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html