Re: [libvirt] [PATCH 0/6] Improve backing store error reporting

On Thu, Sep 11, 2014 at 07:47:46PM +0200, Peter Krempa wrote: > Peter Krempa (6): > util: Add function to check if a virStorageSource is "empty" > qemu: Drop unused formatting of uuid > util: storage: Allow metadata crawler to report useful errors > qemu: Report better errors from broken backing chains > storage: Improve error message when traversing backing chains > qemu: Improve check for local storage > > src/libvirt_private.syms | 1 + > src/qemu/qemu_domain.c | 38 ++++++-------------------------------- > src/qemu/qemu_process.c | 5 ++--- > src/security/virt-aa-helper.c | 2 +- > src/storage/storage_driver.c | 40 +++++++++++++++++++++++++++++----------- > src/storage/storage_driver.h | 3 ++- > src/util/virstoragefile.c | 20 ++++++++++++++++++++ > src/util/virstoragefile.h | 1 + > tests/virstoragetest.c | 2 +- > 9 files changed, 63 insertions(+), 49 deletions(-) I don't suppose you found a way to test this? I'd like to, but it's difficult (especially without qemu:///session working as root).

I'll have to fire up a VM with a patched libvirtd at some point, but not today. Rich.