From: "Daniel P. Berrangé" <berrange@redhat.com>
To: "Alex Bennée" <alex.bennee@linaro.org>,
qemu-devel@nongnu.org, stefanha@redhat.com,
"Paolo Bonzini" <pbonzini@redhat.com>
Subject: Re: [PULL 28/54] configure: build ROMs with container-based cross compilers
Date: Mon, 10 Oct 2022 14:07:53 +0100 [thread overview]
Message-ID: <Y0QZKSHRDLlOHEgP@redhat.com> (raw)
In-Reply-To: <Y0QV+B+Wz6fxceh1@redhat.com>
On Mon, Oct 10, 2022 at 01:54:16PM +0100, Daniel P. Berrangé wrote:
> On Tue, Oct 04, 2022 at 02:01:12PM +0100, Alex Bennée wrote:
> > From: Paolo Bonzini <pbonzini@redhat.com>
> >
> > s390-ccw remains a bit more complex, because the -march=z900 test is done
> > only for the native cross compiler. Otherwise, all that is needed is
> > to pass the (now mandatory) target argument to write_target_makefile.
> >
> > Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> > Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> > Message-Id: <20220929114231.583801-29-alex.bennee@linaro.org>
>
> I'm not at all convinced this change was/is a good idea.
>
> First of all, it causes 'make' to now download about 1 GB of
> container images
>
> $ ./configure --target-list=x86_64-softmmu
> $ make
> ...snip...
> BUILD debian-powerpc-test-cross
> Trying to pull registry.gitlab.com/qemu-project/qemu/qemu/debian-powerpc-test-cross:latest...
> Getting image source signatures
> Copying blob 2a205c8a1d36 [=>------------------------------------] 12.4MiB / 257.2MiB
>
> ...
> ...snip...
>
> Despite downloading this image, it then proceeded to rebuild the
> image from scratch, requiring another few 100MBs of downloads
> of dpkgs. This time the download was without progress information
> until it entirely failed due to a dead Debia mirror server, needing
> a retry.
This failure seems worse than I realized. It also failed on a
non-responsive mirror when re-building the s390 image. WHen I
re-ran 'make' it definitely did not retry the re-build process,
as 'BUILD debian-s390x-cross' was instantaneous. So it looks
like it is using the downloaded cached image, despite the previous
'make' apparently considering that to have been outdated content
needing a rebuild.
FWIW the failure output was:
BUILD debian-s390x-cross
Trying to pull registry.gitlab.com/qemu-project/qemu/qemu/debian-s390x-cross:latest...
Getting image source signatures
Copying blob fc8d65e34cd5 [===================================>--] 341.6MiB / 360.2MiB
Copying blob fc8d65e34cd5 done
Copying blob bd159e379b3b skipped: already exists
Copying blob 13224e2971af done
Copying config 67a127f7cd done
Writing manifest to image destination
Storing signatures
debconf: delaying package configuration, since apt-utils is not installed
E: Failed to fetch http://deb.debian.org/debian/pool/main/s/systemd/systemd_247.3-7%2bdeb11u1_amd64.deb 400 Bad Request [IP: 199.232.122.132 80]
E: Failed to fetch http://deb.debian.org/debian/pool/main/p/perl/perl_5.32.1-4%2bdeb11u2_amd64.deb Error reading from server - read (104: Connection reset by peer) [IP: 199.232.122.132 80]
E: Unable to fetch some archives, maybe run apt-get update or try with --fix-missing?
Error: error building at STEP "RUN export DEBIAN_FRONTEND=noninteractive && apt-get update && apt-get install -y eatmydata && eatmydata apt-get dist-upgrade -y && eatmydata apt-get install --no-install-recommends -y bash bc bison bsdextrautils bzip2 ca-certificates ccache dbus debianutils diffutils exuberant-ctags findutils flex gcovr genisoimage gettext git hostname libglib2.0-dev libpcre2-dev libspice-protocol-dev llvm locales make meson ncat ninja-build openssh-client perl-base pkgconf python3 python3-numpy python3-opencv python3-pillow python3-pip python3-sphinx python3-sphinx-rtd-theme python3-venv python3-yaml rpm2cpio sed sparse tar tesseract-ocr tesseract-ocr-eng texinfo && eatmydata apt-get autoremove -y && eatmydata apt-get autoclean -y && sed -Ei 's,^# (en_US\.UTF-8 .*)$,\1,' /etc/locale.gen && dpkg-reconfigure locales": error while running runtime: exit status 100
Traceback (most recent call last):
File "/home/berrange/src/virt/qemu/tests/docker/docker.py", line 683, in <module>
sys.exit(main())
File "/home/berrange/src/virt/qemu/tests/docker/docker.py", line 679, in main
return args.cmdobj.run(args, argv)
File "/home/berrange/src/virt/qemu/tests/docker/docker.py", line 493, in run
dkr.build_image(tag, docker_dir, dockerfile,
File "/home/berrange/src/virt/qemu/tests/docker/docker.py", line 343, in build_image
self._do_check(build_args,
File "/home/berrange/src/virt/qemu/tests/docker/docker.py", line 247, in _do_check
return subprocess.check_call(self._command + cmd, **kwargs)
File "/usr/lib64/python3.10/subprocess.py", line 369, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['podman', 'build', '-t', 'qemu/debian-s390x-cross', '-f', '/tmp/docker_buildt03fabg5/tmp3xpdzx90.docker', '--cache-from', 'registry.gitlab.com/qemu-project/qemu/qemu/debian-s390x-cross', '/tmp/docker_buildt03fabg5']' returned non-zero exit status 100.
make[1]: *** [/home/berrange/src/virt/qemu/tests/docker/Makefile.include:38: docker-image-debian-s390x-cross] Error 1
make[1]: Leaving directory '/home/berrange/src/virt/qemu/build'
make: *** [GNUmakefile:11: all] Error 2
With regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
next prev parent reply other threads:[~2022-10-10 13:10 UTC|newest]
Thread overview: 64+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-04 13:00 [PULL 00/54] testing, gdbstub, plugin and gitdm updates Alex Bennée
2022-10-04 13:00 ` [PULL 01/54] scripts/ci/setup: ninja missing from build-environment Alex Bennée
2022-10-04 13:00 ` [PULL 02/54] scripts/ci/setup: Fix libxen requirements Alex Bennée
2022-10-04 13:00 ` [PULL 03/54] scripts/ci/setup: spice-server only on x86 aarch64 Alex Bennée
2022-10-04 13:00 ` [PULL 04/54] tests/docker: run script use realpath instead of readlink Alex Bennée
2022-10-04 13:00 ` [PULL 05/54] configure: move detected gdb to TCG's config-host.mak Alex Bennée
2022-10-04 13:00 ` [PULL 06/54] target/hexagon: add flex/bison/glib2 to qemu.yml Alex Bennée
2022-10-04 13:00 ` [PULL 07/54] target/hexagon: regenerate docker/cirrus files Alex Bennée
2022-10-04 13:00 ` [PULL 08/54] target/hexagon: manually add flex/bison/glib2 to remaining containers Alex Bennée
2022-10-04 13:00 ` [PULL 09/54] tests/docker: update fedora-win[32|64]-cross with lcitool Alex Bennée
2022-10-04 13:00 ` [PULL 10/54] tests/docker: move alpine from edge to tagged release Alex Bennée
2022-10-04 13:00 ` [PULL 11/54] tests/qtest: bump up QOS_PATH_MAX_ELEMENT_SIZE Alex Bennée
2022-10-04 13:00 ` [PULL 12/54] configure: do not invoke as/ld directly for pc-bios/optionrom Alex Bennée
2022-10-04 13:00 ` [PULL 13/54] pc-bios/optionrom: detect CC options just once Alex Bennée
2022-10-04 13:00 ` [PULL 14/54] pc-bios/s390-ccw: " Alex Bennée
2022-10-04 13:00 ` [PULL 15/54] vof: add distclean target Alex Bennée
2022-10-04 13:01 ` [PULL 16/54] build: add recursive distclean rules Alex Bennée
2022-10-04 13:01 ` [PULL 17/54] configure: return status code from probe_target_compiler Alex Bennée
2022-10-04 13:01 ` [PULL 18/54] configure: store container engine in config-host.mak Alex Bennée
2022-10-04 13:01 ` [PULL 19/54] tests: simplify Makefile invocation for tests/tcg Alex Bennée
2022-10-04 13:01 ` [PULL 20/54] tests/tcg: remove -f from Makefile invocation Alex Bennée
2022-10-04 13:01 ` [PULL 21/54] tests/tcg: add distclean rule Alex Bennée
2022-10-04 13:01 ` [PULL 22/54] tests/tcg: unify ppc64 and ppc64le Makefiles Alex Bennée
2022-10-04 13:01 ` [PULL 23/54] tests/tcg: clean up calls to run-test Alex Bennée
2022-10-04 13:01 ` [PULL 24/54] tests/tcg: move compiler tests to Makefiles Alex Bennée
2022-10-04 13:01 ` [PULL 25/54] configure: move tests/tcg/Makefile.prereqs to root build directory Alex Bennée
2022-10-04 13:01 ` [PULL 26/54] configure: unify creation of cross-compilation Makefiles Alex Bennée
2022-10-06 20:33 ` Stefan Hajnoczi
2022-10-04 13:01 ` [PULL 27/54] configure: cleanup creation of tests/tcg target config Alex Bennée
2022-10-04 13:01 ` [PULL 28/54] configure: build ROMs with container-based cross compilers Alex Bennée
2022-10-06 20:37 ` Stefan Hajnoczi
2022-10-10 12:54 ` Daniel P. Berrangé
2022-10-10 13:07 ` Daniel P. Berrangé [this message]
2022-10-10 13:47 ` Alex Bennée
2022-10-04 13:01 ` [PULL 29/54] pc-bios/optionrom: Adopt meson style Make output Alex Bennée
2022-10-04 13:01 ` [PULL 30/54] pc-bios/s390-ccw: " Alex Bennée
2022-10-04 13:01 ` [PULL 31/54] pc-bios/vof: " Alex Bennée
2022-10-04 13:01 ` [PULL 32/54] monitor: expose monitor_puts to rest of code Alex Bennée
2022-10-04 13:01 ` [PULL 33/54] disas: generalise plugin_printf and use for monitor_disas Alex Bennée
2022-10-04 13:01 ` [PULL 34/54] disas: use result of ->read_memory_func Alex Bennée
2022-10-04 13:01 ` [PULL 35/54] plugins: extend execlog to filter matches Alex Bennée
2022-10-04 13:01 ` [PULL 36/54] plugins: Assert mmu_idx in range before use in qemu_plugin_get_hwaddr Alex Bennée
2022-10-04 13:01 ` [PULL 37/54] docs/devel: clean-up qemu invocations in tcg-plugins Alex Bennée
2022-10-04 13:01 ` [PULL 38/54] docs/devel: move API to end of tcg-plugins.rst Alex Bennée
2022-10-04 13:01 ` [PULL 39/54] contrib/plugins: reset skip when matching in execlog Alex Bennée
2022-10-04 13:01 ` [PULL 40/54] docs/devel: document the test plugins Alex Bennée
2022-10-04 13:01 ` [PULL 41/54] semihosting: update link to spec Alex Bennée
2022-10-04 13:01 ` [PULL 42/54] gdbstub: move into its own sub directory Alex Bennée
2022-10-04 13:01 ` [PULL 43/54] gdbstub: move sstep flags probing into AccelClass Alex Bennée
2022-10-04 13:01 ` [PULL 44/54] gdbstub: move breakpoint logic to accel ops Alex Bennée
2022-10-04 13:01 ` [PULL 45/54] gdbstub: move guest debug support check to ops Alex Bennée
2022-10-04 13:01 ` [PULL 46/54] accel/kvm: move kvm_update_guest_debug to inline stub Alex Bennée
2022-10-04 13:01 ` [PULL 47/54] contrib/gitdm: add mapping for Loongson Technology Alex Bennée
2022-10-04 13:01 ` [PULL 48/54] contrib/gitdm: add Paul to individual contributors Alex Bennée
2022-10-04 13:01 ` [PULL 49/54] contrib/gitdm: add WANG Xuerui to individual contributers Alex Bennée
2022-10-04 13:01 ` [PULL 50/54] contrib/gitdm: add ISCAS to the academics group Alex Bennée
2022-10-04 13:01 ` [PULL 51/54] contrib/gitdm: add China Telecom to the domain map Alex Bennée
2022-10-04 13:01 ` [PULL 52/54] contrib/gitdm: add Simon to individual contributors Alex Bennée
2022-10-04 16:06 ` Simon Safar via
2022-10-04 13:01 ` [PULL 53/54] contrib/gitdm: add Université Grenoble Alpes Alex Bennée
2022-10-04 13:01 ` [PULL 54/54] plugins: add [pre|post]fork helpers to linux-user Alex Bennée
2022-10-05 14:16 ` [PULL 00/54] testing, gdbstub, plugin and gitdm updates Stefan Hajnoczi
2022-10-05 15:23 ` Alex Bennée
2022-10-05 23:17 ` Paolo Bonzini
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=Y0QZKSHRDLlOHEgP@redhat.com \
--to=berrange@redhat.com \
--cc=alex.bennee@linaro.org \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.