[v2] qemu: report block job errors from qemu to the user

[libvirt] [PATCH v3 REBASE v2 0/2] qemu: report block job errors from qemu to the user

Nikolay Shirokovskiy posted 2 patches 7 years, 6 months ago

Diff against v3
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/1509107843-275234-1-git-send-email-nshirokovskiy@virtuozzo.com

src/qemu/qemu_blockjob.c     | 14 +++++++++--
src/qemu/qemu_blockjob.h     |  3 ++-
src/qemu/qemu_domain.c       | 10 +++++++-
src/qemu/qemu_domain.h       |  1 +
src/qemu/qemu_driver.c       |  4 ++--
src/qemu/qemu_migration.c    | 55 +++++++++++++++++++++++++++++++-------------
src/qemu/qemu_monitor.c      |  5 ++--
src/qemu/qemu_monitor.h      |  4 +++-
src/qemu/qemu_monitor_json.c |  4 +++-
src/qemu/qemu_process.c      |  4 ++++
10 files changed, 78 insertions(+), 26 deletions(-)

Expand all Fold all

[libvirt] [PATCH v3 REBASE v2 0/2] qemu: report block job errors from qemu to the user

Posted by Nikolay Shirokovskiy 7 years, 6 months ago

So that you can see nice report on migration:

"error: operation failed: migration of disk sda failed: No space left on device"

diff from v2:
============
1. split into 2 patches
2. change formal documentation where it is present accordingly
3. add variable initialization for safety

Nikolay Shirokovskiy (2):
  qemu: prepare blockjob complete event error usage
  qemu: report drive mirror errors on migration

 src/qemu/qemu_blockjob.c     | 14 +++++++++--
 src/qemu/qemu_blockjob.h     |  3 ++-
 src/qemu/qemu_domain.c       | 10 +++++++-
 src/qemu/qemu_domain.h       |  1 +
 src/qemu/qemu_driver.c       |  4 ++--
 src/qemu/qemu_migration.c    | 55 +++++++++++++++++++++++++++++++-------------
 src/qemu/qemu_monitor.c      |  5 ++--
 src/qemu/qemu_monitor.h      |  4 +++-
 src/qemu/qemu_monitor_json.c |  4 +++-
 src/qemu/qemu_process.c      |  4 ++++
 10 files changed, 78 insertions(+), 26 deletions(-)

-- 
1.8.3.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH v3 REBASE v2 0/2] qemu: report block job errors from qemu to the user

Posted by Nikolay Shirokovskiy 7 years, 6 months ago

ping

On 27.10.2017 15:37, Nikolay Shirokovskiy wrote:
> So that you can see nice report on migration:
> 
> "error: operation failed: migration of disk sda failed: No space left on device"
> 
> diff from v2:
> ============
> 1. split into 2 patches
> 2. change formal documentation where it is present accordingly
> 3. add variable initialization for safety
> 
> Nikolay Shirokovskiy (2):
>   qemu: prepare blockjob complete event error usage
>   qemu: report drive mirror errors on migration
> 
>  src/qemu/qemu_blockjob.c     | 14 +++++++++--
>  src/qemu/qemu_blockjob.h     |  3 ++-
>  src/qemu/qemu_domain.c       | 10 +++++++-
>  src/qemu/qemu_domain.h       |  1 +
>  src/qemu/qemu_driver.c       |  4 ++--
>  src/qemu/qemu_migration.c    | 55 +++++++++++++++++++++++++++++++-------------
>  src/qemu/qemu_monitor.c      |  5 ++--
>  src/qemu/qemu_monitor.h      |  4 +++-
>  src/qemu/qemu_monitor_json.c |  4 +++-
>  src/qemu/qemu_process.c      |  4 ++++
>  10 files changed, 78 insertions(+), 26 deletions(-)
> 

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list