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 What are the chances of that bug actually getting fixed any time in the forseeable future ? savevm has been a pain for a long time since it blocks QEMU and we've been waiting for it to be fixedd by QEMU devs for at least 7-8 years now, and I don't see anyone from QEMU block maintainers commenting on this new BZ to indicate they're going to fix this latest problem. > 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'd only be in favour of enabling it if there is some clear time frame on whicht hat bug above is likely to be fixed. I'm guesing it is pretty unlikely to be fixed before our freeze, so I thinking speculatively enabling it is premature. > Any other ideas? We already have the option you added to allow capabilities to be force enabled for people to test with, though admittedly that is going to get fairly limited usage. Regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :| -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list