[PATCH] Make physical_resources_check cope with qemu change on 'info block'

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux