All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Alex Bennée" <alex.bennee@linaro.org>
To: Richard Henderson <richard.henderson@linaro.org>
Cc: qemu-devel@nongnu.org,  Ilya Leoshkevich <iii@linux.ibm.com>
Subject: Re: [PATCH v3 00/17] testing: various docker clean-ups post meson tcg conversion
Date: Wed, 16 Sep 2026 16:44:28 +0100	[thread overview]
Message-ID: <871pat9p5v.fsf@draig.linaro.org> (raw)
In-Reply-To: <63ab8536-5e31-4101-94f4-005bbc6eda27@linaro.org> (Richard Henderson's message of "Wed, 16 Sep 2026 00:47:31 -1000")

Richard Henderson <richard.henderson@linaro.org> writes:

> On 9/15/26 22:53, 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.
>> v3
>>   - Update make2test to trigger docker build from Makefile.include.
>>   - add Daniels's suggested console: true patch
>>   - fixed up some unref vars from the reverts
>> v2
>> I've restored the configure based compiler checks for so we can
>> build
>> roms and the vdso blobs again. It looks like we could never build roms
>> with container based images so I've left it for now. Unlike the vdso
>> build we can't trigger a container build through Makefile.prereqs.
>> TODO
>> ----
>> I've still got Ilya's s390 test image patches to look at but I'm
>> sending v2 because enough people are hitting issues.
>> I also need to look at the timestamp stuff that Pierrick has
>> suggested
>> to see if that improves the caching from the registry.
>> The following still need review:
>>    tests/tcg: add console property to docker image custom target
>>    scripts/make2mtest: add docker dependencies for check-tcg-% targets
>>    tests/docker: fix brown bag typo for docker-ccache
>>    tests/docker: swap --quiet for --verbose
>>    tests/docker: add container registry to configure and use
>>    tests/tcg: use --add-current-user when building containers
>>    configure: re-purpose the tcg test compiler configs for vdso building
>>    Revert "tests/tcg/meson.build: implement ./configure -cross-cc-* options"
>
> Still missing 20260915110855.491985-1-iii@linux.ibm.com
> ("[PATCH] tests/tcg: Fix building outside of the source tree")
>
> which is second behind --disable-containers in my current pain points.

Both those and the SKIP issue are addressed in v4.

>
>
> r~

-- 
Alex Bennée
Virtualisation Tech Lead @ Linaro


      reply	other threads:[~2026-09-16 15:45 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-16  8:53 [PATCH v3 00/17] testing: various docker clean-ups post meson tcg conversion Alex Bennée
2026-09-16  8:53 ` [PATCH v3 01/17] Revert "configure: remove cross_cc and gdb detection logic" Alex Bennée
2026-09-16  8:53 ` [PATCH v3 02/17] Revert "tests/tcg/meson.build: implement ./configure -cross-cc-* options" Alex Bennée
2026-09-16  8:53 ` [PATCH v3 03/17] Revert "tests/tcg/meson.build: implement ./configure containers options" Alex Bennée
2026-09-16 13:32   ` Jared Rossi
2026-09-16  8:53 ` [PATCH v3 04/17] configure: correctly honour --disable-containers Alex Bennée
2026-09-16  8:53 ` [PATCH v3 05/17] configure: re-purpose the tcg test compiler configs for vdso building Alex Bennée
2026-09-16  8:53 ` [PATCH v3 06/17] tests/tcg: improve check for working cross compilers Alex Bennée
2026-09-16  8:53 ` [PATCH v3 07/17] tests/tcg: prefix meson built docker tags with qemu/ Alex Bennée
2026-09-16  8:53 ` [PATCH v3 08/17] tests/tcg: use --add-current-user when building containers Alex Bennée
2026-09-16  8:53 ` [PATCH v3 09/17] tests/docker: add container registry to configure and use Alex Bennée
2026-09-16  8:53 ` [PATCH v3 10/17] tests/docker: remove used _read_qemu_dockerfile Alex Bennée
2026-09-16  8:53 ` [PATCH v3 11/17] tests/docker: strip remaining caching logic from docker.py Alex Bennée
2026-09-16  8:53 ` [PATCH v3 12/17] tests/docker: add specific command to parser Alex Bennée
2026-09-16  8:53 ` [PATCH v3 13/17] tests/docker: swap --quiet for --verbose Alex Bennée
2026-09-16  8:53 ` [PATCH v3 14/17] tests/docker: fix brown bag typo for docker-ccache Alex Bennée
2026-09-16  8:53 ` [PATCH v3 15/17] gitlab: update the @user id mapping file Alex Bennée
2026-09-16  8:53 ` [PATCH v3 16/17] scripts/make2mtest: add docker dependencies for check-tcg-% targets Alex Bennée
2026-09-16  9:56   ` Pierrick Bouvier
2026-09-16 10:03     ` Daniel P. Berrangé
2026-09-16 15:24       ` Alex Bennée
2026-09-16  8:53 ` [PATCH v3 17/17] tests/tcg: add console property to docker image custom target Alex Bennée
2026-09-16 10:47 ` [PATCH v3 00/17] testing: various docker clean-ups post meson tcg conversion Richard Henderson
2026-09-16 15:44   ` Alex Bennée [this message]

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=871pat9p5v.fsf@draig.linaro.org \
    --to=alex.bennee@linaro.org \
    --cc=iii@linux.ibm.com \
    --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 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.