<br><tt><font size=2>libvirt-cim-bounces@redhat.com wrote on 2008-09-24
10:02:52:<br>
<br>
> > +    <br>
> > +    status, linux_cs = check_sblim(options.ip)<br>
> > +    if status == PASS:<br>
> > +        name = 'Linux_ComputerSystem'<br>
> > +        system = linux_cs[0]<br>
> > +    else:<br>
> <br>
> You'll want to verify that if check_sblim() passes, that enumerate
<br>
> HostSystem does not return an instance.</font></tt>
<br><tt><font size=2>  </font></tt>
<br><tt><font size=2>  Good catch. Add this verification in check_sblim()
function.</font></tt>
<br><tt><font size=2><br>
> <br>
> > +        try:<br>
> > +            hs = enumclass.enumerate(options.ip,
'HostSystem', <br>
> keys, options.virt)<br>
> > +            name = get_typed_class(options.virt,
'HostSystem')<br>
> >          <br>
> > -        if len(hs) != 1:<br>
> > -            logger.error("Expected
1 %s instance returned" % name)<br>
> > -            return FAIL<br>
> > +            if len(hs) != 1:<br>
> > +                logger.error("Expected
1 %s instance returned" % name)<br>
> > +                return
FAIL<br>
> <br>
> You'll want to return an XFAIL here.<br>
> <br>
> -- <br>
> Kaitlin Rupert<br>
> IBM Linux Technology Center<br>
> kaitlin@linux.vnet.ibm.com<br>
> <br>
> _______________________________________________<br>
> Libvirt-cim mailing list<br>
> Libvirt-cim@redhat.com<br>
> https://www.redhat.com/mailman/listinfo/libvirt-cim<br>
</font></tt>