Re: [PATCH for-6.0] migration: Drop redundant query-migrate result @blocked

[Dr. David Alan Gilbert]

* Peter Maydell (peter.maydell@linaro.org) wrote:
> On Mon, 19 Apr 2021 at 17:27, Markus Armbruster <armbru@redhat.com> wrote:
> >
> > Result @blocked is true when and only when result @blocked-reasons is
> > present.  It's always non-empty when present.  @blocked is redundant;
> > drop.
> >
> > Signed-off-by: Markus Armbruster <armbru@redhat.com>

So I'm OK with it in principal and I think the code is OK, so

Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>

> "for-6.0" needs to be accompanied by a justification of why it's
> important to go in the release at this point...

I guess the argument is that when we hit 6.0 it becomes API and removing
the 'blocked' becomes a matter of deprecation which is a pain.

Hmm; I agree it's the right change, but I'm not sure I can justify it
this late in the release.

Dave

> thanks
> -- PMM
> 
-- 
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK