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(a)linux.vnet.ibm.com