Re: [jenkins-ci PATCH 2/3] guests: Don't expose 'jenkins' pseudo-package

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

 



On Wed, 2018-03-21 at 15:56 +0100, Pavel Hrdina wrote:
> > +    # Install packages needed for the Jenkins agent
> > +    - include: tasks/packages.yml
> > +      vars:
> > +        project: jenkins
> > +      when:
> > +        - flavor == "jenkins"
> > +
> >      # Configure the Jenkins agent
> >      - include: tasks/jenkins.yml
> >        when:
> 
> Would it make sense to move the installation of packages needed
> for Jenkins agent into the jenkins task to create a single unit
> out of it?

Maybe. However, I really like the current flat structure, where
there is no task that includes another task, and site.yml is the
one including everything as needed. I think it makes it easier to
understand what's going on, so I'd prefer keeping it this way.

-- 
Andrea Bolognani / Red Hat / Virtualization

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[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