From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:39197) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YFvUp-0001ft-KP for qemu-devel@nongnu.org; Mon, 26 Jan 2015 21:03:16 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YFvUj-0003Ae-Fr for qemu-devel@nongnu.org; Mon, 26 Jan 2015 21:03:15 -0500 Received: from mx1.redhat.com ([209.132.183.28]:56955) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YFvUj-0003AW-7r for qemu-devel@nongnu.org; Mon, 26 Jan 2015 21:03:09 -0500 From: Max Reitz Date: Mon, 26 Jan 2015 21:02:58 -0500 Message-Id: <1422324179-32106-4-git-send-email-mreitz@redhat.com> In-Reply-To: <1422324179-32106-1-git-send-email-mreitz@redhat.com> References: <1422324179-32106-1-git-send-email-mreitz@redhat.com> Subject: [Qemu-devel] [PATCH 3/4] iotests: Fix 104 for NBD List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Kevin Wolf , Paolo Bonzini , Peter Lieven , Stefan Hajnoczi , Max Reitz _make_test_img sets up an NBD server, _cleanup_test_img shuts it down; thus, _cleanup_test_img has to be called before _make_test_img is invoked another time. Furthermore, the pipe through _filter_test_img was unnecessary; _make_test_img already takes care of that. And finally, a filter is added to _filter_img_info to replace "nbd://127.0.0.1:10810" by "TEST_DIR/t.IMGFMT", since the former is the way to express the full image path (normally the latter) for NBD tests. Signed-off-by: Max Reitz --- tests/qemu-iotests/104 | 9 +++------ tests/qemu-iotests/common.filter | 1 + 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/tests/qemu-iotests/104 b/tests/qemu-iotests/104 index b471aa5..f32752b 100755 --- a/tests/qemu-iotests/104 +++ b/tests/qemu-iotests/104 @@ -28,11 +28,7 @@ here=`pwd` tmp=/tmp/$$ status=1 # failure is the default! -_cleanup() -{ - _cleanup_test_img -} -trap "_cleanup; exit \$status" 0 1 2 3 15 +trap "exit \$status" 0 1 2 3 15 # get standard environment, filters and checks . ./common.rc @@ -47,8 +43,9 @@ echo image_sizes="1024 1234" for s in $image_sizes; do - _make_test_img $s | _filter_img_create + _make_test_img $s _img_info | _filter_img_info + _cleanup_test_img done # success, all done diff --git a/tests/qemu-iotests/common.filter b/tests/qemu-iotests/common.filter index b73c70b..06e1bb0 100644 --- a/tests/qemu-iotests/common.filter +++ b/tests/qemu-iotests/common.filter @@ -200,6 +200,7 @@ _filter_img_info() sed -e "s#$IMGPROTO:$TEST_DIR#TEST_DIR#g" \ -e "s#$TEST_DIR#TEST_DIR#g" \ -e "s#$IMGFMT#IMGFMT#g" \ + -e 's#nbd://127.0.0.1:10810$#TEST_DIR/t.IMGFMT#g' \ -e "/encrypted: yes/d" \ -e "/cluster_size: [0-9]\\+/d" \ -e "/table_size: [0-9]\\+/d" \ -- 2.1.0