Dan Smith wrote:
KR> Is this test really checking to make sure EnumInstances() doesn't KR> return an error? If so, the name of the test is misleading. And KR> other tests cover this already (CS 01_enum.py).
Well, I looked at the current test, as well as the original version of it and I'm all confused now. I thought it was doing something different than it is (and was). So, as you say, I think it makes sense to remove Xen and XenFV from the list of runnable platforms and let this test ensure that the providers can handle the case where there are no domains on the system for LXC and KVM.
Sure, that works for me. Right now, I have Xen and XenFV in the list (the test should always skip on them). I'll send an updated patch that fixes this test so that it is KVM and LXC only. Thanks! -- Kaitlin Rupert IBM Linux Technology Center kaitlin@linux.vnet.ibm.com