Re: [libvirt PATCH 3/5] ci: Use GitLab container registry

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 2020-06-02 at 11:33 +0100, Daniel P. Berrangé wrote:
> On Fri, May 29, 2020 at 03:00:42PM +0200, Andrea Bolognani wrote:
> > +# We build many containers which can be useful to debug problems but are not
> > +# needed for the pipeline itself to complete: those sometimes fail, and when
> > +# that happens it's mostly because of temporary issues with Debian sid. We
> > +# don't want those failures to affect the overall pipeline status
> > +.container_optional_job_template: &container_optional_job_definition
> > +  <<: *container_job_definition
> > +  allow_failure: true
> 
> I don't think we should be building container images that we're not going
> to be using in any of the jobs, as it can only ever slow down the build
> overall.

These same containers are also available for use outside of CI, eg.
with 'make ci-build', so I think we should keep building them.

As for slowing down builds, that still only applies to the first
build after Dockerfiles are updated, so I don't think it ultimately
matters very much.

> > +# Container build jobs
> > +
> > +centos-7-container:
> 
> IMHO we should name these to match the build  job. eg arch, then distro
> 
>    x64-centos-7-container

Okay.

> > +debian-9-cross-armv6l-container:
> > +  <<: *container_job_definition
> > +  variables:
> > +    NAME: debian-9-cross-armv6l
> 
> This container, and many others are only used by the "extra" build
> jobs, so should be subject to the same filtering.

Okay, even though as we discussed separately the whole idea of
splitting jobs between regular and extra might be more trouble than
it's worth and be more confusing than helpful.

-- 
Andrea Bolognani / Red Hat / Virtualization




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux