Re: [PATCH] shippable.yml: Remove jobs duplicated on Gitlab-CI

[Daniel P . Berrangé]

On Fri, Jan 08, 2021 at 04:14:30PM +0100, Paolo Bonzini wrote:
> On 08/01/21 15:51, Philippe Mathieu-Daudé wrote:
> > The following jobs are duplicated on Gitlab-CI since commit
> > 6bcb5fc0f7a ("gitlab-ci: Add cross-compiling build tests"):
> > 
> > - IMAGE=debian-armel-cross
> > 
> >    TARGET_LIST=arm-softmmu               -> cross-armel-system
> >    TARGET_LIST=arm-linux-user            -> cross-armel-user
> >    TARGET_LIST=armeb-linux-user          -> cross-armel-user
> > 
> > - IMAGE=debian-armhf-cross
> > 
> >    TARGET_LIST=arm-softmmu               -> cross-armhf-system
> >    TARGET_LIST=arm-linux-user            -> cross-armhf-user
> >    TARGET_LIST=armeb-linux-user          -> cross-armhf-user
> > 
> > - IMAGE=debian-arm64-cross
> > 
> >    TARGET_LIST=aarch64-softmmu           -> cross-arm64-system
> >    TARGET_LIST=aarch64-linux-user        -> cross-arm64-user
> > 
> > - IMAGE=debian-s390x-cross
> > 
> >    TARGET_LIST=s390x-softmmu             -> cross-s390x-system
> >    TARGET_LIST=s390x-linux-user          -> cross-s390x-user
> > 
> > - IMAGE=debian-mips-cross
> > 
> >    TARGET_LIST=mipsel-linux-user         -> cross-mips-user
> > 
> > - IMAGE=debian-mips64el-cross
> > 
> >    TARGET_LIST=mips64el-softmmu          -> cross-mips64el-system
> >    TARGET_LIST=mips64el-linux-user       -> cross-mips64el-user
> > 
> > - IMAGE=debian-ppc64el-cross
> > 
> >    TARGET_LIST=ppc64-softmmu             -> cross-ppc64el-system
> >    TARGET_LIST=ppc64-linux-user          -> cross-ppc64el-user
> >    TARGET_LIST=ppc64abi32-linux-user     -> cross-ppc64el-user
> > 
> > Remove them from Shippable CI.
> > 
> > Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> > ---
> >   .shippable.yml | 14 +-------------
> >   1 file changed, 1 insertion(+), 13 deletions(-)
> > 
> > diff --git a/.shippable.yml b/.shippable.yml
> > index 14350e6de8a..97bfa2a0f38 100644
> > --- a/.shippable.yml
> > +++ b/.shippable.yml
> > @@ -7,20 +7,8 @@ env:
> >     matrix:
> >       - IMAGE=debian-amd64
> >         TARGET_LIST=x86_64-softmmu,x86_64-linux-user
> > -    - IMAGE=debian-armel-cross
> > -      TARGET_LIST=arm-softmmu,arm-linux-user,armeb-linux-user
> > -    - IMAGE=debian-armhf-cross
> > -      TARGET_LIST=arm-softmmu,arm-linux-user,armeb-linux-user
> > -    - IMAGE=debian-arm64-cross
> > -      TARGET_LIST=aarch64-softmmu,aarch64-linux-user
> > -    - IMAGE=debian-s390x-cross
> > -      TARGET_LIST=s390x-softmmu,s390x-linux-user
> >       - IMAGE=debian-mips-cross
> > -      TARGET_LIST=mips-softmmu,mipsel-linux-user
> > -    - IMAGE=debian-mips64el-cross
> > -      TARGET_LIST=mips64el-softmmu,mips64el-linux-user
> > -    - IMAGE=debian-ppc64el-cross
> > -      TARGET_LIST=ppc64-softmmu,ppc64-linux-user,ppc64abi32-linux-user
> > +      TARGET_LIST=mips-softmmu
> 
> The debian-mips-cross/mips-softmmu pair (and also mipsel-softmmu) is
> currently excluded by Gitlab's cross-mips-system, is there a reason for
> that?  If we add it there, we can just remove shippable altogether.

Some softmmu targets were disable to keep the build time under control.

We could enable a different subset of targets for each cross host arch,
so that overall we get representative coverage of the full set.

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|