[Kimchi-devel] [PATCH] [Kimchi] Bug fix: Remove storage volume file while removing the storage volume

[Aline Manera]

When removing a storage volume, it was only removed from a libvirt
perspective, ie, the storage volume file kept in the system.
It may confuse user as he/she will not be able to create a new storage
volume with the same name from the storage volume removed before as the file
exists in the system.
To avoid it, remove the file from the system after removing it from a
libvirt perspective.

This issue was identified due a failure in the test suite.
2 files related to storage volume tests were kept on the system causing
errors when running the test suite multiple times.

To verify this patch, please, make sure to do not have any leftovers in the
'default' storage pool directory (/var/lib/libvirt/images).
After that, run the test suite mutliple times and you will see no
leftovers will be in the system when the test completes.

Signed-off-by: Aline Manera <alinefm at linux.vnet.ibm.com>
---
 model/storagevolumes.py | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/model/storagevolumes.py b/model/storagevolumes.py
index d010bcd..d5fdff9 100644
--- a/model/storagevolumes.py
+++ b/model/storagevolumes.py
@@ -373,6 +373,12 @@ class StorageVolumeModel(object):
             raise OperationFailed("KCHVOL0010E",
                                   {'name': name, 'err': e.get_error_message()})
 
+        try:
+            os.remove(pool_info['path'])
+        except OSError, e:
+            wok_log.error("Unable to delete storage volume file: %s."
+                          "Details: %s" % (pool_info['path'], e.message))
+
     def resize(self, pool, name, size):
         volume = StorageVolumeModel.get_storagevolume(pool, name, self.conn)
 
-- 
2.5.0