
Dan Smith wrote:
DK> Why don't we just remove Dom-0 from the list and then carry on DK> with the test case for Xen and XenFV ?
Agreed. At one point, this test did exactly that, and worked on Xen systems with no (other) guests defined.
My thinking was that the purpose was to verify the EnumInstance() of CS returned no instances when no guests were defined on the system. I'm not sure I understand the purpose of testing with no other guests defined on Xen, because Dom0 is treated as a guest by the providers. The original code was: cs = computersystem.enumerate(options.ip, options.virt) if cs.__class__ == str: logger.error("Got error instead of empty list: %s" % cs) status = FAIL The error message here is misleading - EnumInstances() won't return an empty list on Xen because an instance for Dom0 is returned. And we're not really checking for an empty list anyway. Is this test really checking to make sure EnumInstances() doesn't return an error? If so, the name of the test is misleading. And other tests cover this already (CS 01_enum.py). -- Kaitlin Rupert IBM Linux Technology Center kaitlin@linux.vnet.ibm.com