]> xenbits.xensource.com Git - libvirt.git/commitdiff
Revert "storage: Better describe logical pool creation/definition parameters"
authorJán Tomko <jtomko@redhat.com>
Tue, 28 Mar 2017 07:45:10 +0000 (09:45 +0200)
committerJán Tomko <jtomko@redhat.com>
Tue, 28 Mar 2017 11:19:47 +0000 (13:19 +0200)
This reverts commit ca4515d2639057020c749470f390fe1f5981e91e
which also included a functional change that broke logical storage pools
not named after their volume groups.

src/conf/storage_conf.c
tools/virsh.pod

index 5213503c2edaf79bb81eb674748d6217d96afd6e..585ca71aaf6837dc8845358eee9cb88d3c6626e0 100644 (file)
@@ -760,14 +760,6 @@ virStoragePoolDefParseXML(xmlXPathContextPtr ctxt)
             if (VIR_STRDUP(ret->source.name, ret->name) < 0)
                 goto error;
         }
-        if (ret->type == VIR_STORAGE_POOL_LOGICAL &&
-            STRNEQ(ret->name, ret->source.name)) {
-                virReportError(VIR_ERR_XML_ERROR,
-                               _("for a logical pool, the pool name='%s' "
-                                 "must match the pool source name='%s'"),
-                               ret->name, ret->source.name);
-                goto error;
-        }
     }
 
     if ((options->flags & VIR_STORAGE_POOL_SOURCE_ADAPTER) &&
index 55b71a9b9f23f481dc07ed5453d9f39cc3be5308..43124ba55801e8f2854a22c0c3c1acc6f7442917 100644 (file)
@@ -3639,13 +3639,6 @@ follow-up command to build the pool. The I<--overwrite> and
 I<--no-overwrite> flags follow the same rules as B<pool-build>. If
 just I<--build> is provided, then B<pool-build> is called with no flags.
 
-For a "logical" pool only [I<--name>] needs to be provided. The [I<--name>]
-must match the Volume Group name for which the pool is being defined or
-created. The [I<--source-name>] if provided must match the Volume Group
-name. If not provided, one will be generated using the [I<--name>]. If
-provided the [I<--target>] is ignored and a target source is generated
-using the [I<--source-name>] (or as generated from the [I<--name>]).
-
 =item B<pool-define> I<file>
 
 Define an inactive persistent storage pool or modify an existing persistent one