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