[libvirt] [PATCHv2] libvirt: add daemon itself as shutdown reason

Nikolay Shirokovskiy nshirokovskiy at virtuozzo.com
Wed Nov 14 06:51:38 UTC 2018



On 13.11.2018 23:36, John Ferlan wrote:
> From: Nikolay Shirokovskiy <nshirokovskiy at virtuozzo.com>
> 
> This patch introduces a new shutdown reason "daemon" in order
> to indicate that the daemon needed to force shutdown the domain
> as the best course of action to take at the moment.
> 
> This action would occur during reconnection when processing
> encounters an error once the monitor reconnection is successful.
> 
> Signed-off-by: Nikolay Shirokovskiy <nshirokovskiy at virtuozzo.com>
> Reviewed-by: John Ferlan <jferlan at redhat.com>
> ---
> 
>  Since I hadn't seen a yay or nay on my last response related to
>  changes - figured I'd post things as they are now just to make it
>  easier to determine whether the changes made were acceptible
>  This is essentially your original patch merged with my other
>  changes and the removal of the qemu_process.c change.
> 
>  include/libvirt/libvirt-domain.h | 2 ++
>  src/conf/domain_conf.c           | 3 ++-
>  src/qemu/qemu_process.c          | 6 +++++-
>  tools/virsh-domain-monitor.c     | 3 ++-
>  4 files changed, 11 insertions(+), 3 deletions(-)
> 

I thought you are going to push the patch after last letter)

ACK




More information about the libvir-list mailing list