[libvirt] bug: try to take disk snapshot for LVM2 Volume

Eric Blake eblake at redhat.com
Thu Nov 17 13:45:03 UTC 2011


On 11/17/2011 02:11 AM, shu ming wrote:
> Checking the source of the disk will exclude the case when the source
> disk is block device while the snapshot target is given manually by the
> users with "snapshot-create --xmlfile" or "virsh snapshot-create-as
> --diskspec".   Why not check the snapshot directory where it will be
> created?

Because it is assumed that if you are bothering to take the time to pass
--xmlfile or --diskspec, then you are also aware of the issues of that
file name and can deal with the consequences of naming a bogus location
for the resulting qcow2 file.  The patch below is only a crutch to make
the default behavior, with no explicit name, less likely to cause
confusion; while still leaving the user with the full power of the tool
when they go with explicit file names.

>>>>
>>>> * src/conf/domain_conf.c (virDomainSnapshotAlignDisks): Only
>>>> generate names if backing file exists as regular file.
>>>> Reported by MATSUDA Daiki.

-- 
Eric Blake   eblake at redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 620 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20111117/6b0eb8dd/attachment-0001.sig>


More information about the libvir-list mailing list