Re: [libvirt] [PATCH] Drop virStorageBackendLogicalMatchPoolSource

On 06/15/2016 01:19 PM, Ján Tomko wrote: > Regression introduced by commit 71b803a for [1] that prevents starting up > a logical pool created with <source><device path=''></source> > after it has been moved to a different physical volume. Is there a bug for this? XML examples? Is an empty source device path string a valid value? Reading http://libvirt.org/formatstorage.html doesn't give me that impression.

"device Provides the source for pools backed by physical devices (pool types fs, logical, disk, iscsi, zfs). May be repeated multiple times depending on backend driver. Contains a required attribute path which is either the fully qualified path to the block device node or for iscsi the iSCSI Qualified Name (IQN). Since 0.4.1" > > For logical pools <source><name> contains the name of the volume group > and uniquely identifies the VG on the host. > > This also speeds up startup for pools that do not have any <device>s > specified. > > [1] https://bugzilla.redhat.com/show_bug.cgi?id=1025230 > --- > src/storage/storage_backend_logical.c | 104 +--------------------------------- > 1 file changed, 2 insertions(+), 102 deletions(-) > This essentially reverts a patch that was used to resolve a bz without a patch to resolve the issue in the bug. What's the proposal/patch to resolve the issue from the bug?