From: "Alex Bennée" <alex.bennee@linaro.org>
To: Richard Henderson <richard.henderson@linaro.org>
Cc: qemu-devel@nongnu.org
Subject: Re: [PATCH 0/9] testing: various docker clean-ups post meson tcg conversion
Date: Sat, 12 Sep 2026 18:24:35 +0100 [thread overview]
Message-ID: <878q562xik.fsf@draig.linaro.org> (raw)
In-Reply-To: <4e17f3d5-94df-44c4-a284-12422eda0446@linaro.org> (Richard Henderson's message of "Fri, 11 Sep 2026 22:46:04 -1000")
Richard Henderson <richard.henderson@linaro.org> writes:
> On 9/10/26 08:14, Alex Bennée wrote:
>> This cleans up a few bits and pieces although doesn't totally solve
>> the issue of an apparent stall while meson builds the docker image.
>> However with these changes there should be no difference between the
>> meson build and the equivalent "make docker-image-foo" invocation.
>> We also finally get rid of the ill-advised magic caching support in
>> docker.py.
>> Please review.
>> Alex.
>> Alex Bennée (8):
>> scripts/meson-buildoptions.sh: update missing options
>> tests/tcg: prefix meson built docker tags with qemu/
>> tests/tcg: use --add-current-user when building containers
>> tests/docker: add container registry to configure and use
>> tests/docker: remove used _read_qemu_dockerfile
>> tests/docker: strip remaining caching logic from docker.py
>> tests/docker: add specific command to parser
>> tests/docker: swap --quiet for --verbose
> I haven't had time to pull this into a local branch, but I don't
> suppose something in here fixes
>
>> [6/572] Generating tests/tcg/alpha-softmmu-hello with a custom command
>> FAILED: [code=1] tests/tcg/alpha-softmmu-hello.test
>> /home/rth/qemu/src/tests/tcg/../docker/docker.py --command podman
>> run --run-as-current-user -w /home/rth/qemu/bld-g -v
yes - the -v is missing it's args which is fixed by the argparse patch
before I switched --quiet and --verbose which exposed the latent bug.
>> /home/rth/qemu/src:/home/rth/qemu/src debian-all-test-cross
>> alpha-linux-gnu-gcc
>> /home/rth/qemu/src/tests/tcg/multiarch/system/hello.c -o
>> tests/tcg/alpha-softmmu-hello.test -static -MMD -MF
>> tests/tcg/alpha-softmmu-hello.d -Wall -Werror -O0 -g
>> -fno-strict-aliasing -nostdlib -ffreestanding -Wa,--noexecstack -I
>> /home/rth/qemu/src/tests/tcg/alpha/system/../../minilib
>> /home/rth/qemu/src/tests/tcg/alpha/system/../../minilib/printf.c
>> ../src/tests/tcg/alpha/system/boot.S -Wl,-T
>> ../src/tests/tcg/alpha/system/kernel.ld -mcpu=ev6
>> -Wl,--no-warn-rwx-segments -lgcc
>> Error: workdir "/home/rth/qemu/bld-g" does not exist on container 7670fc4a955a94d801b57dc0a6f6cd13ece35536a09b40634a37a83d733af942
>> Traceback (most recent call last):
>> File "/home/rth/qemu/src/tests/tcg/../docker/docker.py", line 657, in <module>
>> sys.exit(main())
>> ~~~~^^
>> File "/home/rth/qemu/src/tests/tcg/../docker/docker.py", line 653, in main
>> return args.cmdobj.run(args, argv)
>> ~~~~~~~~~~~~~~~^^^^^^^^^^^^
>> File "/home/rth/qemu/src/tests/tcg/../docker/docker.py", line 395, in run
>> return Docker(args.command).run(argv, args.keep, quiet=args.quiet,
>> ~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>> as_user=args.run_as_current_user)
>> ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>> File "/home/rth/qemu/src/tests/tcg/../docker/docker.py", line 353, in run
>> ret = self._do_check(["run", "--rm", "--label",
>> "com.qemu.instance.uuid=" + label] + cmd,
>> quiet=quiet)
>> File "/home/rth/qemu/src/tests/tcg/../docker/docker.py", line 230, in _do_check
>> return subprocess.check_call(self._command + cmd, **kwargs)
>> ~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>> File "/usr/lib/python3.14/subprocess.py", line 420, in check_call
>> raise CalledProcessError(retcode, cmd)
>> subprocess.CalledProcessError: Command '['podman', 'run', '--rm', '--label', 'com.qemu.instance.uuid=9ca55b05b3e540f7a0cda353bf6f1086', '--userns=keep-id', '-u', '1000', '-w', '/home/rth/qemu/bld-g', '-v', '/home/rth/qemu/src:/home/rth/qemu/src', 'debian-all-test-cross', 'alpha-linux-gnu-gcc', '/home/rth/qemu/src/tests/tcg/multiarch/system/hello.c', '-o', 'tests/tcg/alpha-softmmu-hello.test', '-static', '-MMD', '-MF', 'tests/tcg/alpha-softmmu-hello.d', '-Wall', '-Werror', '-O0', '-g', '-fno-strict-aliasing', '-nostdlib', '-ffreestanding', '-Wa,--noexecstack', '-I', '/home/rth/qemu/src/tests/tcg/alpha/system/../../minilib', '/home/rth/qemu/src/tests/tcg/alpha/system/../../minilib/printf.c', '../src/tests/tcg/alpha/system/boot.S', '-Wl,-T', '../src/tests/tcg/alpha/system/kernel.ld', '-mcpu=ev6', '-Wl,--no-warn-rwx-segments', '-lgcc']' returned non-zero exit status 126.
>> filter=--filter=label=com.qemu.instance.uuid=9ca55b05b3e540f7a0cda353bf6f1086
>> ninja: build stopped: subcommand failed.
>> make: *** [Makefile:170: run-ninja] Error 1
>
> For avoidance of doubt, this is immediately after
>
> make docker-image-debian-all-test-cross NOCACHE=1 V=1
>
>
> r~
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
next prev parent reply other threads:[~2026-09-12 17:25 UTC|newest]
Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-10 18:14 [PATCH 0/9] testing: various docker clean-ups post meson tcg conversion Alex Bennée
2026-09-10 18:14 ` [PATCH 1/9] scripts/meson-buildoptions.sh: update missing options Alex Bennée
2026-09-10 20:03 ` Fabiano Rosas
2026-09-10 22:17 ` Alex Bennée
2026-09-14 5:00 ` Pierrick Bouvier
2026-09-14 10:22 ` Paolo Bonzini
2026-09-10 18:14 ` [PATCH 2/9] tests/tcg: improve check for working cross compilers Alex Bennée
2026-09-14 5:08 ` Pierrick Bouvier
2026-09-14 8:34 ` Daniel P. Berrangé
2026-09-14 16:05 ` Pierrick Bouvier
2026-09-14 16:07 ` Alex Bennée
2026-09-14 23:55 ` Ilya Leoshkevich
2026-09-15 0:21 ` Pierrick Bouvier
2026-09-10 18:14 ` [PATCH 3/9] tests/tcg: prefix meson built docker tags with qemu/ Alex Bennée
2026-09-14 5:10 ` Pierrick Bouvier
2026-09-10 18:14 ` [PATCH 4/9] tests/tcg: use --add-current-user when building containers Alex Bennée
2026-09-14 5:15 ` Pierrick Bouvier
2026-09-14 7:27 ` Alex Bennée
2026-09-14 15:59 ` Pierrick Bouvier
2026-09-10 18:15 ` [PATCH 5/9] tests/docker: add container registry to configure and use Alex Bennée
2026-09-14 5:21 ` Pierrick Bouvier
2026-09-14 7:28 ` Alex Bennée
2026-09-14 16:15 ` Pierrick Bouvier
2026-09-14 16:18 ` Pierrick Bouvier
2026-09-15 17:10 ` Alex Bennée
2026-09-15 17:49 ` Pierrick Bouvier
2026-09-10 18:15 ` [PATCH 6/9] tests/docker: remove used _read_qemu_dockerfile Alex Bennée
2026-09-14 5:21 ` Pierrick Bouvier
2026-09-10 18:15 ` [PATCH 7/9] tests/docker: strip remaining caching logic from docker.py Alex Bennée
2026-09-14 5:22 ` Pierrick Bouvier
2026-09-10 18:15 ` [PATCH 8/9] tests/docker: add specific command to parser Alex Bennée
2026-09-14 5:22 ` Pierrick Bouvier
2026-09-10 18:15 ` [PATCH 9/9] tests/docker: swap --quiet for --verbose Alex Bennée
2026-09-12 8:46 ` [PATCH 0/9] testing: various docker clean-ups post meson tcg conversion Richard Henderson
2026-09-12 17:24 ` Alex Bennée [this message]
2026-09-12 10:52 ` Richard Henderson
2026-09-12 19:37 ` Alex Bennée
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=878q562xik.fsf@draig.linaro.org \
--to=alex.bennee@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=richard.henderson@linaro.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox