Re: [PATCH 2/3] .gitlab-ci.d: Drop cross-win32-system job

[Thomas Huth]

On 20/02/2024 19.55, Daniel P. Berrangé wrote:
> On Tue, Feb 20, 2024 at 07:46:37PM +0100, Philippe Mathieu-Daudé wrote:
> > On 20/2/24 18:44, Peter Maydell wrote:
> > > We don't support 32-bit Windows any more, so we don't need to defend it
> > > with this CI job.
> > >
> > > Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> > > ---
> > >    .gitlab-ci.d/container-cross.yml              |   5 -
> > >    .gitlab-ci.d/crossbuilds.yml                  |  14 ---
> > >    .../dockerfiles/fedora-win32-cross.docker     | 111 ------------------
> > >    tests/lcitool/refresh                         |   5 -
> > >    4 files changed, 135 deletions(-)
> > >    delete mode 100644 tests/docker/dockerfiles/fedora-win32-cross.docker
> >
> >
> > > diff --git a/.gitlab-ci.d/crossbuilds.yml b/.gitlab-ci.d/crossbuilds.yml
> > > index d19d98cde05..987ba9694ba 100644
> > > --- a/.gitlab-ci.d/crossbuilds.yml
> > > +++ b/.gitlab-ci.d/crossbuilds.yml
> > > @@ -159,20 +159,6 @@ cross-mips64el-kvm-only:
> > >        IMAGE: debian-mips64el-cross
> > >        EXTRA_CONFIGURE_OPTS: --disable-tcg --target-list=mips64el-softmmu
> > > -cross-win32-system:
> > > -  extends: .cross_system_build_job
> > > -  needs:
> > > -    job: win32-fedora-cross-container
> > > -  variables:
> > > -    IMAGE: fedora-win32-cross
> > > -    EXTRA_CONFIGURE_OPTS: --enable-fdt=internal
> > > -    CROSS_SKIP_TARGETS: alpha-softmmu avr-softmmu hppa-softmmu m68k-softmmu
> > > -                        microblazeel-softmmu mips64el-softmmu nios2-softmmu
> >
> > Thomas, is it possible to add mips64el-softmmu to the cross-win64-system
> > job or is it already at the timeout limit?
>
> If its too close to the limit, we could add  across-win64-system-part2
> job now we freed time from the deleting the win32 job.

Looking at a recent run:

 https://gitlab.com/qemu-project/qemu/-/jobs/6189303285

It seems like we're in the 50 minutes range here, and timeout is set to 
1h20m, so I think there should still be enough headroom for adding 
mips64el-softmmu to that job.

 Thomas