[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/6] iotests: Skip test 079 if it is not possible to create l
From: |
Alex Bennée |
Subject: |
Re: [PATCH 2/6] iotests: Skip test 079 if it is not possible to create large files |
Date: |
Fri, 22 Nov 2019 12:57:45 +0000 |
User-agent: |
mu4e 1.3.5; emacs 27.0.50 |
Thomas Huth <address@hidden> writes:
> Test 079 fails in the arm64, s390x and ppc64le LXD containers, which
> apparently do not allow large files to be created. Test 079 tries to
> create a 4G sparse file, so check first whether we can really create
> such files before executing the test.
>
> Signed-off-by: Thomas Huth <address@hidden>
> ---
> tests/qemu-iotests/079 | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/tests/qemu-iotests/079 b/tests/qemu-iotests/079
> index 81f0c21f53..e9b81419b7 100755
> --- a/tests/qemu-iotests/079
> +++ b/tests/qemu-iotests/079
> @@ -39,6 +39,12 @@ trap "_cleanup; exit \$status" 0 1 2 3 15
> _supported_fmt qcow2
> _supported_proto file nfs
>
> +# Some containers (e.g. non-x86 on Travis) do not allow large files
> +if ! truncate --size=4G "$TEST_IMG"; then
> + _notrun "file system on $TEST_DIR does not support large enough files"
> +fi
> +rm "$TEST_IMG"
> +
Hmm we are repeating ourselves here. Maybe the test should be wrapped up
as __supported_filesize and the test can just do:
__supported_filesize 4G
along with the other tests above.
--
Alex Bennée
Re: [PATCH 2/6] iotests: Skip test 079 if it is not possible to create large files,
Alex Bennée <=
[PATCH 3/6] tests/hd-geo-test: Skip test when images can not be created, Thomas Huth, 2019/11/19
[PATCH 4/6] tests/test-util-filemonitor: Skip test on non-x86 Travis containers, Thomas Huth, 2019/11/19
[PATCH 5/6] travis.yml: drop 32 bit systems from MAIN_SOFTMMU_TARGETS, Thomas Huth, 2019/11/19
[PATCH 6/6] travis.yml: Enable builds on arm64, ppc64le and s390x, Thomas Huth, 2019/11/19