Re: [Qemu-devel] [RFC v4 21/21] blockjobs: add manual_mgmt option to transactions

[Kevin Wolf <kwolf@redhat.com>]

Am 27.02.2018 um 21:24 hat Eric Blake geschrieben:
> On 02/23/2018 05:51 PM, John Snow wrote:
> > This allows us to easily force the option for all jobs belonging
> > to a transaction to ensure consistency with how all those jobs
> > will be handled.
> > 
> > This is purely a convenience.
> > 
> > Signed-off-by: John Snow <jsnow@redhat.com>
> > ---
> 
> > +++ b/qapi/transaction.json
> > @@ -79,7 +79,8 @@
> >   ##
> >   { 'struct': 'TransactionProperties',
> >     'data': {
> > -       '*completion-mode': 'ActionCompletionMode'
> > +       '*completion-mode': 'ActionCompletionMode',
> > +       '*manual-mgmt': 'bool'
> 
> Missing QAPI documentation (what you have elsewhere in the C code can
> probably be copied here, though).
> 
> The UI aspect makes sense (I can declare one manual at the transaction level
> instead of multiple manual declarations per member level within the
> transaction).

I'm not so sure if I like the interface, it duplicates functionality in
two places.

At th very least I would make job creation without BLOCK_JOB_MANUAL an
error if the transaction requires it instead of silently overriding the
option that was given to the individual job. But honestly, it might be
better to just leave this one away.

Kevin