<br><tt><font size=2>libvirt-cim-bounces@redhat.com wrote on 2009-01-17
05:17:57:<br>
<br>
> # HG changeset patch<br>
> # User Kaitlin Rupert <karupert@us.ibm.com><br>
> # Date 1232140666 28800<br>
> # Node ID 2500265f6c78b23eaddaffe3595bb2dee127cc13<br>
> # Parent  57f3bc7c3105bb779b41e19b1c03a1e5f214cec1<br>
> [TEST] Add some error checking when detecting hypervisor version<br>
> <br>
> Signed-off-by: Kaitlin Rupert <karupert@us.ibm.com><br>
> <br>
> diff -r 57f3bc7c3105 -r 2500265f6c78 suites/libvirt-<br>
> cim/lib/XenKvmLib/reporting.py<br>
> --- a/suites/libvirt-cim/lib/XenKvmLib/reporting.py   Fri Jan
16 11:<br>
> 51:26 2009 -0800<br>
> +++ b/suites/libvirt-cim/lib/XenKvmLib/reporting.py   Fri Jan
16 13:<br>
> 17:46 2009 -0800<br>
> @@ -50,8 +50,11 @@<br>
>                  libvirt_ver
= virsh_ver.splitlines()[0].split()[4]<br>
>  <br>
>              if virsh_ver.splitlines()[3].find("hypervisor"):<br>
> -                hyp_ver =
virsh_ver.splitlines()[3].split("hypervisor")[1]<br>
> -                hyp_ver =
hyp_ver.split(": ")[1]<br>
> +                tok = virsh_ver.splitlines()[3].split("hypervisor")<br>
> +                if len(tok)
> 1:<br>
> +                    tok
= tok[1].split(": ")<br>
> +                    if
len(tok) > 1:<br>
> +                    
   hyp_ver = tok[1]<br>
>  <br>
>      return libvirt_ver, hyp_ver<br>
>  <br>
> +1 from me. </font></tt>
<br><tt><font size=2>  It's better to print out error log if len(tok)<=
1.</font></tt>
<br><tt><font size=2><br>
> _______________________________________________<br>
> Libvirt-cim mailing list<br>
> Libvirt-cim@redhat.com<br>
> https://www.redhat.com/mailman/listinfo/libvirt-cim<br>
</font></tt>