| Message ID | 20191204154618.23560-3-thuth@redhat.com |
|---|---|
| State | New |
| Headers | show |
| Series | Enable Travis builds on arm64, ppc64le and s390x | expand |
On 12/4/19 4:46 PM, Thomas Huth wrote: > Test 060 fails in the arm64, s390x and ppc64le LXD containers on Travis > (which we will hopefully enable in our CI soon). These containers > apparently do not allow large files to be created. The repair process > in test 060 creates a file of 64 GiB, so test first whether such large > files are possible and skip the test if that's not the case. > > Signed-off-by: Thomas Huth <thuth@redhat.com> > --- > tests/qemu-iotests/060 | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/tests/qemu-iotests/060 b/tests/qemu-iotests/060 > index b91d8321bb..d96f17a484 100755 > --- a/tests/qemu-iotests/060 > +++ b/tests/qemu-iotests/060 > @@ -49,6 +49,9 @@ _supported_fmt qcow2 > _supported_proto file > _supported_os Linux > > +# The repair process will create a large file - so check for availability first > +_require_large_file 64G > + > rt_offset=65536 # 0x10000 (XXX: just an assumption) > rb_offset=131072 # 0x20000 (XXX: just an assumption) > l1_offset=196608 # 0x30000 (XXX: just an assumption) > Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
On Wed, Dec 04, 2019 at 04:46:13PM +0100, Thomas Huth wrote: > Test 060 fails in the arm64, s390x and ppc64le LXD containers on Travis > (which we will hopefully enable in our CI soon). These containers > apparently do not allow large files to be created. The repair process > in test 060 creates a file of 64 GiB, so test first whether such large > files are possible and skip the test if that's not the case. > > Signed-off-by: Thomas Huth <thuth@redhat.com> > --- > tests/qemu-iotests/060 | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/tests/qemu-iotests/060 b/tests/qemu-iotests/060 > index b91d8321bb..d96f17a484 100755 > --- a/tests/qemu-iotests/060 > +++ b/tests/qemu-iotests/060 > @@ -49,6 +49,9 @@ _supported_fmt qcow2 > _supported_proto file > _supported_os Linux > > +# The repair process will create a large file - so check for availability first > +_require_large_file 64G > + > rt_offset=65536 # 0x10000 (XXX: just an assumption) > rb_offset=131072 # 0x20000 (XXX: just an assumption) > l1_offset=196608 # 0x30000 (XXX: just an assumption) > -- > 2.18.1 > The behavior and failure is indeed pretty consistent accross those architectures: - arm64: https://travis-ci.org/clebergnu/qemu/jobs/621238740#L4217 - ppc64le: https://travis-ci.org/clebergnu/qemu/jobs/621238741#L4252 - s390x: https://travis-ci.org/clebergnu/qemu/jobs/621238742#L4265 And with this, 060 gets skipped properly: - arm64: https://travis-ci.org/clebergnu/qemu/jobs/621248591#L4202 - ppc64le: https://travis-ci.org/clebergnu/qemu/jobs/621248592#L4236 - s390x: https://travis-ci.org/clebergnu/qemu/jobs/621248593#L4250 Reviewed-by: Cleber Rosa <crosa@redhat.com> Tested-by: Cleber Rosa <crosa@redhat.com>
diff --git a/tests/qemu-iotests/060 b/tests/qemu-iotests/060 index b91d8321bb..d96f17a484 100755 --- a/tests/qemu-iotests/060 +++ b/tests/qemu-iotests/060 @@ -49,6 +49,9 @@ _supported_fmt qcow2 _supported_proto file _supported_os Linux +# The repair process will create a large file - so check for availability first +_require_large_file 64G + rt_offset=65536 # 0x10000 (XXX: just an assumption) rb_offset=131072 # 0x20000 (XXX: just an assumption) l1_offset=196608 # 0x30000 (XXX: just an assumption)
Test 060 fails in the arm64, s390x and ppc64le LXD containers on Travis (which we will hopefully enable in our CI soon). These containers apparently do not allow large files to be created. The repair process in test 060 creates a file of 64 GiB, so test first whether such large files are possible and skip the test if that's not the case. Signed-off-by: Thomas Huth <thuth@redhat.com> --- tests/qemu-iotests/060 | 3 +++ 1 file changed, 3 insertions(+)