iotests/229: Use blkdebug to inject an error
229 relies on the mirror running into an I/O error when the target is smaller than the source. After changing mirror to catch this condition while starting the job, this test case won't get a job that is paused for an I/O error any more. Use blkdebug instead to inject an error. Signed-off-by: Kevin Wolf <kwolf@redhat.com> Reviewed-by: Eric Blake <eblake@redhat.com> Message-Id: <20200511135825.219437-3-kwolf@redhat.com> Reviewed-by: Max Reitz <mreitz@redhat.com> Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
This commit is contained in:
parent
ffa41a62d0
commit
d89ac3cf30
@ -33,6 +33,7 @@ _cleanup()
|
|||||||
_cleanup_test_img
|
_cleanup_test_img
|
||||||
_rm_test_img "$TEST_IMG"
|
_rm_test_img "$TEST_IMG"
|
||||||
_rm_test_img "$DEST_IMG"
|
_rm_test_img "$DEST_IMG"
|
||||||
|
rm -f "$TEST_DIR/blkdebug.conf"
|
||||||
}
|
}
|
||||||
trap "_cleanup; exit \$status" 0 1 2 3 15
|
trap "_cleanup; exit \$status" 0 1 2 3 15
|
||||||
|
|
||||||
@ -49,11 +50,10 @@ _supported_os Linux
|
|||||||
|
|
||||||
DEST_IMG="$TEST_DIR/d.$IMGFMT"
|
DEST_IMG="$TEST_DIR/d.$IMGFMT"
|
||||||
TEST_IMG="$TEST_DIR/b.$IMGFMT"
|
TEST_IMG="$TEST_DIR/b.$IMGFMT"
|
||||||
|
BLKDEBUG_CONF="$TEST_DIR/blkdebug.conf"
|
||||||
|
|
||||||
_make_test_img 2M
|
_make_test_img 2M
|
||||||
|
TEST_IMG=$DEST_IMG _make_test_img 2M
|
||||||
# destination for mirror will be too small, causing error
|
|
||||||
TEST_IMG=$DEST_IMG _make_test_img 1M
|
|
||||||
|
|
||||||
$QEMU_IO -c 'write 0 2M' "$TEST_IMG" | _filter_qemu_io
|
$QEMU_IO -c 'write 0 2M' "$TEST_IMG" | _filter_qemu_io
|
||||||
|
|
||||||
@ -67,11 +67,18 @@ echo
|
|||||||
echo '=== Starting drive-mirror, causing error & stop ==='
|
echo '=== Starting drive-mirror, causing error & stop ==='
|
||||||
echo
|
echo
|
||||||
|
|
||||||
|
cat > "$BLKDEBUG_CONF" <<EOF
|
||||||
|
[inject-error]
|
||||||
|
event = "write_aio"
|
||||||
|
errno = "5"
|
||||||
|
once = "on"
|
||||||
|
EOF
|
||||||
|
|
||||||
_send_qemu_cmd $QEMU_HANDLE \
|
_send_qemu_cmd $QEMU_HANDLE \
|
||||||
"{'execute': 'drive-mirror',
|
"{'execute': 'drive-mirror',
|
||||||
'arguments': {'device': 'testdisk',
|
'arguments': {'device': 'testdisk',
|
||||||
'format': '$IMGFMT',
|
'format': '$IMGFMT',
|
||||||
'target': '$DEST_IMG',
|
'target': 'blkdebug:$BLKDEBUG_CONF:$DEST_IMG',
|
||||||
'sync': 'full',
|
'sync': 'full',
|
||||||
'mode': 'existing',
|
'mode': 'existing',
|
||||||
'on-source-error': 'stop',
|
'on-source-error': 'stop',
|
||||||
@ -90,7 +97,8 @@ success_or_failure="y" _send_qemu_cmd $QEMU_HANDLE \
|
|||||||
'arguments': { 'device': 'testdisk',
|
'arguments': { 'device': 'testdisk',
|
||||||
'force': true}}" \
|
'force': true}}" \
|
||||||
"BLOCK_JOB_CANCELLED" "Assertion" \
|
"BLOCK_JOB_CANCELLED" "Assertion" \
|
||||||
| grep -v '"BLOCK_JOB_ERROR"'
|
| grep -v '"BLOCK_JOB_ERROR"' \
|
||||||
|
| _filter_block_job_offset
|
||||||
|
|
||||||
# success, all done
|
# success, all done
|
||||||
echo "*** done"
|
echo "*** done"
|
||||||
|
@ -1,6 +1,6 @@
|
|||||||
QA output created by 229
|
QA output created by 229
|
||||||
Formatting 'TEST_DIR/b.IMGFMT', fmt=IMGFMT size=2097152
|
Formatting 'TEST_DIR/b.IMGFMT', fmt=IMGFMT size=2097152
|
||||||
Formatting 'TEST_DIR/d.IMGFMT', fmt=IMGFMT size=1048576
|
Formatting 'TEST_DIR/d.IMGFMT', fmt=IMGFMT size=2097152
|
||||||
wrote 2097152/2097152 bytes at offset 0
|
wrote 2097152/2097152 bytes at offset 0
|
||||||
2 MiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
|
2 MiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
|
||||||
{'execute': 'qmp_capabilities'}
|
{'execute': 'qmp_capabilities'}
|
||||||
@ -8,7 +8,7 @@ wrote 2097152/2097152 bytes at offset 0
|
|||||||
|
|
||||||
=== Starting drive-mirror, causing error & stop ===
|
=== Starting drive-mirror, causing error & stop ===
|
||||||
|
|
||||||
{'execute': 'drive-mirror', 'arguments': {'device': 'testdisk', 'format': 'IMGFMT', 'target': 'TEST_DIR/d.IMGFMT', 'sync': 'full', 'mode': 'existing', 'on-source-error': 'stop', 'on-target-error': 'stop' }}
|
{'execute': 'drive-mirror', 'arguments': {'device': 'testdisk', 'format': 'IMGFMT', 'target': 'blkdebug:TEST_DIR/blkdebug.conf:TEST_DIR/d.IMGFMT', 'sync': 'full', 'mode': 'existing', 'on-source-error': 'stop', 'on-target-error': 'stop' }}
|
||||||
{"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP}, "event": "JOB_STATUS_CHANGE", "data": {"status": "created", "id": "testdisk"}}
|
{"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP}, "event": "JOB_STATUS_CHANGE", "data": {"status": "created", "id": "testdisk"}}
|
||||||
{"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP}, "event": "JOB_STATUS_CHANGE", "data": {"status": "running", "id": "testdisk"}}
|
{"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP}, "event": "JOB_STATUS_CHANGE", "data": {"status": "running", "id": "testdisk"}}
|
||||||
{"return": {}}
|
{"return": {}}
|
||||||
@ -21,5 +21,5 @@ wrote 2097152/2097152 bytes at offset 0
|
|||||||
{"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP}, "event": "JOB_STATUS_CHANGE", "data": {"status": "running", "id": "testdisk"}}
|
{"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP}, "event": "JOB_STATUS_CHANGE", "data": {"status": "running", "id": "testdisk"}}
|
||||||
{"return": {}}
|
{"return": {}}
|
||||||
{"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP}, "event": "JOB_STATUS_CHANGE", "data": {"status": "aborting", "id": "testdisk"}}
|
{"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP}, "event": "JOB_STATUS_CHANGE", "data": {"status": "aborting", "id": "testdisk"}}
|
||||||
{"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP}, "event": "BLOCK_JOB_CANCELLED", "data": {"device": "testdisk", "len": 2097152, "offset": 1048576, "speed": 0, "type": "mirror"}}
|
{"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP}, "event": "BLOCK_JOB_CANCELLED", "data": {"device": "testdisk", "len": 2097152, "offset": OFFSET, "speed": 0, "type": "mirror"}}
|
||||||
*** done
|
*** done
|
||||||
|
Loading…
Reference in New Issue
Block a user