Re: [libvirt PATCH 4/5] gitlab: add several native CI jobs

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

 



On Fri, Mar 20, 2020 at 03:52:15PM +0100, Erik Skultety wrote:
> On Tue, Mar 10, 2020 at 10:09:44AM +0000, Daniel P. Berrangé wrote:
> > With GitLab CI aiming to replace Jenkins and Travis for CI purposes, we
> > need to expand the coverage to include native builds. This patch adds
> > all the jobs currently run in Travis. Compared to Jenkins we obviously
> > miss the FreeBSD jobs, but also Debian 10 and Fedora 30, but we gain the
> > Ubuntu 1804 job as a substitute for Debian.
> >
> > Signed-off-by: Daniel P. Berrangé <berrange@xxxxxxxxxx>
> > ---
> >  .gitlab-ci.yml | 41 +++++++++++++++++++++++++++++++++++++++++
> >  1 file changed, 41 insertions(+)
> >
> > diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
> > index e28ec584ea..3e15d08d17 100644
> > --- a/.gitlab-ci.yml
> > +++ b/.gitlab-ci.yml
> > @@ -1,7 +1,39 @@
> >  stages:
> >    - website
> > +  - native_build
> >    - cross_build
> >
> > +
> > +.native_build_job_template: &native_build_job_definition
> > +  stage: native_build
> > +  script:
> > +    - mkdir build
> > +    - cd build
> > +    - ../autogen.sh $CONFIGURE_OPTS || (cat config.log && exit 1)
> > +    - make -j $(getconf _NPROCESSORS_ONLN) syntax-check
> > +    - make -j $(getconf _NPROCESSORS_ONLN) distcheck
> 
> I think ^this should more closely follow what we have in the lcitool playbooks,
> e.g. start with:
>     - rm -rf build

The source tree is already pristine because this is always executed in
a fresh container environment, so there's nothing that will need deleting.

> Also, since I've been playing with migrating other machines to PSI for a while,
> 'make' should be replaced with $MAKE otherwise native_build job reference won't
> work on FreeBSD.

I'll need to check if $MAKE is actually set or not.

> Maybe even do make install to VIRT_PREFIX?

'distcheck' does an install step.  There's no shared install tree between
jobs, so the VIRT_PREFIX concept isn't applicable.

> Otherwise looks good to me.
> Reviewed-by: Erik Skultety <skultety.erik@xxxxxxxxx>

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





[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