All of lore.kernel.org
 help / color / mirror / Atom feed
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 :|



  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.