[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] iotests: Make 144 deterministic again
From: |
Stefan Hajnoczi |
Subject: |
Re: [PATCH] iotests: Make 144 deterministic again |
Date: |
Mon, 12 Feb 2024 10:20:05 -0500 |
On Fri, Feb 09, 2024 at 06:31:03PM +0100, Kevin Wolf wrote:
> Since commit effd60c8 changed how QMP commands are processed, the order
> of the block-commit return value and job events in iotests 144 wasn't
> fixed and more and caused the test to fail intermittently.
>
> Change the test to cache events first and then print them in a
> predefined order.
>
> Waiting three times for JOB_STATUS_CHANGE is a bit uglier than just
> waiting for the JOB_STATUS_CHANGE that has "status": "ready", but the
> tooling we have doesn't seem to allow the latter easily.
>
> Fixes: effd60c878176bcaf97fa7ce2b12d04bb8ead6f7
> Resolves: https://gitlab.com/qemu-project/qemu/-/issues/2126
> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
> ---
> tests/qemu-iotests/144 | 12 +++++++++++-
> tests/qemu-iotests/144.out | 2 +-
> 2 files changed, 12 insertions(+), 2 deletions(-)
Thank you!
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
signature.asc
Description: PGP signature