9:25 a.m.
cimtest currently doesn't work too well on s390 systems,
the two main areas of failure are the attempt to
define systems with IDE hard drivers and VGA graphics,
both not supported on this host type.
Changing from IDE to virtio disks fixed a lot of the
test FAILs with no adverse impacts to the x86 runs
on our systems at least. It would be nice if somebody
could verify this.
Further, we do not add the graphics RASD to the default
guest definition if the hypervisor host is an s390
system. In principle, we could have removed the graphics
RASD altogether, because libvirt-cim will define a
default graphics device for x86 but since it is only a
small change, it seems safer to do it this way.
Thilo Boehm (1):
cimtest: Use vda instead of hda for disk RASDs
Viktor Mihajlovski (1):
cimtest: Don't generate graphics for s390
.../cimtest/ElementAllocatedFromPool/01_forward.py | 2 +-
.../cimtest/ElementAllocatedFromPool/02_reverse.py | 2 +-
.../ElementAllocatedFromPool/03_reverse_errs.py | 2 +-
.../ElementAllocatedFromPool/04_forward_errs.py | 2 +-
.../cimtest/HostedAccessPoint/01_forward.py | 2 +-
.../cimtest/HostedAccessPoint/02_reverse.py | 2 +-
.../02_guest_add_mod_rem_rasd_ind.py | 2 +-
.../cimtest/ServiceAccessBySAP/01_forward.py | 2 +-
.../cimtest/ServiceAccessBySAP/02_reverse.py | 2 +-
.../cimtest/SettingsDefine/01_forward.py | 2 +-
.../cimtest/SettingsDefine/02_reverse.py | 2 +-
.../cimtest/SettingsDefine/03_sds_fwd_errs.py | 2 +-
.../cimtest/SettingsDefine/04_sds_rev_errs.py | 2 +-
.../libvirt-cim/cimtest/SystemDevice/01_forward.py | 2 +-
.../libvirt-cim/cimtest/SystemDevice/03_fwderrs.py | 2 +-
suites/libvirt-cim/cimtest/VSSD/04_vssd_to_rasd.py | 2 +-
.../13_refconfig_additional_devs.py | 2 +-
.../14_define_sys_disk.py | 2 +-
.../30_dynamic_disk_mod.py | 2 +-
.../02_reverse.py | 2 +-
.../03_vssdc_fwd_errs.py | 2 +-
suites/libvirt-cim/lib/XenKvmLib/const.py | 2 +-
suites/libvirt-cim/lib/XenKvmLib/vsms.py | 2 +-
suites/libvirt-cim/lib/XenKvmLib/vxml.py | 5 +++--
suites/libvirt-cim/lib/XenKvmLib/xm_virt_util.py | 7 +++++++
25 files changed, 33 insertions(+), 25 deletions(-)
--
1.7.9.5
9:25 a.m.
New subject: [PATCH 1/2] cimtest: Use vda instead of hda for disk RASDs
From: Thilo Boehm <tboehm(a)linux.vnet.ibm.com>
Running cimtest against a s390 host will fail almost every testcase
involving a disk definition in a domain. This is because the testcase
instantiate IDE disk drives implicitly by using hdx as disk identifiers.
Changing the testcases to use virtio disks will enable s390 domains
to be started, while the testcases will still function under x86.
Signed-off-by: Thilo Boehm <tboehm(a)linux.vnet.ibm.com>
Signed-off-by: Viktor Mihajlovski <mihajlov(a)linux.vnet.ibm.com>
---
.../cimtest/ElementAllocatedFromPool/01_forward.py | 2 +-
.../cimtest/ElementAllocatedFromPool/02_reverse.py | 2 +-
.../ElementAllocatedFromPool/03_reverse_errs.py | 2 +-
.../ElementAllocatedFromPool/04_forward_errs.py | 2 +-
.../cimtest/HostedAccessPoint/01_forward.py | 2 +-
.../cimtest/HostedAccessPoint/02_reverse.py | 2 +-
.../02_guest_add_mod_rem_rasd_ind.py | 2 +-
.../cimtest/ServiceAccessBySAP/01_forward.py | 2 +-
.../cimtest/ServiceAccessBySAP/02_reverse.py | 2 +-
.../cimtest/SettingsDefine/01_forward.py | 2 +-
.../cimtest/SettingsDefine/02_reverse.py | 2 +-
.../cimtest/SettingsDefine/03_sds_fwd_errs.py | 2 +-
.../cimtest/SettingsDefine/04_sds_rev_errs.py | 2 +-
.../libvirt-cim/cimtest/SystemDevice/01_forward.py | 2 +-
.../libvirt-cim/cimtest/SystemDevice/03_fwderrs.py | 2 +-
suites/libvirt-cim/cimtest/VSSD/04_vssd_to_rasd.py | 2 +-
.../13_refconfig_additional_devs.py | 2 +-
.../14_define_sys_disk.py | 2 +-
.../30_dynamic_disk_mod.py | 2 +-
.../02_reverse.py | 2 +-
.../03_vssdc_fwd_errs.py | 2 +-
suites/libvirt-cim/lib/XenKvmLib/const.py | 2 +-
suites/libvirt-cim/lib/XenKvmLib/vsms.py | 2 +-
23 files changed, 23 insertions(+), 23 deletions(-)
diff --git a/suites/libvirt-cim/cimtest/ElementAllocatedFromPool/01_forward.py
b/suites/libvirt-cim/cimtest/ElementAllocatedFromPool/01_forward.py
index f7503dd..82b12a5 100755
--- a/suites/libvirt-cim/cimtest/ElementAllocatedFromPool/01_forward.py
+++ b/suites/libvirt-cim/cimtest/ElementAllocatedFromPool/01_forward.py
@@ -146,7 +146,7 @@ def main():
if virt == 'Xen':
test_disk = 'xvda'
else:
- test_disk = 'hda'
+ test_disk = 'vda'
# Getting the VS list and deleting the test_dom if it already exists.
destroy_and_undefine_all(server)
diff --git a/suites/libvirt-cim/cimtest/ElementAllocatedFromPool/02_reverse.py
b/suites/libvirt-cim/cimtest/ElementAllocatedFromPool/02_reverse.py
index e682f11..30367cd 100644
--- a/suites/libvirt-cim/cimtest/ElementAllocatedFromPool/02_reverse.py
+++ b/suites/libvirt-cim/cimtest/ElementAllocatedFromPool/02_reverse.py
@@ -176,7 +176,7 @@ def main():
if virt == 'Xen':
test_disk = 'xvdb'
else:
- test_disk = 'hda'
+ test_disk = 'vda'
# Getting the VS list and deleting the test_dom if it already exists.
virt_type = get_class(virt)
diff --git a/suites/libvirt-cim/cimtest/ElementAllocatedFromPool/03_reverse_errs.py
b/suites/libvirt-cim/cimtest/ElementAllocatedFromPool/03_reverse_errs.py
index bb273b8..7787c1d 100644
--- a/suites/libvirt-cim/cimtest/ElementAllocatedFromPool/03_reverse_errs.py
+++ b/suites/libvirt-cim/cimtest/ElementAllocatedFromPool/03_reverse_errs.py
@@ -180,7 +180,7 @@ def main():
if virt == "Xen":
test_disk = "xvda"
else:
- test_disk = "hda"
+ test_disk = "vda"
virt_type = get_class(virt)
vsxml = virt_type (test_dom, vcpus = test_vcpus, mac = test_mac,
disk = test_disk)
diff --git a/suites/libvirt-cim/cimtest/ElementAllocatedFromPool/04_forward_errs.py
b/suites/libvirt-cim/cimtest/ElementAllocatedFromPool/04_forward_errs.py
index 3281942..fc1dcf7 100644
--- a/suites/libvirt-cim/cimtest/ElementAllocatedFromPool/04_forward_errs.py
+++ b/suites/libvirt-cim/cimtest/ElementAllocatedFromPool/04_forward_errs.py
@@ -496,7 +496,7 @@ def main():
if virt == "Xen":
test_disk = "xvda"
else:
- test_disk = "hda"
+ test_disk = "vda"
destroy_and_undefine_all(options.ip)
vsxml = get_class(virt)(test_dom, vcpus = test_vcpus,
mac = test_mac, disk = test_disk)
diff --git a/suites/libvirt-cim/cimtest/HostedAccessPoint/01_forward.py
b/suites/libvirt-cim/cimtest/HostedAccessPoint/01_forward.py
index 8493ed7..9ffd0fd 100644
--- a/suites/libvirt-cim/cimtest/HostedAccessPoint/01_forward.py
+++ b/suites/libvirt-cim/cimtest/HostedAccessPoint/01_forward.py
@@ -41,7 +41,7 @@ def setup_env(server, virt):
if virt == 'Xen':
test_disk = 'xvda'
else:
- test_disk = 'hda'
+ test_disk = 'vda'
virt_xml = get_class(virt)
if virt == 'LXC':
diff --git a/suites/libvirt-cim/cimtest/HostedAccessPoint/02_reverse.py
b/suites/libvirt-cim/cimtest/HostedAccessPoint/02_reverse.py
index 74850ed..2d6daf1 100644
--- a/suites/libvirt-cim/cimtest/HostedAccessPoint/02_reverse.py
+++ b/suites/libvirt-cim/cimtest/HostedAccessPoint/02_reverse.py
@@ -41,7 +41,7 @@ def setup_env(server, virt):
if virt == 'Xen':
test_disk = 'xvda'
else:
- test_disk = 'hda'
+ test_disk = 'vda'
virt_xml = get_class(virt)
if virt == 'LXC':
diff --git a/suites/libvirt-cim/cimtest/RASDIndications/02_guest_add_mod_rem_rasd_ind.py
b/suites/libvirt-cim/cimtest/RASDIndications/02_guest_add_mod_rem_rasd_ind.py
index 329c7b4..6d513bd 100644
--- a/suites/libvirt-cim/cimtest/RASDIndications/02_guest_add_mod_rem_rasd_ind.py
+++ b/suites/libvirt-cim/cimtest/RASDIndications/02_guest_add_mod_rem_rasd_ind.py
@@ -151,7 +151,7 @@ def main():
ndpath = cxml.secondary_disk_path
if virt == 'KVM':
- nddev = 'hdb'
+ nddev = 'vdb'
else:
nddev = 'xvdb'
diff --git a/suites/libvirt-cim/cimtest/ServiceAccessBySAP/01_forward.py
b/suites/libvirt-cim/cimtest/ServiceAccessBySAP/01_forward.py
index 525e05f..4840522 100644
--- a/suites/libvirt-cim/cimtest/ServiceAccessBySAP/01_forward.py
+++ b/suites/libvirt-cim/cimtest/ServiceAccessBySAP/01_forward.py
@@ -43,7 +43,7 @@ def setup_env(server, virt):
if virt == 'Xen':
test_disk = 'xvda'
else:
- test_disk = 'hda'
+ test_disk = 'vda'
virt_xml = get_class(virt)
diff --git a/suites/libvirt-cim/cimtest/ServiceAccessBySAP/02_reverse.py
b/suites/libvirt-cim/cimtest/ServiceAccessBySAP/02_reverse.py
index 1c1c913..14c0db1 100644
--- a/suites/libvirt-cim/cimtest/ServiceAccessBySAP/02_reverse.py
+++ b/suites/libvirt-cim/cimtest/ServiceAccessBySAP/02_reverse.py
@@ -42,7 +42,7 @@ def setup_env(server, virt):
if virt == 'Xen':
test_disk = 'xvda'
else:
- test_disk = 'hda'
+ test_disk = 'vda'
virt_xml = get_class(virt)
diff --git a/suites/libvirt-cim/cimtest/SettingsDefine/01_forward.py
b/suites/libvirt-cim/cimtest/SettingsDefine/01_forward.py
index 850f3ce..5850090 100755
--- a/suites/libvirt-cim/cimtest/SettingsDefine/01_forward.py
+++ b/suites/libvirt-cim/cimtest/SettingsDefine/01_forward.py
@@ -45,7 +45,7 @@ def setup_env(server, virt):
if virt == 'Xen':
test_disk = 'xvda'
else:
- test_disk = 'hda'
+ test_disk = 'vda'
virt_xml = get_class(virt)
if virt == 'LXC':
cxml = virt_xml(test_dom)
diff --git a/suites/libvirt-cim/cimtest/SettingsDefine/02_reverse.py
b/suites/libvirt-cim/cimtest/SettingsDefine/02_reverse.py
index fdae709..1917113 100644
--- a/suites/libvirt-cim/cimtest/SettingsDefine/02_reverse.py
+++ b/suites/libvirt-cim/cimtest/SettingsDefine/02_reverse.py
@@ -70,7 +70,7 @@ def setup_env(server, virt):
if virt == 'Xen':
test_disk = 'xvdb'
else:
- test_disk = 'hdb'
+ test_disk = 'vdb'
virt_xml = get_class(virt)
if virt == 'LXC':
cxml = virt_xml(test_dom)
diff --git a/suites/libvirt-cim/cimtest/SettingsDefine/03_sds_fwd_errs.py
b/suites/libvirt-cim/cimtest/SettingsDefine/03_sds_fwd_errs.py
index 58b9cbc..d4f7c89 100644
--- a/suites/libvirt-cim/cimtest/SettingsDefine/03_sds_fwd_errs.py
+++ b/suites/libvirt-cim/cimtest/SettingsDefine/03_sds_fwd_errs.py
@@ -211,7 +211,7 @@ def main():
if options.virt == 'Xen':
test_disk = 'xvda'
else:
- test_disk = 'hda'
+ test_disk = 'vda'
virt_xml = vxml.get_class(options.virt)
if options.virt == 'LXC':
diff --git a/suites/libvirt-cim/cimtest/SettingsDefine/04_sds_rev_errs.py
b/suites/libvirt-cim/cimtest/SettingsDefine/04_sds_rev_errs.py
index 43484e9..fdd0086 100644
--- a/suites/libvirt-cim/cimtest/SettingsDefine/04_sds_rev_errs.py
+++ b/suites/libvirt-cim/cimtest/SettingsDefine/04_sds_rev_errs.py
@@ -101,7 +101,7 @@ def main():
if options.virt == 'Xen':
test_disk = 'xvda'
else:
- test_disk = 'hda'
+ test_disk = 'vda'
virt_xml = vxml.get_class(options.virt)
if options.virt == 'LXC':
diff --git a/suites/libvirt-cim/cimtest/SystemDevice/01_forward.py
b/suites/libvirt-cim/cimtest/SystemDevice/01_forward.py
index 330c6fb..15a4ff5 100644
--- a/suites/libvirt-cim/cimtest/SystemDevice/01_forward.py
+++ b/suites/libvirt-cim/cimtest/SystemDevice/01_forward.py
@@ -52,7 +52,7 @@ def main():
elif virt == 'LXC':
test_disk = '/tmp'
else:
- test_disk = 'hdb'
+ test_disk = 'vdb'
status = PASS
virt_xml = vxml.get_class(virt)
diff --git a/suites/libvirt-cim/cimtest/SystemDevice/03_fwderrs.py
b/suites/libvirt-cim/cimtest/SystemDevice/03_fwderrs.py
index 0e7e167..e48bcf5 100644
--- a/suites/libvirt-cim/cimtest/SystemDevice/03_fwderrs.py
+++ b/suites/libvirt-cim/cimtest/SystemDevice/03_fwderrs.py
@@ -60,7 +60,7 @@ def main():
if options.virt == 'Xen':
test_disk = 'xvda'
else:
- test_disk = 'hda'
+ test_disk = 'vda'
status = PASS
virt_xml = vxml.get_class(options.virt)
diff --git a/suites/libvirt-cim/cimtest/VSSD/04_vssd_to_rasd.py
b/suites/libvirt-cim/cimtest/VSSD/04_vssd_to_rasd.py
index 4d24b39..04b0b6e 100755
--- a/suites/libvirt-cim/cimtest/VSSD/04_vssd_to_rasd.py
+++ b/suites/libvirt-cim/cimtest/VSSD/04_vssd_to_rasd.py
@@ -214,7 +214,7 @@ def main():
if virt == "LXC":
test_disk = "/tmp"
else:
- test_disk = "hda"
+ test_disk = "vda"
status, vsxml = setup_env(virt)
if status != PASS:
diff --git
a/suites/libvirt-cim/cimtest/VirtualSystemManagementService/13_refconfig_additional_devs.py
b/suites/libvirt-cim/cimtest/VirtualSystemManagementService/13_refconfig_additional_devs.py
index a47db8f..5e9e780 100644
---
a/suites/libvirt-cim/cimtest/VirtualSystemManagementService/13_refconfig_additional_devs.py
+++
b/suites/libvirt-cim/cimtest/VirtualSystemManagementService/13_refconfig_additional_devs.py
@@ -125,7 +125,7 @@ def main():
test_disk = 'xvdb'
disk_path = Xen_secondary_disk_path
else:
- test_disk = 'hdb'
+ test_disk = 'vdb'
disk_path = KVM_secondary_disk_path
cxml2 = virt_xml(test_dom2, mac=mac2,
diff --git
a/suites/libvirt-cim/cimtest/VirtualSystemManagementService/14_define_sys_disk.py
b/suites/libvirt-cim/cimtest/VirtualSystemManagementService/14_define_sys_disk.py
index 294c275..4adabbd 100644
--- a/suites/libvirt-cim/cimtest/VirtualSystemManagementService/14_define_sys_disk.py
+++ b/suites/libvirt-cim/cimtest/VirtualSystemManagementService/14_define_sys_disk.py
@@ -67,7 +67,7 @@ def get_rasd_list(ip, virt, addr, disk_type):
if virt == 'Xen':
test_disk = 'xvda'
else:
- test_disk = 'hda'
+ test_disk = 'vda'
for rasd in rasds:
if rasd.classname == drasd_cn:
diff --git
a/suites/libvirt-cim/cimtest/VirtualSystemManagementService/30_dynamic_disk_mod.py
b/suites/libvirt-cim/cimtest/VirtualSystemManagementService/30_dynamic_disk_mod.py
index 8e6a3ae..5d81a21 100644
--- a/suites/libvirt-cim/cimtest/VirtualSystemManagementService/30_dynamic_disk_mod.py
+++ b/suites/libvirt-cim/cimtest/VirtualSystemManagementService/30_dynamic_disk_mod.py
@@ -39,7 +39,7 @@ def main():
options = main.options
if options.virt == 'KVM':
- nddev = 'vda'
+ nddev = 'vdb'
else:
nddev = 'xvdb'
diff --git a/suites/libvirt-cim/cimtest/VirtualSystemSettingDataComponent/02_reverse.py
b/suites/libvirt-cim/cimtest/VirtualSystemSettingDataComponent/02_reverse.py
index 0624b1d..3a13b2a 100644
--- a/suites/libvirt-cim/cimtest/VirtualSystemSettingDataComponent/02_reverse.py
+++ b/suites/libvirt-cim/cimtest/VirtualSystemSettingDataComponent/02_reverse.py
@@ -159,7 +159,7 @@ def main():
elif options.virt == "LXC":
test_disk = "/tmp"
else:
- test_disk = "hdb"
+ test_disk = "vdb"
virt_xml = vxml.get_class(options.virt)
if options.virt == 'LXC':
diff --git
a/suites/libvirt-cim/cimtest/VirtualSystemSettingDataComponent/03_vssdc_fwd_errs.py
b/suites/libvirt-cim/cimtest/VirtualSystemSettingDataComponent/03_vssdc_fwd_errs.py
index 492a55f..52f8c48 100644
--- a/suites/libvirt-cim/cimtest/VirtualSystemSettingDataComponent/03_vssdc_fwd_errs.py
+++ b/suites/libvirt-cim/cimtest/VirtualSystemSettingDataComponent/03_vssdc_fwd_errs.py
@@ -105,7 +105,7 @@ def main():
if virt == "Xen":
test_disk = "xvda"
else:
- test_disk = "hda"
+ test_disk = "vda"
virt_xml = vxml.get_class(virt)
if virt == 'LXC':
diff --git a/suites/libvirt-cim/lib/XenKvmLib/const.py
b/suites/libvirt-cim/lib/XenKvmLib/const.py
index 6701d36..1d655d0 100755
--- a/suites/libvirt-cim/lib/XenKvmLib/const.py
+++ b/suites/libvirt-cim/lib/XenKvmLib/const.py
@@ -86,7 +86,7 @@ Xen_default_mac = '88:22:33:aa:bb:cc'
KVM_default_emulator = '/usr/bin/qemu-system-x86_64'
KVM_disk_path = os.path.join(_image_dir, 'default-kvm-dimage')
KVM_secondary_disk_path = os.path.join(_image_dir, 'default-kvm-dimage.2ND')
-KVM_default_disk_dev = 'hda'
+KVM_default_disk_dev = 'vda'
KVM_default_cdrom_dev = 'hdc'
KVM_default_mac = '88:22:33:aa:bb:cc'
diff --git a/suites/libvirt-cim/lib/XenKvmLib/vsms.py
b/suites/libvirt-cim/lib/XenKvmLib/vsms.py
index 03cbe30..d7f33f9 100755
--- a/suites/libvirt-cim/lib/XenKvmLib/vsms.py
+++ b/suites/libvirt-cim/lib/XenKvmLib/vsms.py
@@ -339,7 +339,7 @@ def default_vssd_rasd_str(dom_name='test_domain',
class_dasd = get_dasd_class(virt)
if virt == 'KVM':
- disk_dev = 'hda'
+ disk_dev = 'vda'
disk_source = const.KVM_disk_path
elif virt == 'XenFV':
disk_dev = 'hda'
--
1.7.9.5
9:25 a.m.
New subject: [PATCH 2/2] cimtest: Don't generate graphics for s390
Make sure we're not trying to generate a graphics device on s390
which fails every testcase starting a domain.
For this purpose a new utility function has been provided to
query the hypervisor node's CPU model.
Signed-off-by: Viktor Mihajlovski <mihajlov(a)linux.vnet.ibm.com>
---
suites/libvirt-cim/lib/XenKvmLib/vxml.py | 5 +++--
suites/libvirt-cim/lib/XenKvmLib/xm_virt_util.py | 7 +++++++
2 files changed, 10 insertions(+), 2 deletions(-)
diff --git a/suites/libvirt-cim/lib/XenKvmLib/vxml.py
b/suites/libvirt-cim/lib/XenKvmLib/vxml.py
index 82ab501..74c4f23 100644
--- a/suites/libvirt-cim/lib/XenKvmLib/vxml.py
+++ b/suites/libvirt-cim/lib/XenKvmLib/vxml.py
@@ -41,7 +41,7 @@ from time import sleep
from VirtLib import utils, live
from XenKvmLib.xm_virt_util import get_bridge_from_network_xml, bootloader, \
- net_list
+ net_list, host_cpu_model
from XenKvmLib.test_doms import set_uuid, viruuid
from XenKvmLib import vsms
from XenKvmLib import const
@@ -650,7 +650,8 @@ class VirtCIM:
curr_cim_rev, changeset = get_provider_version(self.virt, ip)
if curr_cim_rev >= vsms_graphics_sup:
- if self.gasd is not None:
+ if self.gasd is not None and \
+ not str(host_cpu_model(ip, self.virt)).startswith('s390'):
res_settings.append(str(self.gasd))
if curr_cim_rev >= vsms_inputdev_sup:
diff --git a/suites/libvirt-cim/lib/XenKvmLib/xm_virt_util.py
b/suites/libvirt-cim/lib/XenKvmLib/xm_virt_util.py
index 7749fb7..33edd6a 100644
--- a/suites/libvirt-cim/lib/XenKvmLib/xm_virt_util.py
+++ b/suites/libvirt-cim/lib/XenKvmLib/xm_virt_util.py
@@ -417,3 +417,10 @@ def network_macs(ip, vs_name):
return ret
+def host_cpu_model(server, virt="KVM"):
+ cmd = "virsh -c %s nodeinfo 2>/dev/null | grep 'CPU model'" %
virt2uri(virt)
+ ret, out = utils.run_remote(server, cmd)
+ if ret != 0:
+ return None
+ left, right = out.split(':')
+ return right.strip()
--
1.7.9.5
9:29 a.m.
On 09/05/2013 10:25 AM, Viktor Mihajlovski wrote:
cimtest currently doesn't work too well on s390 systems,
the two main areas of failure are the attempt to
define systems with IDE hard drivers and VGA graphics,
both not supported on this host type.
Changing from IDE to virtio disks fixed a lot of the
test FAILs with no adverse impacts to the x86 runs
on our systems at least. It would be nice if somebody
could verify this.
Further, we do not add the graphics RASD to the default
guest definition if the hypervisor host is an s390
system. In principle, we could have removed the graphics
RASD altogether, because libvirt-cim will define a
default graphics device for x86 but since it is only a
small change, it seems safer to do it this way.
Thilo Boehm (1):
cimtest: Use vda instead of hda for disk RASDs
Viktor Mihajlovski (1):
cimtest: Don't generate graphics for s390
.../cimtest/ElementAllocatedFromPool/01_forward.py | 2 +-
.../cimtest/ElementAllocatedFromPool/02_reverse.py | 2 +-
.../ElementAllocatedFromPool/03_reverse_errs.py | 2 +-
.../ElementAllocatedFromPool/04_forward_errs.py | 2 +-
.../cimtest/HostedAccessPoint/01_forward.py | 2 +-
.../cimtest/HostedAccessPoint/02_reverse.py | 2 +-
.../02_guest_add_mod_rem_rasd_ind.py | 2 +-
.../cimtest/ServiceAccessBySAP/01_forward.py | 2 +-
.../cimtest/ServiceAccessBySAP/02_reverse.py | 2 +-
.../cimtest/SettingsDefine/01_forward.py | 2 +-
.../cimtest/SettingsDefine/02_reverse.py | 2 +-
.../cimtest/SettingsDefine/03_sds_fwd_errs.py | 2 +-
.../cimtest/SettingsDefine/04_sds_rev_errs.py | 2 +-
.../libvirt-cim/cimtest/SystemDevice/01_forward.py | 2 +-
.../libvirt-cim/cimtest/SystemDevice/03_fwderrs.py | 2 +-
suites/libvirt-cim/cimtest/VSSD/04_vssd_to_rasd.py | 2 +-
.../13_refconfig_additional_devs.py | 2 +-
.../14_define_sys_disk.py | 2 +-
.../30_dynamic_disk_mod.py | 2 +-
.../02_reverse.py | 2 +-
.../03_vssdc_fwd_errs.py | 2 +-
suites/libvirt-cim/lib/XenKvmLib/const.py | 2 +-
suites/libvirt-cim/lib/XenKvmLib/vsms.py | 2 +-
suites/libvirt-cim/lib/XenKvmLib/vxml.py | 5 +++--
suites/libvirt-cim/lib/XenKvmLib/xm_virt_util.py | 7 +++++++
25 files changed, 33 insertions(+), 25 deletions(-)
I ran this on my Fedora19 environment without issues - seems reasonable
to me. Hopefully others can 'git am' the changes and run/test as well.
I'd venture to guess that the defaults these tests have chosen haven't
been changed in quite a while and could use a few adjustments here and
there to account for more recent changes...
ACK,
John
7:07 a.m.
On 09/05/2013 10:25 AM, Viktor Mihajlovski wrote:
cimtest currently doesn't work too well on s390 systems,
the two main areas of failure are the attempt to
define systems with IDE hard drivers and VGA graphics,
both not supported on this host type.
Changing from IDE to virtio disks fixed a lot of the
test FAILs with no adverse impacts to the x86 runs
on our systems at least. It would be nice if somebody
could verify this.
Further, we do not add the graphics RASD to the default
guest definition if the hypervisor host is an s390
system. In principle, we could have removed the graphics
RASD altogether, because libvirt-cim will define a
default graphics device for x86 but since it is only a
small change, it seems safer to do it this way.
Thilo Boehm (1):
cimtest: Use vda instead of hda for disk RASDs
Viktor Mihajlovski (1):
cimtest: Don't generate graphics for s390
.../cimtest/ElementAllocatedFromPool/01_forward.py | 2 +-
.../cimtest/ElementAllocatedFromPool/02_reverse.py | 2 +-
.../ElementAllocatedFromPool/03_reverse_errs.py | 2 +-
.../ElementAllocatedFromPool/04_forward_errs.py | 2 +-
.../cimtest/HostedAccessPoint/01_forward.py | 2 +-
.../cimtest/HostedAccessPoint/02_reverse.py | 2 +-
.../02_guest_add_mod_rem_rasd_ind.py | 2 +-
.../cimtest/ServiceAccessBySAP/01_forward.py | 2 +-
.../cimtest/ServiceAccessBySAP/02_reverse.py | 2 +-
.../cimtest/SettingsDefine/01_forward.py | 2 +-
.../cimtest/SettingsDefine/02_reverse.py | 2 +-
.../cimtest/SettingsDefine/03_sds_fwd_errs.py | 2 +-
.../cimtest/SettingsDefine/04_sds_rev_errs.py | 2 +-
.../libvirt-cim/cimtest/SystemDevice/01_forward.py | 2 +-
.../libvirt-cim/cimtest/SystemDevice/03_fwderrs.py | 2 +-
suites/libvirt-cim/cimtest/VSSD/04_vssd_to_rasd.py | 2 +-
.../13_refconfig_additional_devs.py | 2 +-
.../14_define_sys_disk.py | 2 +-
.../30_dynamic_disk_mod.py | 2 +-
.../02_reverse.py | 2 +-
.../03_vssdc_fwd_errs.py | 2 +-
suites/libvirt-cim/lib/XenKvmLib/const.py | 2 +-
suites/libvirt-cim/lib/XenKvmLib/vsms.py | 2 +-
suites/libvirt-cim/lib/XenKvmLib/vxml.py | 5 +++--
suites/libvirt-cim/lib/XenKvmLib/xm_virt_util.py | 7 +++++++
25 files changed, 33 insertions(+), 25 deletions(-)
This is now pushed.
John
4119
days inactive
4125
days old
4 comments
2 participants
participants (2)
-
John Ferlan -
Viktor Mihajlovski