Commit b1b30ff4 authored by Kevin Wolf's avatar Kevin Wolf
Browse files

iotests/030: Reduce run time by unthrottling job earlier 



test_overlapping_3() throttles its active commit job so it can be sure
the job is still busy when it checks that you can't start a conflicting
streaming job.

However, it only sets the commit job back to full speed when it is
ready, which takes a few seconds while it's throttled. We can already
reset the limit after having checked that block-stream returns an error
and save these seconds.

Signed-off-by: default avatarKevin Wolf <kwolf@redhat.com>
Message-Id: <20200513100025.33543-1-kwolf@redhat.com>
Reviewed-by: default avatarAlberto Garcia <berto@igalia.com>
Signed-off-by: default avatarKevin Wolf <kwolf@redhat.com>
parent c5f12a80

tests/qemu-iotests/030

+3 −3
Original line number Diff line number Diff line
@@ -354,14 +354,14 @@ class TestParallelOps(iotests.QMPTestCase): 
        self.assert_qmp(result, 'error/desc',

            "Node 'node5' is busy: block device is in use by block job: commit")



        result = self.vm.qmp('block-job-set-speed', device='commit-drive0', speed=0)

        self.assert_qmp(result, 'return', {})



        event = self.vm.event_wait(name='BLOCK_JOB_READY')

        self.assert_qmp(event, 'data/device', 'commit-drive0')

        self.assert_qmp(event, 'data/type', 'commit')

        self.assert_qmp_absent(event, 'data/error')



        result = self.vm.qmp('block-job-set-speed', device='commit-drive0', speed=0)

        self.assert_qmp(result, 'return', {})



        result = self.vm.qmp('block-job-complete', device='commit-drive0')

        self.assert_qmp(result, 'return', {})