Re: [libvirt-jenkins-ci PATCH 1/3] lcitool: Include paths in the inventory

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

 



On Fri, Mar 27, 2020 at 06:26:25PM +0100, Andrea Bolognani wrote:
> Figuring them out at runtime is neat, but in practice they don't
> change frequently enough for it to be necessary; more importantly,
> including them in the inventory means we can use them in the
> Dockerfile generator in addition to the Ansible playbooks.
> 
> Signed-off-by: Andrea Bolognani <abologna@xxxxxxxxxx>
> ---
>  guests/host_vars/libvirt-centos-7/main.yml    |  9 +++
>  guests/host_vars/libvirt-centos-8/main.yml    |  9 +++
>  guests/host_vars/libvirt-debian-10/main.yml   |  9 +++
>  guests/host_vars/libvirt-debian-9/main.yml    |  9 +++
>  guests/host_vars/libvirt-debian-sid/main.yml  |  9 +++
>  guests/host_vars/libvirt-fedora-30/main.yml   |  9 +++
>  guests/host_vars/libvirt-fedora-31/main.yml   |  9 +++
>  .../host_vars/libvirt-fedora-rawhide/main.yml |  9 +++
>  guests/host_vars/libvirt-freebsd-11/main.yml  |  9 +++
>  guests/host_vars/libvirt-freebsd-12/main.yml  |  9 +++
>  .../libvirt-freebsd-current/main.yml          |  9 +++
>  .../host_vars/libvirt-opensuse-151/main.yml   |  9 +++
>  guests/host_vars/libvirt-ubuntu-1604/main.yml |  9 +++
>  guests/host_vars/libvirt-ubuntu-1804/main.yml |  9 +++
>  guests/playbooks/update/main.yml              |  1 -
>  guests/playbooks/update/tasks/paths.yml       | 71 -------------------
>  16 files changed, 126 insertions(+), 72 deletions(-)
>  delete mode 100644 guests/playbooks/update/tasks/paths.yml


Reviewed-by: Daniel P. Berrangé <berrange@xxxxxxxxxx>


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