<br><font size=2 face="sans-serif">+1 from me.</font>
<br>
<br>
<br><tt><font size=2>libvirt-cim-bounces@redhat.com wrote on 2009-04-21
02:19:13:<br>
<br>
> # HG changeset patch<br>
> # User Kaitlin Rupert <karupert@us.ibm.com><br>
> # Date 1240002683 25200<br>
> # Node ID eea660ff3ad6e589f76bf17b3697d30465e54398<br>
> # Parent  ced161a8198115797a6036f3f22e02d234439a76<br>
> [TEST] (#2) Improve enum volumes<br>
> <br>
> The providers don't return a template RASD for volumes that libvirt
is<br>
> unable to get volume info for.  So the testsuite needs to do
the same.<br>
> <br>
> Also, don't use a hardcoded disk pool name - use the default value
<br>
> from const.py<br>
> <br>
> Updates:<br>
>   -Remove test code<br>
>   -Allow caller to specify pool name, or use default pool name
if one isn't<br>
>    specified<br>
> <br>
> Signed-off-by: Kaitlin Rupert <karupert@us.ibm.com><br>
> <br>
> diff -r ced161a81981 -r eea660ff3ad6 suites/libvirt-cim/lib/XenKvmLib/pool.py<br>
> --- a/suites/libvirt-cim/lib/XenKvmLib/pool.py   Wed Apr 15 20:19:312009
-0700<br>
> +++ b/suites/libvirt-cim/lib/XenKvmLib/pool.py   Fri Apr 17 14:11:232009
-0700<br>
> @@ -24,7 +24,7 @@<br>
>  from CimTest.Globals import logger<br>
>  from CimTest.ReturnCodes import PASS, FAIL<br>
>  from XenKvmLib.classes import get_typed_class<br>
> -from XenKvmLib.const import get_provider_version <br>
> +from XenKvmLib.const import get_provider_version, default_pool_name
<br>
>  from XenKvmLib.enumclass import EnumInstances<br>
>  from VirtLib.utils import run_remote<br>
>  from XenKvmLib.xm_virt_util import virt2uri<br>
> @@ -80,16 +80,20 @@<br>
>  <br>
>      return pool_insts, PASS<br>
>  <br>
> -def enum_volumes(virt, server):<br>
> +def enum_volumes(virt, server, pooln=default_pool_name):<br>
>      volume = 0<br>
>      cmd = "virsh -c %s vol-list %s | sed -e '1,2
d' -e '$ d'" % \<br>
> -          (virt2uri(virt), 'cimtest-diskpool')<br>
> +          (virt2uri(virt), default_pool_name)<br>
>      ret, out = run_remote(server ,cmd)<br>
>      if ret != 0:<br>
>          return None<br>
>      lines = out.split("\n")<br>
>      for line in lines:<br>
> -        volume = volume + 1<br>
> +        vol = line.split()[0]   <br>
> +        cmd = "virsh -c %s vol-info --pool
%s %s" % <br>
> (virt2uri(virt), pooln, vol)<br>
> +        ret, out = run_remote(server ,cmd)<br>
> +        if ret == 0:<br>
> +            volume = volume + 1<br>
>  <br>
>      return volume<br>
>  <br>
> <br>
> _______________________________________________<br>
> Libvirt-cim mailing list<br>
> Libvirt-cim@redhat.com<br>
> https://www.redhat.com/mailman/listinfo/libvirt-cim<br>
</font></tt>