Re: [libvirt PATCH v2 6/9] gitlab: add x86_64 native CI jobs

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

 



On Wed, 2020-03-25 at 09:35 +0100, Erik Skultety wrote:
> On Tue, Mar 24, 2020 at 06:21:24PM +0000, Daniel P. Berrangé wrote:
> > On Tue, Mar 24, 2020 at 06:47:01PM +0100, Andrea Bolognani wrote:
> > >   * pick a selection of jobs that includes both native and cross
> > >     build, across various distros of different vintage, such that
> > >     they can all run without waiting on shared runners. This can be
> > >     used by developers as a reasonably quick (~20 minutes) smoke
> > >     test while working on a feature;
> > 
> > "without waiting on shared runners" is undefined, as whether you
> > wait & how long, is dependant on many things outside our control.
> > As notedin the cover letter though, this minimal set of native
> > + cross build jobs gives about a 35 min turn around based on
> > load I see today. I think that's acceptably fast.
> 
> Related to the availability of shared runners, for merge_requests/post_merge
> builds only, the PSI/OpenShift infra may actually help in order to achieve
> more stable execution times due to the usage of private runners.

As explained by Dan, that will not help for the merge request builds,
because those are executed in the context of the submitter's fork and
as such don't have access to any runners we might deploy ourselves.

It would help speed up the post merge phase, but once we move to a
merge request workflow we'll want to test as much as possible at the
time the merge request is submitted, and there won't be much that
needs to happen after the changes hit master... So dedicated runners
might actually not be useful outside of supporting non-Linux targets.

-- 
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