[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH v2] snapshot: fix rollback failure in transaction mode

On 09/13/2012 03:41 AM, Guannan Ren wrote:
> BZ:https://bugzilla.redhat.com/show_bug.cgi?id=843372
> when qemu supports the 'transaction' monitor command,
> and libvirt's --reuse-ext flag was not specified, libvirt created
> a stub file with zero size in first place. After the failure of
> QEMU transaction command performing qcow2 snapshots on more than
> one drives, the stub file is left behind with non-empty
> by the QEMU transaction command.
> In order to unlink the file, the patch removes the file size checking.

I was still hoping you'd list the reproduction steps here in the commit
message, rather than making me chase down a link.  Remember, a year down
the road, when reading 'git log', the more self-contained the log is,
the easier it will be to understand WHY the patch is appropriate.

Conditional ACK - no need to post a v3; I'm okay if you push once you
amend the commit message.  Keep the mention of the BZ number, but also
add this text (from your previous message).

     qemu left non-empty file.
     1, Create a qemu instance with two drive images of qcow2 type (root
          /usr/libexec/qemu-kvm -m 1024 -smp 1 -name "rhel6u1" \
-qmp stdio

     2, Initialize qemu qmp

     3, Remove the second drive image file
         rm -f /var/lib/libvirt/images/secondqcow2

     4, Run 'transaction' command with snapshot qemu commands in.

     5,  Got the error as follows:
         {"id": "libvirt-6", "error": {"class": "OpenFileFailed",
"desc": "Could not open '/var/lib/libvirt/images/secondqcow2-snapshot.img'",
                                  "data": {"filename":

     6, List first newly-created snapshot file:
         -rw-r--r--. 1 root root     262144 Sep 13 11:43

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

Attachment: signature.asc
Description: OpenPGP digital signature

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]