Re: [libvirt-ci PATCH v2] guests: add libvirt-php project packages

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

 



On Tue, May 05, 2020 at 03:31:55PM +0200, Andrea Bolognani wrote:
> On Tue, 2020-05-05 at 14:19 +0100, Daniel P. Berrangé wrote:
> > On Tue, May 05, 2020 at 03:05:42PM +0200, Andrea Bolognani wrote:
> > > This patch is also missing the part where the libvirt-php project is
> > > added to the 'projects' fact for all targets that actually support
> > > building it, which would be all of them but CentOS 8 in this case I
> > > guess?
> > > 
> > > We're also missing build steps for the project. I understand that,
> > > as we move stuff off CentOS CI and into GitLab CI containers become
> > > more important than VMs, but I think keeping 'lcitool build' working
> > > is still important, if nothing else as a development tool. It
> > > certainly makes my life easier as a reviewer... Not to mention it is
> > > the only way we can currently validate FreeBSD changes.
> > 
> > There's a bunch of projects we don't have in Jenkins CI right now, and
> > I'm trying to get them all wired up into GitLab CI. I don't want to
> > invest any more time in VM build aspects right now, as I don't have
> > the time to spend testing & debugging that part too. So I'm only going
> > to do the minimal required to get container builds work, which is defining
> > the package lists used to feed the container build. The extra VM build
> > stuff can be done separately at a later date if we decide we need that
> > still for CI.
> 
> I guess I'll have to put my money where my mouth is and add the build
> steps myself. That's fair. Please at least update the inventory so
> that it accurately records which targets are supposed to be able to
> build the new project.

That's only required for VM builds, and would  use of the "all"
wildcard when building VMs, so I'm not wanting to do that, as I
won't have tested anything related to VMs.

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