All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Roger Pau Monné" <roger.pau@citrix.com>
To: Stefano Stabellini <sstabellini@kernel.org>
Cc: xen-devel@lists.xenproject.org, Doug Goldstein <cardoe@cardoe.com>
Subject: Re: [PATCH 6/6] automation: switch to multi-platform images when possible
Date: Fri, 17 Nov 2023 11:15:13 +0100	[thread overview]
Message-ID: <ZVc9MZQ3lJpL2Nz1@macbook.local> (raw)
In-Reply-To: <alpine.DEB.2.22.394.2311161711560.773207@ubuntu-linux-20-04-desktop>

On Thu, Nov 16, 2023 at 05:14:23PM -0800, Stefano Stabellini wrote:
> On Thu, 16 Nov 2023, Roger Pau Monne wrote:
> > Instead of using specific architecture image, switch to using multi-arch ones
> > and specify the desired architecture using the --platform option.
> > 
> > Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
> > ---
> > I haven't touched the Yocto dockerfile because I'm not sure how it's used.
> 
> We are missing:
> 
> automation/build/debian/buster-gcc-ibt.dockerfile

That file was updated in patch 5/6:

https://lore.kernel.org/xen-devel/20231116121310.72210-6-roger.pau@citrix.com/

> automation/build/debian/bookworm-cppcheck.dockerfile

Not sure I'm following, bookworm-cppcheck.dockerfile is updated...

> automation/tests-artifacts/*

Oh, didn't realize about those, I will do in a separate patch.

> Aside from that, it is fine.
> 
> How did you test the updated containers? Have you already pushed them to
> the registry?

I've pushed them to my local registry and changed the registry in one
of my Xen branches, see:

https://gitlab.com/xen-project/people/royger/xen/-/pipelines/1074512137

Some jobs failed because the runners run out of space.

Thanks, Roger.


  reply	other threads:[~2023-11-17 10:15 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-11-16 12:13 [PATCH 0/6] automation: minor fixes and improvements Roger Pau Monne
2023-11-16 12:13 ` [PATCH 1/6] automation: remove CR characters from QEMU serial Roger Pau Monne
2023-11-17  1:05   ` Stefano Stabellini
2023-11-17  1:07     ` Stefano Stabellini
2023-11-17 10:07     ` Roger Pau Monné
2023-11-18  2:25       ` Stefano Stabellini
2023-11-16 12:13 ` [PATCH 2/6] automation: add timestamps to QEMU non-smoke x86-64 test Roger Pau Monne
2023-11-17  1:07   ` Stefano Stabellini
2023-11-16 12:13 ` [PATCH 3/6] automation: fix jessie/stretch images to use archive.debian.org apt repos Roger Pau Monne
2023-11-17  1:15   ` Stefano Stabellini
2023-11-16 12:13 ` [PATCH 4/6] automation: update tests to use Debian Bookworm Roger Pau Monne
2023-11-17  1:09   ` Stefano Stabellini
2023-11-16 12:13 ` [PATCH 5/6] automation: set architecture in docker files Roger Pau Monne
2023-11-16 12:13 ` [PATCH 6/6] automation: switch to multi-platform images when possible Roger Pau Monne
2023-11-17  1:14   ` Stefano Stabellini
2023-11-17 10:15     ` Roger Pau Monné [this message]
2023-11-18  2:27       ` Stefano Stabellini
2023-11-20 11:19         ` Roger Pau Monné

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=ZVc9MZQ3lJpL2Nz1@macbook.local \
    --to=roger.pau@citrix.com \
    --cc=cardoe@cardoe.com \
    --cc=sstabellini@kernel.org \
    --cc=xen-devel@lists.xenproject.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.