Re: [PATCH 00/14] introduce virDomainSetLifecycleAction() API

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Oct 18, 2017 at 04:39:10PM -0400, John Ferlan wrote:
> 
> 
> On 10/16/2017 07:06 AM, Pavel Hrdina wrote:
> > Pavel Hrdina (14):
> >   conf: rename lifecycle enum values to correspond with typedef keyword
> >   conf: rename virDomainLifecycleAction enum functions
> >   conf: introduce virDomainLifecycle enum to list all lifecycle types
> >   conf: merge virDomainLifecycleCrashAction with
> >     virDomainLifecycleAction
> >   qemu: pass virDomainObjPtr to qemuBuildCommandLine
> >   qemu: pass priv data to qemuBuildMonitorCommandLine
> >   qemu: pass priv data to qemuBuildPMCommandLine
> >   qemu: pass priv data to qemuBuildMasterKeyCommandLine
> >   qemu: pass priv data instead of qemuCaps and autoNodeset
> >   lib: introduce virDomainSetLifecycleAction() API
> >   virsh: introduce set-lifecycle-action command
> >   qemu: move detection whether to use -no-reboot to qemu_domain
> >   qemu: send allowReboot in migration cookie
> >   qemu: implement virDomainSetLifecycleAction() API
> > 
> >  docs/formatdomain.html.in        |   6 +++
> >  include/libvirt/libvirt-domain.h |  28 ++++++++++
> >  src/conf/domain_conf.c           |  84 ++++++++++++++++++++++++-----
> >  src/conf/domain_conf.h           |  26 ++-------
> >  src/driver-hypervisor.h          |   7 +++
> >  src/libvirt-domain.c             |  60 +++++++++++++++++++++
> >  src/libvirt_private.syms         |   7 ++-
> >  src/libvirt_public.syms          |   5 ++
> >  src/libxl/libxl_conf.c           |  43 ++++-----------
> >  src/libxl/libxl_domain.c         |  40 +++++++-------
> >  src/lxc/lxc_native.c             |   6 +--
> >  src/qemu/qemu_command.c          |  93 +++++++++++++-------------------
> >  src/qemu/qemu_command.h          |  11 +---
> >  src/qemu/qemu_domain.c           |  40 ++++++++++++++
> >  src/qemu/qemu_domain.h           |   9 ++++
> >  src/qemu/qemu_driver.c           | 112 +++++++++++++++++++++++++++++++++++----
> >  src/qemu/qemu_migration.c        |   7 ++-
> >  src/qemu/qemu_migration_cookie.c |  25 ++++++++-
> >  src/qemu/qemu_migration_cookie.h |   5 ++
> >  src/qemu/qemu_parse_command.c    |   8 +--
> >  src/qemu/qemu_process.c          |  53 ++++++++++++------
> >  src/remote/remote_driver.c       |   1 +
> >  src/remote/remote_protocol.x     |  14 ++++-
> >  src/test/test_driver.c           |   8 +--
> >  src/vmx/vmx.c                    |   6 +--
> >  src/vz/vz_sdk.c                  |  12 ++---
> >  src/xenapi/xenapi_utils.c        |  40 +++++++-------
> >  src/xenapi/xenapi_utils.h        |   4 +-
> >  src/xenconfig/xen_common.c       |  12 ++---
> >  src/xenconfig/xen_sxpr.c         |  18 +++----
> >  tests/qemuxml2xmltest.c          |   3 +-
> >  tools/virsh-domain.c             | 102 +++++++++++++++++++++++++++++++++++
> >  tools/virsh.pod                  |   7 +++
> >  33 files changed, 658 insertions(+), 244 deletions(-)
> > 
> 
> Reviewed-by: John Ferlan <jferlan@xxxxxxxxxx>
> (series)
> 
> As long as you take care of the couple things I noted.

Thanks for review, I've fixed the issues and pushed it.

Pavel

Attachment: signature.asc
Description: PGP signature

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

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux