[libvirt] [PATCH 0/2] qemu: Enable -blockdev support (blockdev-add saga)

Martin Kletzander mkletzan at redhat.com
Fri Sep 13 21:35:47 UTC 2019


On Fri, Sep 13, 2019 at 02:43:53PM +0200, Peter Krempa wrote:
>To my knowledge, everything in libvirt is now prepared to fully use
>-blockdev way to configure disks in qemu.
>
>There is one known qemu bug though: Internal snapshots don't work with
>-blockdev:
>
>https://bugzilla.redhat.com/show_bug.cgi?id=1658981
>
>Since I can't in good faith ask for merging this patchset yet I'd like
>to give it some more testing I'm suggesting that we push it and revert
>it during freeze or add a capability check once qemu is fixed.
>

I am very much in favour of getting some testing before releasing such big
changes.  Any way of not including this in release tarballs is fine, but we
should also not completely give up on non-blockdev approach until we do actually
release it, if only because by having this in for the whole development period
only to disable it for release would mean that we are releasing something we did
not test at all for a whole cycle.

Yeah, I know the above does not give a clear answer or solution, but that is
because I am ambivalent about it.  But I'm not against having this in git to
test it.

Since the change itself is fairly trivial you can consider it
Reviewed-but-not-tested-by: me =)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20190913/9a16ee08/attachment-0001.sig>


More information about the libvir-list mailing list