Re: [PATCH] tests/qtest: failover: fix infinite loop

[Thomas Huth]

On 29/03/2022 15.27, Peter Maydell wrote:
> On Tue, 29 Mar 2022 at 14:25, Thomas Huth <thuth@redhat.com> wrote:
> > On 29/03/2022 15.23, Peter Maydell wrote:
> > > On Tue, 29 Mar 2022 at 13:47, Thomas Huth <thuth@redhat.com> wrote:
> > > > On 29/03/2022 14.42, Laurent Vivier wrote:
> > > > > If the migration is over before we cancel it, we are
> > > > > waiting in a loop a state that never comes because the state
> > > > > is already "completed".
> > > > >
> > > > > To avoid an infinite loop, skip the test if the migration
> > > > > is "completed" before we were able to cancel it.
> > >
> > > > Is this still urgent for 7.0, or can it wait for the 7.1 cycle?
> > >
> > > It's a test case change that fixes at least one hang I've seen
> > > in "make check". I prefer those to go in, at least before rc3,
> > > because the CI loop being unreliable makes the whole release
> > > process slower and more annoying.
> >
> > Ok. Do you want to pick it directly, or shall I create a pull request for
> > this? (I don't have much else queued right now, that's why I ask)
>
> I can apply it directly if you don't have anything else you're
> sending anyway.

Ok, then please apply directly! Thanks!

 Thomas