Re: [PATCH 6/6] automation: switch to multi-platform images when possible

[Roger Pau Monné <roger.pau@xxxxxxxxxx>]

On Fri, Nov 17, 2023 at 06:27:50PM -0800, Stefano Stabellini wrote:
> On Fri, 17 Nov 2023, Roger Pau Monné wrote:
> > 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@xxxxxxxxxx>
> > > > ---
> > > > 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@xxxxxxxxxx/
> > 
> > > 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.
> 
> Thanks!
> 
> 
> > > 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.
> 
> Oh, OK. It is going to be a lot of work to rebuild and push all the
> containers and I wouldn't mind you doing that once the patches are
> acked. In fact it would be great if you pushed the containers once you
> tests that they work as expected. If you don't have the right access
> permissions, I can do that too

For the patches only the x86 bookworm container needs to be rebuild in
order to contain the packages required by the test step.

The rest can be updated either before or after the series has been
committed.

Thanks, Roger.