Currently, the maximum tolerable downtime for a domain being migrated is
write-only. This patch implements a way to query that value nondestructively.

Changes from [v1]:

    * Fixed formatting and style problems
    * Add additional JSON error check 
    * Better patch submission mail, I hope

[v1} https://www.redhat.com/archives/libvir-list/2017-July/msg00908.html

Scott Garfinkle (3):
    migrate-getmaxdowntime command qemu side
    migrate-getmaxdowntime public symbols and table indices
    migrate-getmaxdowntime local/libvirt enablement

 src/qemu/qemu_driver.c       | 45 ++++++++++++++++++++++++++++++++++++++++++++
 src/qemu/qemu_monitor.h      |  3 +++
 src/qemu/qemu_monitor_json.c | 40 +++++++++++++++++++++++++++++++++++++++
 src/qemu/qemu_monitor_json.h |  3 +++
 src/libvirt_public.syms      |  4 ++++
 src/remote/remote_protocol.x | 16 +++++++++++++++-
 src/remote_protocol-structs  |  8 ++++++++
 include/libvirt/libvirt-domain.h |  4 ++++
 src/driver-hypervisor.h          |  6 ++++++
 src/libvirt-domain.c             | 45 +++++++++++++++++++++++++++++++++++++++
 src/qemu/qemu_monitor.c          | 12 +++++++++++
 src/remote/remote_driver.c       |  1 +
 tools/virsh-domain.c             | 46 ++++++++++++++++++++++++++++++++++++++++
 tools/virsh.pod                  | 18 ++++++++++++++++
 14 files changed, 240 insertions(+), 1 deletion(-)

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

On Mon, Jul 24, 2017 at 11:21:43 -0500, seg@us.ibm.com wrote:
> 
> 
> Currently, the maximum tolerable downtime for a domain being migrated is
> write-only. This patch implements a way to query that value nondestructively.
> 
> Changes from [v1]:
> 
>     * Fixed formatting and style problems
>     * Add additional JSON error check 
>     * Better patch submission mail, I hope
> 
> [v1} https://www.redhat.com/archives/libvir-list/2017-July/msg00908.html
> 
> Scott Garfinkle (3):
>     migrate-getmaxdowntime command qemu side
>     migrate-getmaxdowntime public symbols and table indices
>     migrate-getmaxdowntime local/libvirt enablement

I only skimmed through this series, so only a few high level notes:

The series is supposed to successfully compile after each patch, which
means you need to shuffle the code between patches a bit. The public
API, RPC protocol, and driver-hypervisor.h changes should go first. The
two additional patches can go in any order, either qemu implementation
first followed by virsh changes or you can change virsh first. However,
migrate-getmaxdowntime is a virsh command, the patch subjects should
really talk about the API name instead. So, for example:

    1/3 Add virDomainMigrateGetMaxDowntime public API
    2/3 qemu: Implement virDomainMigrateGetMaxDowntime
    3/3 virsh: Add support for virDomainMigrateGetMaxDowntime

In the qemu driver we already have a monitor API for querying migration
parameters (qemuMonitorJSONGetMigrationParams) and we should extend it
to support downtime-limit rather than adding a new monitor API.

Jirka

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