[Libvirt-cim] [PATCH] [TEST] For Xen enabled systems, a storage volume template is generated for
Kaitlin Rupert
kaitlin at linux.vnet.ibm.com
Fri Dec 18 20:47:51 UTC 2009
Kaitlin Rupert wrote:
> Richard Maciel wrote:
>> On 12/11/2009 11:16 PM, Kaitlin Rupert wrote:
>>> # HG changeset patch
>>> # User Kaitlin Rupert<karupert at us.ibm.com>
>>> # Date 1260576721 28800
>>> # Node ID 2a52495fd1c2c47440b65f723366ccc3826f1548
>>> # Parent 5978eee0beb98fb1d2d8780376fc8edd825589d7
>>> [TEST] For Xen enabled systems, a storage volume template is
>>> generated for ...
>>>
>>> Both para virt guests and full virt guests.
>>>
>>> Signed-off-by: Kaitlin Rupert<karupert at us.ibm.com>
>>>
>>> diff -r 5978eee0beb9 -r 2a52495fd1c2
>>> suites/libvirt-cim/cimtest/ResourcePoolConfigurationService/10_create_storagevolume.py
>>>
>>> ---
>>> a/suites/libvirt-cim/cimtest/ResourcePoolConfigurationService/10_create_storagevolume.py
>>> Fri Dec 11 16:12:01 2009 -0800
>>> +++
>>> b/suites/libvirt-cim/cimtest/ResourcePoolConfigurationService/10_create_storagevolume.py
>>> Fri Dec 11 16:12:01 2009 -0800
>>> @@ -122,8 +122,15 @@
>>> if item['Address'] == exp_vol_path and item['PoolID'] ==
>>> dp_inst_id:
>>> dv_rasds.append(item)
>>>
>>> - if len(dv_rasds) != 4:
>>> - logger.error("Got '%s' StorageVolRASD's expected 4",
>>> len(dv_rasds))
>>> + exp_template_rasd = 4
>>> +
>>> + if virt == "Xen" or virt == "XenFV":
>>> + #There's one of each RASD type for both paravirt Xen and
>>> full virt Xen
>>> + exp_template_rasd = exp_template_rasd * 2
>>
>> Since you can have a system with only paravirt Xen installed, this
>> test will fail (it expects 8 template_rasd's when it should only
>> expect 4).
>>
>
> SDC generates both PV and FV templates; this happens regardless of
> whether the system supports FV. Is this test failing for you? On your
> PV-only Xen system, do you not see the FV templates?
>
Ack - my mistake here. This test is for storage volumes. Not for
assigning a disk to a guest. So in this context, PV and FV don't apply.
This patch doesn't address the issue. I'll send an updated version.
--
Kaitlin Rupert
IBM Linux Technology Center
kaitlin at linux.vnet.ibm.com
More information about the Libvirt-cim
mailing list