09.08.2019 19:14, Max Reitz wrote: > complete_and_wait() and wait_ready() currently only work for mirror > jobs. Let them work for active commit jobs, too. > > Signed-off-by: Max Reitz <mre...@redhat.com> > --- > tests/qemu-iotests/iotests.py | 10 +++++++--- > 1 file changed, 7 insertions(+), 3 deletions(-) > > diff --git a/tests/qemu-iotests/iotests.py b/tests/qemu-iotests/iotests.py > index 84438e837c..3ef846d1dc 100644 > --- a/tests/qemu-iotests/iotests.py > +++ b/tests/qemu-iotests/iotests.py > @@ -761,8 +761,12 @@ class QMPTestCase(unittest.TestCase): > > def wait_ready(self, drive='drive0'): > '''Wait until a block job BLOCK_JOB_READY event''' > - f = {'data': {'type': 'mirror', 'device': drive } } > - event = self.vm.event_wait(name='BLOCK_JOB_READY', match=f) > + event = self.vm.events_wait([ > + ('BLOCK_JOB_READY', > + {'data': {'type': 'mirror', 'device': drive } }), > + ('BLOCK_JOB_READY', > + {'data': {'type': 'commit', 'device': drive } }) > + ]) > > def wait_ready_and_cancel(self, drive='drive0'): > self.wait_ready(drive=drive) > @@ -780,7 +784,7 @@ class QMPTestCase(unittest.TestCase): > self.assert_qmp(result, 'return', {}) > > event = self.wait_until_completed(drive=drive) > - self.assert_qmp(event, 'data/type', 'mirror') > + self.assertTrue(event['data']['type'] in ['mirror', 'commit']) > > def pause_wait(self, job_id='job0'): > with Timeout(1, "Timeout waiting for job to pause"): >
Reviewed-by: Vladimir Sementsov-Ogievskiy <vsement...@virtuozzo.com> -- Best regards, Vladimir