Re: [libvirt] [PATCH] qemu: Warn of restore with managed save being risky

On 12/31/19 5:42 PM, Michael Weiser wrote: Ah, I overlooked the "or force the operation" part of the commit message after reading the discussions in [1] and [2]. Instead of updating the commit message, I think you can update the error message to mention the '--force' option, e.g. : + virReportError(VIR_ERR_SNAPSHOT_REVERT_RISKY, "%s", + _("revert to snapshot while there is a managed " + "saved state will cause corruption when run, " + "remove saved state first or use the " + "--force option")); I also noticed that the man page for 'snapshot-revert' mentions two cases where the use of --force is required. One of them talks about snapshots created using old Libvirt versions. The other goes as follows: ---- (docs/manpages/virsh.rst) The other is the case of reverting from a running domain to an active state where a new hypervisor has to be created rather than reusing the existing hypervisor, because it implies drawbacks such as breaking any existing VNC or Spice connections; this condition happens with an active snapshot that uses a provably incompatible configuration, as well as with an inactive snapshot that is combined with the --start or --pause flag. ---- I am almost certain that scenario you're handling here is not covered by this excerpt. In this case, since you're adding a new '--force' condition, I believe a second patch adding this new '--force' condition in the documentation is a nice touch. Thanks, DHB