[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] dump: introduce dump-cancel QMP command
From: |
Markus Armbruster |
Subject: |
Re: [PATCH] dump: introduce dump-cancel QMP command |
Date: |
Thu, 21 Jul 2022 12:57:23 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) |
Daniel P. Berrangé <berrange@redhat.com> writes:
> On Thu, Jul 21, 2022 at 02:21:18PM +0800, Hogan Wang via wrote:
>> There's no way to cancel the current executing dump process, lead to the
>> virtual machine manager daemon((e.g. libvirtd) cannot restore the dump
>> job after daemon restart.
>>
>> Add the 'cancelling' and 'cancelled' dump states.
>>
>> Use 'dump-cancel' qmp command Set the dump state as 'cancelling'.
>> The dump process check the 'cancelling' state and break loops.
>> The 'cancelled' state mark the dump process cancelled success.
>
> On the one hand this patch is fairly simple which is obviously
> desirable.
>
> On the other hand though, this feels like it is further re-inventing
> the jobs concept.
>
> IMHO ideally the 'dump' command probably ought to get a 'job-id'
> parameter, and integrate with the generic background jobs framework.
> This would unlock the ability to use existing commands like
> 'job-cancel', 'job-pause', 'job-resume', 'queyr-jobs' to interact
> with it.
Seconded.
Hogan Wang, would you be interested in rebasing the dump feature onto
the jobs infrastructure?