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

2022-04-04 Thread Dr. David Alan Gilbert
* Laurent Vivier (lviv...@redhat.com) 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

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

2022-03-29 Thread Peter Maydell
On Tue, 29 Mar 2022 at 13:43, 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 t

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

2022-03-29 Thread Thomas Huth
On 29/03/2022 15.27, Peter Maydell wrote: On Tue, 29 Mar 2022 at 14:25, Thomas Huth wrote: On 29/03/2022 15.23, Peter Maydell wrote: On Tue, 29 Mar 2022 at 13:47, Thomas Huth wrote: On 29/03/2022 14.42, Laurent Vivier wrote: If the migration is over before we cancel it, we are waiting in

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

2022-03-29 Thread Peter Maydell
On Tue, 29 Mar 2022 at 14:25, Thomas Huth wrote: > > On 29/03/2022 15.23, Peter Maydell wrote: > > On Tue, 29 Mar 2022 at 13:47, Thomas Huth 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

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

2022-03-29 Thread Thomas Huth
On 29/03/2022 15.23, Peter Maydell wrote: On Tue, 29 Mar 2022 at 13:47, Thomas Huth 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

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

2022-03-29 Thread Peter Maydell
On Tue, 29 Mar 2022 at 13:47, Thomas Huth 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

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

2022-03-29 Thread Thomas Huth
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. Signed-

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

2022-03-29 Thread Laurent Vivier
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. Signed-off-by: Laurent Vivier --- tests/qtest/virt