[libvirt] [PATCH] qemuDomainObjStart: Warn on corrupted image

[Eric Blake]

On 11/11/2013 08:58 AM, Peter Krempa wrote:
> On 11/11/13 16:48, Michal Privoznik wrote:
>> If the managedsave image is corrupted, e.g. the XML part is, we fail to
>> parse it and throw an error, e.g.:
>>
>> error: Failed to start domain jms8
>> error: XML error: missing security model when using multiple labels
>>
>> This is okay, as we can't really start the machine and avoid undefined
>> qemu behaviour. On the other hand, the error message doesn't give a
>> clue to users what should they do. The consensus here would be to thrown
> 
> Well a normal user won't check the logs as a first operation. This is a
> very good example place where libvirt would really benefit from having
> "stacked" error messages as the low level message doesn't give the user
> a clue about what happened.
> 
>> a warning to logs saying "Hey, you've got a corrupted file".
>>
>> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
>> ---
>>  src/qemu/qemu_driver.c | 5 ++++-
>>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> I'm NOT ACKing this right away as I personally don't like the change of
> behavior to report an error in case of a corrupt managed save. We will
> now require the users to manually delete the corrupted file instead.

This patch had no change in behavior other than an added log message.
The problem is that we have pre-existing different behavior for two
different cases of broken save images: behaving differently for broken
XML (requiring the user to edit the corrupted file, or manually delete
it) than we are for working XML but broken image (where we log a
message, but otherwise pretend the save file never existed).

> 
> Peter
> 
> 
> 
> 
> 
> --
> libvir-list mailing list
> libvir-list at redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list
> 

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 621 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20131111/9f78dbfd/attachment-0001.sig>