On 01/28/2014 11:29 AM, Ramon Medeiros wrote:

Well,

you can also test using virsh. Libvirt is not removing the vg and the pv when you delete a logical storage pool.

The question is: is it better to fix libvirt, submiting a patch for they, or just remove the vg and pv on kimchi?

I don't think it is a libvirt problem.
Deleting a storage pool should not affect the system as storage pool is just a concept.

From kimchi perspective, we should allow the user create the logical storage pool with any name independent
if it was created earlier or not

What we need to do is investigating if there is a way to create a logical pool with name "my-logical-pool"
and the vg with a unique key, like "my-logical-%(uuid)s"
So we guarantee the logical pool name won't affect the vg name.