[libvirt] [PATCH] Don't check volume format for lvm backing store

[Cole Robinson <crobinso@xxxxxxxxxx>]

With the recently added COW support, we were trying to validate the
passed logical volume's format, when there won't ever be one pulled from
the xml (logical volumes don't have any valid format values).

Just remove this check, and let any validation in this manner be done by
the actual call to 'lvcreate'.

Thanks,
Cole

diff --git a/src/storage_backend_logical.c b/src/storage_backend_logical.c
index 702a191..cbd2765 100644
--- a/src/storage_backend_logical.c
+++ b/src/storage_backend_logical.c
@@ -582,12 +582,6 @@ virStorageBackendLogicalCreateVol(virConnectPtr conn,
     const char **cmdargv = cmdargvnew;
 
     if (vol->backingStore.path) {
-        if (vol->backingStore.format !=
-            VIR_STORAGE_POOL_LOGICAL_LVM2) {
-            virStorageReportError(conn, VIR_ERR_INTERNAL_ERROR, "%s",
-                                  _("LVM snapshots must be backed by another LVM volume"));
-            return -1;
-        }
         cmdargv = cmdargvsnap;
     }
 

--
Libvir-list mailing list
Libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list