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