Re: [libvirt] [PATCH] qemu: Do not override config XML in case of snapshot revert

On 5/14/19 8:37 AM, Michal Privoznik wrote: Sadly, if we were to modify snapshot XML to store both the active and inactive XML with a snapshot, all existing snapshots of a running guest would be incomplete (right now, the snapshot XML only allows the storage of a single <domain> sub-element). I think the best we can do is store an active XML with a snapshot of an active domain, and an inactive XML with a snapshot of an offline domain. Reverting to an offline snapshot only has to worry about inactive XML; reverting to an online snapshot needs the active XML to properly restore the domain, but we then have to decide whether to also update the inactive XML to match the fact that the active XML has changed. So changing both inactive and active XML to describe the same state during a revert to a snapshot of an active domain seems like the best approach to me, short of rewriting snapshot XML to store two different domain definitions at once. -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3226 Virtualization: qemu.org | libvirt.org