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

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

 




On 13.11.2018 23:36, John Ferlan wrote:
> From: Nikolay Shirokovskiy <nshirokovskiy@xxxxxxxxxxxxx>
> 
> 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@xxxxxxxxxxxxx>
> Reviewed-by: John Ferlan <jferlan@xxxxxxxxxx>
> ---
> 
>  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

--
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