[PATCH] qemu: validate: Allow 'preserve' action for on_crash lifecycle action
Christian Borntraeger
borntraeger at de.ibm.com
Wed Sep 15 11:21:57 UTC 2021
On 15.09.21 13:14, Peter Krempa wrote:
> In fact keeping the VM around for debugging is a desirable configuration
> and actually the implementation has no code as we keep the VM around.
>
> Remove the validation and add a note that it's actually used.
>
> Fixes: b1b85a475fb251b9068b75f629479f5c452f1b43
> Reported-by: Christian Borntraeger <borntraeger at de.ibm.com>
Acked-by: Christian Borntraeger <borntraeger at de.ibm.com>
> Signed-off-by: Peter Krempa <pkrempa at redhat.com>
> ---
> src/qemu/qemu_driver.c | 1 +
> src/qemu/qemu_validate.c | 5 ++---
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
> index dfc27572c4..6ae678b165 100644
> --- a/src/qemu/qemu_driver.c
> +++ b/src/qemu/qemu_driver.c
> @@ -3647,6 +3647,7 @@ processGuestPanicEvent(virQEMUDriver *driver,
> break;
>
> case VIR_DOMAIN_LIFECYCLE_ACTION_PRESERVE:
> + /* the VM is kept around for debugging */
> break;
>
> default:
> diff --git a/src/qemu/qemu_validate.c b/src/qemu/qemu_validate.c
> index 9d93f373ab..6b685881a8 100644
> --- a/src/qemu/qemu_validate.c
> +++ b/src/qemu/qemu_validate.c
> @@ -1083,10 +1083,9 @@ qemuValidateLifecycleAction(virDomainLifecycleAction onPoweroff,
> /* The qemu driver doesn't yet implement any meaningful handling for
> * VIR_DOMAIN_LIFECYCLE_ACTION_PRESERVE */
> if (onPoweroff == VIR_DOMAIN_LIFECYCLE_ACTION_PRESERVE ||
> - onReboot == VIR_DOMAIN_LIFECYCLE_ACTION_PRESERVE ||
> - onCrash == VIR_DOMAIN_LIFECYCLE_ACTION_PRESERVE) {
> + onReboot == VIR_DOMAIN_LIFECYCLE_ACTION_PRESERVE) {
> virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
> - _("qemu driver doesn't support the 'preserve' action for 'on_reboot'/'on_poweroff'/'on_crash'"));
> + _("qemu driver doesn't support the 'preserve' action for 'on_reboot'/'on_poweroff'"));
> return -1;
> }
>
More information about the libvir-list
mailing list