[Libvirt-cim] [PATCH] [TEST] For Xen enabled systems, a storage volume template is generated for

[Richard Maciel]

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).

> +
> +    if len(dv_rasds) != exp_template_rasd:
> +        logger.error("Got '%s' StorageVolRASD's expected %s", len(dv_rasds),
> +                     exp_template_rasd)
>           return FAIL
>
>       return PASS
>
> _______________________________________________
> Libvirt-cim mailing list
> Libvirt-cim at redhat.com
> https://www.redhat.com/mailman/listinfo/libvirt-cim


-- 
Richard Maciel, MSc
IBM Linux Technology Center
rmaciel at linux.vnet.ibm.com