[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PATCH for-2.10 5/7] qemu-iotests/162: Fix
From: |
Jeff Cody |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PATCH for-2.10 5/7] qemu-iotests/162: Fix leaked temporary files |
Date: |
Fri, 28 Jul 2017 17:18:23 -0400 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Fri, Jul 28, 2017 at 03:14:50PM +0200, Kevin Wolf wrote:
> qemu-iotests 162 left qemu-nbd.pid behind in the scratch directory, and
> potentially a file called '42' in the current directory. Make sure to
> clean it up after completing the tests.
>
> Signed-off-by: Kevin Wolf <address@hidden>
> ---
> tests/qemu-iotests/162 | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/tests/qemu-iotests/162 b/tests/qemu-iotests/162
> index cad2bd70ab..477a806360 100755
> --- a/tests/qemu-iotests/162
> +++ b/tests/qemu-iotests/162
> @@ -28,6 +28,13 @@ echo "QA output created by $seq"
> here="$PWD"
> status=1 # failure is the default!
>
> +_cleanup()
> +{
> + rm -f "${TEST_DIR}/qemu-nbd.pid"
> + rm -f 42
> +}
> +trap "_cleanup; exit \$status" 0 1 2 3 15
> +
> # get standard environment, filters and checks
> . ./common.rc
> . ./common.filter
> --
> 2.13.3
>
>
Reviewed-by: Jeff Cody <address@hidden>
- Re: [Qemu-block] [Qemu-devel] [PATCH for-2.10 2/7] qemu-iotests: Remove blkdebug.conf after tests, (continued)
- [Qemu-block] [PATCH for-2.10 1/7] qemu-iotests/041: Fix leaked scratch images, Kevin Wolf, 2017/07/28
- [Qemu-block] [PATCH for-2.10 3/7] qemu-iotests/141: Fix image cleanup, Kevin Wolf, 2017/07/28
- [Qemu-block] [PATCH for-2.10 4/7] qemu-iotests/153: Fix leaked scratch images, Kevin Wolf, 2017/07/28
- [Qemu-block] [PATCH for-2.10 5/7] qemu-iotests/162: Fix leaked temporary files, Kevin Wolf, 2017/07/28
- Re: [Qemu-block] [Qemu-devel] [PATCH for-2.10 5/7] qemu-iotests/162: Fix leaked temporary files,
Jeff Cody <=
- [Qemu-block] [PATCH for-2.10 6/7] qemu-iotests/063: Fix leaked image, Kevin Wolf, 2017/07/28
- [Qemu-block] [PATCH for-2.10 7/7] qemu-iotests/059: Fix leaked image files, Kevin Wolf, 2017/07/28
- Re: [Qemu-block] [Qemu-devel] [PATCH for-2.10 0/7] qemu-iotests: Fix leaked temporary file fixes, Eric Blake, 2017/07/28