> No complaints on this patch.. just a question. =) Do you plan on
> updating this test case to support deletion of other disk pool types?
> All the pool types should be identical. So if you plan on using
> different test cases, it would be a good idea to put parts of this
> test case into a function (or functions) at some point later on.
>
Yes we should be supporting the deletion of the other disk / net types.
When that happens we can see if we need to write different functions.
For now the DeleteResourcePool() has atleast one test for verification.
Do you think we should immediately implement the deletion of the other
types as well ?
No, this is fine for now. I was just curious how you were planning to
approach it. But this sounds fine to me.
Thanks!
--
Kaitlin Rupert
IBM Linux Technology Center
kaitlin(a)linux.vnet.ibm.com