Philippe Mathieu-Daudé <phi...@redhat.com> writes:
> On 7/23/19 1:46 PM, Alex Bennée wrote: >> >> Peter Maydell <peter.mayd...@linaro.org> writes: >> >>> On Tue, 23 Jul 2019 at 11:36, Alex Bennée <alex.ben...@linaro.org> wrote: >>>> >>>> The following changes since commit >>>> 23da9e297b4120ca9702cabec91599a44255fe96: >>>> >>>> Merge remote-tracking branch >>>> 'remotes/pmaydell/tags/pull-target-arm-20190722' into staging (2019-07-22 >>>> 15:16:48 +0100) >>>> >>>> are available in the Git repository at: >>>> >>>> https://github.com/stsquad/qemu.git tags/pull-testing-230719-1 >>>> >>>> for you to fetch changes up to 2e3b8743cd66f1d0fbc83051b0b6b072a97460cc: >>>> >>>> gitlab-ci: Remove qcow2 tests that are handled by "make check" already >>>> (2019-07-23 11:33:15 +0100) >>>> >>>> ---------------------------------------------------------------- >>>> Final testing updates: >>>> >>>> - docker sphinx updates >>>> - windows build re-enabled in CI >>>> - travis_retry for make check >>>> - quick iotests enabled for make check >>>> - build fixes >>>> >>>> ---------------------------------------------------------------- >>> >>> 'make check-tcg' fails (x86-64 host): >>> >>> make: Entering directory >>> '/home/petmay01/linaro/qemu-for-merges/build/all-linux-static' >>> BUILD debian10 >>> The command '/bin/sh -c DEBIAN_FRONTEND=noninteractive eatmydata >>> apt install -y --no-install-recommends bison >>> build-essential ca-certificates clang flex >>> gettext git pkg-config psmisc >>> python python3-sphinx texinfo $(apt-get -s >>> build-dep qemu | egrep ^Inst | fgrep '[all]' | cut -d\ -f2)' returned >>> a non-zero code: 100 >>> Traceback (most recent call last): >>> File "/home/petmay01/linaro/qemu-for-merges/tests/docker/docker.py", >>> line 615, in <module> >>> sys.exit(main()) >>> File "/home/petmay01/linaro/qemu-for-merges/tests/docker/docker.py", >>> line 611, in main >>> return args.cmdobj.run(args, argv) >>> File "/home/petmay01/linaro/qemu-for-merges/tests/docker/docker.py", >>> line 413, in run >>> extra_files_cksum=cksum) >>> File "/home/petmay01/linaro/qemu-for-merges/tests/docker/docker.py", >>> line 280, in build_image >>> quiet=quiet) >>> File "/home/petmay01/linaro/qemu-for-merges/tests/docker/docker.py", >>> line 207, in _do_check >>> return subprocess.check_call(self._command + cmd, **kwargs) >>> File "/usr/lib/python2.7/subprocess.py", line 190, in check_call >>> raise CalledProcessError(retcode, cmd) >>> subprocess.CalledProcessError: Command '['docker', 'build', '-t', >>> 'qemu:debian10', '-f', '/tmp/docker_buildBgZdCD/tmp6MJLiZ.docker', >>> '/tmp/docker_buildBgZdCD']' returned non-zero exit status 100 >>> /home/petmay01/linaro/qemu-for-merges/tests/docker/Makefile.include:53: >>> recipe for target 'docker-image-debian10' failed > > Yes, we should definitively improve this error report. > >>> make: *** [docker-image-debian10] Error 1 >>> make: Leaving directory >>> '/home/petmay01/linaro/qemu-for-merges/build/all-linux-static' >> >> Odd. Does: >> >> make docker-image-debian10 V=1 >> >> fail for you? Can you retry with: >> >> make docker-image-debian10 V=1 NOCACHE=1 > > Should we use a stable tag for the stable release? Such: > > -- >8 -- > diff --git a/tests/docker/dockerfiles/debian10.docker > b/tests/docker/dockerfiles/debian10.docker > index aeeb151b52..0f2800377d 100644 > --- a/tests/docker/dockerfiles/debian10.docker > +++ b/tests/docker/dockerfiles/debian10.docker > @@ -7,7 +7,7 @@ > # On its own you can't build much but the docker-foo-cross targets > # build on top of the base debian image. > # > -FROM debian:buster-slim > +FROM debian:buster-20190708-slim I don't think so - Buster is stable now - I think the problem is stale cache states from previous interrupted builds. If NOCACHE makes it go away then I think we are good. > > # Duplicate deb line as deb-src > --- > >>> >>> I also got a failure with openbsd which seems to be when >>> the tests run check-block.sh: >>> >>> /home/qemu/qemu-test.Dy1Aj3/src/tests/check-block.sh >>> TEST iotest-qcow2: 001 >>> TEST iotest-qcow2: 002 >>> TEST iotest-qcow2: 003 >>> TEST iotest-qcow2: 004 >>> TEST iotest-qcow2: 005 [not run] >>> TEST iotest-qcow2: 007 [fail] >>> QEMU -- >>> "/home/qemu/qemu-test.Dy1Aj3/build/tests/qemu-iotests/../../aarch64-softmmu/qemu-system-aarch64" >>> -nodefaults -machine virt,accel=qtest >>> QEMU_IMG -- >>> "/home/qemu/qemu-test.Dy1Aj3/build/tests/qemu-iotests/../../qemu-img" >>> QEMU_IO -- >>> "/home/qemu/qemu-test.Dy1Aj3/build/tests/qemu-iotests/../../qemu-io" >>> --cache writeback -f qcow2 >>> QEMU_NBD -- >>> "/home/qemu/qemu-test.Dy1Aj3/build/tests/qemu-iotests/../../qemu-nbd" >>> IMGFMT -- qcow2 (compat=1.1) >>> IMGPROTO -- file >>> PLATFORM -- OpenBSD/amd64 openbsd 6.5 >>> TEST_DIR -- >>> /home/qemu/qemu-test.Dy1Aj3/build/tests/qemu-iotests/scratch >>> SOCKET_SCM_HELPER -- >>> >>> --- /home/qemu/qemu-test.Dy1Aj3/src/tests/qemu-iotests/007.out Tue >>> Jul 23 10:49:15 2019 >>> +++ /home/qemu/qemu-test.Dy1Aj3/build/tests/qemu-iotests/007.out.bad >>> Tue Jul 23 11:10:21 2019 >>> @@ -2,16 +2,7 @@ >>> >>> creating image >>> Formatting 'TEST_DIR/t.IMGFMT', fmt=IMGFMT size=1048576 >>> -savevm 1 >>> -savevm 2 >>> -savevm 3 >>> -savevm 4 >>> -savevm 5 >>> -savevm 6 >>> -savevm 7 >>> -savevm 8 >>> -savevm 9 >>> -savevm 10 >>> +./007: line 51: seq: command not found >>> >>> [and some other failures] >>> >>> I see Thomas Huth has a patch on-list for that, but this >>> didn't manifest as a problem before this pullreq. >> >> OK, I'll add it and rebuild the PR. > > But Thomas got another error later... > >> >> Have you noticed any issues with check-acceptance? >> >>> >>> thanks >>> -- PMM >> >> >> -- >> Alex Bennée >> -- Alex Bennée