[libvirt] [PATCH] qemu: Report nicer error if decompression of save image fails
Daniel P. Berrange
berrange at redhat.com
Thu Feb 7 17:12:36 UTC 2013
On Thu, Feb 07, 2013 at 06:09:50PM +0100, Peter Krempa wrote:
> This patch is a little more specific what failed while trying to restore
> a guest from a save image.
> ---
> src/qemu/qemu_driver.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
> index 979a027..c96ba7a 100644
> --- a/src/qemu/qemu_driver.c
> +++ b/src/qemu/qemu_driver.c
> @@ -4903,6 +4903,8 @@ qemuDomainSaveImageStartVM(virConnectPtr conn,
>
> if (virCommandWait(cmd, NULL) < 0) {
> qemuProcessStop(driver, vm, VIR_DOMAIN_SHUTOFF_FAILED, 0);
> + virReportError(VIR_ERR_OPERATION_FAILED, "%s",
> + _("Decompression of save image failed"));
virCommandWait will already be reporting an error which you're
now overwriting. Historically this has been a very bad thing
for us when troubleshooting because you're throwing away the
more specific details, in favour of a generic error message
with no detail.
Daniel.
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
More information about the libvir-list
mailing list