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

Andrea Bolognani abologna at redhat.com
Fri Mar 27 17:26:25 UTC 2020


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 at redhat.com>
---
 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

diff --git a/guests/host_vars/libvirt-centos-7/main.yml b/guests/host_vars/libvirt-centos-7/main.yml
index becd297..7f22d9d 100644
--- a/guests/host_vars/libvirt-centos-7/main.yml
+++ b/guests/host_vars/libvirt-centos-7/main.yml
@@ -20,4 +20,13 @@ package_manager: 'yum'
 os_name: 'CentOS'
 os_version: '7'
 
+bash: /bin/bash
+ccache: /usr/bin/ccache
+java: /usr/bin/java
+make: /usr/bin/make
+ninja: /usr/bin/ninja-build
+python: /usr/bin/python3
+su: /bin/su
+sudoers: /etc/sudoers
+
 ansible_python_interpreter: /usr/bin/python2
diff --git a/guests/host_vars/libvirt-centos-8/main.yml b/guests/host_vars/libvirt-centos-8/main.yml
index cb49e9c..bcf4224 100644
--- a/guests/host_vars/libvirt-centos-8/main.yml
+++ b/guests/host_vars/libvirt-centos-8/main.yml
@@ -20,4 +20,13 @@ package_manager: 'dnf'
 os_name: 'CentOS'
 os_version: '8'
 
+bash: /bin/bash
+ccache: /usr/bin/ccache
+java: /usr/bin/java
+make: /usr/bin/make
+ninja: /usr/bin/ninja
+python: /usr/bin/python3
+su: /bin/su
+sudoers: /etc/sudoers
+
 ansible_python_interpreter: /usr/bin/python3
diff --git a/guests/host_vars/libvirt-debian-10/main.yml b/guests/host_vars/libvirt-debian-10/main.yml
index c5d0047..88e1634 100644
--- a/guests/host_vars/libvirt-debian-10/main.yml
+++ b/guests/host_vars/libvirt-debian-10/main.yml
@@ -22,4 +22,13 @@ package_manager: 'apt-get'
 os_name: 'Debian'
 os_version: '10'
 
+bash: /bin/bash
+ccache: /usr/bin/ccache
+java: /usr/bin/java
+make: /usr/bin/make
+ninja: /usr/bin/ninja
+python: /usr/bin/python3
+su: /bin/su
+sudoers: /etc/sudoers
+
 ansible_python_interpreter: /usr/bin/python3
diff --git a/guests/host_vars/libvirt-debian-9/main.yml b/guests/host_vars/libvirt-debian-9/main.yml
index 3addcd4..a6f1a56 100644
--- a/guests/host_vars/libvirt-debian-9/main.yml
+++ b/guests/host_vars/libvirt-debian-9/main.yml
@@ -22,4 +22,13 @@ package_manager: 'apt-get'
 os_name: 'Debian'
 os_version: '9'
 
+bash: /bin/bash
+ccache: /usr/bin/ccache
+java: /usr/bin/java
+make: /usr/bin/make
+ninja: /usr/bin/ninja
+python: /usr/bin/python3
+su: /bin/su
+sudoers: /etc/sudoers
+
 ansible_python_interpreter: /usr/bin/python3
diff --git a/guests/host_vars/libvirt-debian-sid/main.yml b/guests/host_vars/libvirt-debian-sid/main.yml
index a60dd1b..5dd51ff 100644
--- a/guests/host_vars/libvirt-debian-sid/main.yml
+++ b/guests/host_vars/libvirt-debian-sid/main.yml
@@ -22,4 +22,13 @@ package_manager: 'apt-get'
 os_name: 'Debian'
 os_version: 'Sid'
 
+bash: /bin/bash
+ccache: /usr/bin/ccache
+java: /usr/bin/java
+make: /usr/bin/make
+ninja: /usr/bin/ninja
+python: /usr/bin/python3
+su: /bin/su
+sudoers: /etc/sudoers
+
 ansible_python_interpreter: /usr/bin/python3
diff --git a/guests/host_vars/libvirt-fedora-30/main.yml b/guests/host_vars/libvirt-fedora-30/main.yml
index e4ad7ea..a15aff2 100644
--- a/guests/host_vars/libvirt-fedora-30/main.yml
+++ b/guests/host_vars/libvirt-fedora-30/main.yml
@@ -35,4 +35,13 @@ package_manager: 'dnf'
 os_name: 'Fedora'
 os_version: '30'
 
+bash: /bin/bash
+ccache: /usr/bin/ccache
+java: /usr/bin/java
+make: /usr/bin/make
+ninja: /usr/bin/ninja
+python: /usr/bin/python3
+su: /bin/su
+sudoers: /etc/sudoers
+
 ansible_python_interpreter: /usr/bin/python3
diff --git a/guests/host_vars/libvirt-fedora-31/main.yml b/guests/host_vars/libvirt-fedora-31/main.yml
index e605873..4be1a19 100644
--- a/guests/host_vars/libvirt-fedora-31/main.yml
+++ b/guests/host_vars/libvirt-fedora-31/main.yml
@@ -23,4 +23,13 @@ package_manager: 'dnf'
 os_name: 'Fedora'
 os_version: '31'
 
+bash: /bin/bash
+ccache: /usr/bin/ccache
+java: /usr/bin/java
+make: /usr/bin/make
+ninja: /usr/bin/ninja
+python: /usr/bin/python3
+su: /bin/su
+sudoers: /etc/sudoers
+
 ansible_python_interpreter: /usr/bin/python3
diff --git a/guests/host_vars/libvirt-fedora-rawhide/main.yml b/guests/host_vars/libvirt-fedora-rawhide/main.yml
index 1a3338c..3a79a3a 100644
--- a/guests/host_vars/libvirt-fedora-rawhide/main.yml
+++ b/guests/host_vars/libvirt-fedora-rawhide/main.yml
@@ -23,4 +23,13 @@ package_manager: 'dnf'
 os_name: 'Fedora'
 os_version: 'Rawhide'
 
+bash: /bin/bash
+ccache: /usr/bin/ccache
+java: /usr/bin/java
+make: /usr/bin/make
+ninja: /usr/bin/ninja
+python: /usr/bin/python3
+su: /bin/su
+sudoers: /etc/sudoers
+
 ansible_python_interpreter: /usr/bin/python3
diff --git a/guests/host_vars/libvirt-freebsd-11/main.yml b/guests/host_vars/libvirt-freebsd-11/main.yml
index 2de64ef..6234e8a 100644
--- a/guests/host_vars/libvirt-freebsd-11/main.yml
+++ b/guests/host_vars/libvirt-freebsd-11/main.yml
@@ -21,4 +21,13 @@ package_manager: 'pkg'
 os_name: 'FreeBSD'
 os_version: '11'
 
+bash: /usr/local/bin/bash
+ccache: /usr/local/bin/ccache
+java: /usr/local/bin/java
+make: /usr/local/bin/gmake
+ninja: /usr/local/bin/ninja
+python: /usr/local/bin/python3
+su: /usr/bin/su
+sudoers: /usr/local/etc/sudoers
+
 ansible_python_interpreter: /usr/local/bin/python3
diff --git a/guests/host_vars/libvirt-freebsd-12/main.yml b/guests/host_vars/libvirt-freebsd-12/main.yml
index 2e3b935..22a3697 100644
--- a/guests/host_vars/libvirt-freebsd-12/main.yml
+++ b/guests/host_vars/libvirt-freebsd-12/main.yml
@@ -21,4 +21,13 @@ package_manager: 'pkg'
 os_name: 'FreeBSD'
 os_version: '12'
 
+bash: /usr/local/bin/bash
+ccache: /usr/local/bin/ccache
+java: /usr/local/bin/java
+make: /usr/local/bin/gmake
+ninja: /usr/local/bin/ninja
+python: /usr/local/bin/python3
+su: /usr/bin/su
+sudoers: /usr/local/etc/sudoers
+
 ansible_python_interpreter: /usr/local/bin/python3
diff --git a/guests/host_vars/libvirt-freebsd-current/main.yml b/guests/host_vars/libvirt-freebsd-current/main.yml
index 9b63d30..fdc7e9b 100644
--- a/guests/host_vars/libvirt-freebsd-current/main.yml
+++ b/guests/host_vars/libvirt-freebsd-current/main.yml
@@ -21,4 +21,13 @@ package_manager: 'pkg'
 os_name: 'FreeBSD'
 os_version: 'Current'
 
+bash: /usr/local/bin/bash
+ccache: /usr/local/bin/ccache
+java: /usr/local/bin/java
+make: /usr/local/bin/gmake
+ninja: /usr/local/bin/ninja
+python: /usr/local/bin/python3
+su: /usr/bin/su
+sudoers: /usr/local/etc/sudoers
+
 ansible_python_interpreter: /usr/local/bin/python3
diff --git a/guests/host_vars/libvirt-opensuse-151/main.yml b/guests/host_vars/libvirt-opensuse-151/main.yml
index 88d5dfd..bcc5ca3 100644
--- a/guests/host_vars/libvirt-opensuse-151/main.yml
+++ b/guests/host_vars/libvirt-opensuse-151/main.yml
@@ -21,4 +21,13 @@ package_manager: 'zypper'
 os_name: 'OpenSUSE'
 os_version: '151'
 
+bash: /bin/bash
+ccache: /usr/bin/ccache
+java: /usr/bin/java
+make: /usr/bin/make
+ninja: /usr/bin/ninja
+python: /usr/bin/python3
+su: /bin/su
+sudoers: /etc/sudoers
+
 ansible_python_interpreter: /usr/bin/python3
diff --git a/guests/host_vars/libvirt-ubuntu-1604/main.yml b/guests/host_vars/libvirt-ubuntu-1604/main.yml
index 2754282..b474264 100644
--- a/guests/host_vars/libvirt-ubuntu-1604/main.yml
+++ b/guests/host_vars/libvirt-ubuntu-1604/main.yml
@@ -21,4 +21,13 @@ package_manager: 'apt-get'
 os_name: 'Ubuntu'
 os_version: '1604'
 
+bash: /bin/bash
+ccache: /usr/bin/ccache
+java: /usr/bin/java
+make: /usr/bin/make
+ninja: /usr/bin/ninja
+python: /usr/bin/python3
+su: /bin/su
+sudoers: /etc/sudoers
+
 ansible_python_interpreter: /usr/bin/python3
diff --git a/guests/host_vars/libvirt-ubuntu-1804/main.yml b/guests/host_vars/libvirt-ubuntu-1804/main.yml
index 027acba..ddfeef2 100644
--- a/guests/host_vars/libvirt-ubuntu-1804/main.yml
+++ b/guests/host_vars/libvirt-ubuntu-1804/main.yml
@@ -22,4 +22,13 @@ package_manager: 'apt-get'
 os_name: 'Ubuntu'
 os_version: '1804'
 
+bash: /bin/bash
+ccache: /usr/bin/ccache
+java: /usr/bin/java
+make: /usr/bin/make
+ninja: /usr/bin/ninja
+python: /usr/bin/python3
+su: /bin/su
+sudoers: /etc/sudoers
+
 ansible_python_interpreter: /usr/bin/python3
diff --git a/guests/playbooks/update/main.yml b/guests/playbooks/update/main.yml
index e82055b..107a78a 100644
--- a/guests/playbooks/update/main.yml
+++ b/guests/playbooks/update/main.yml
@@ -49,7 +49,6 @@
 
     # Configure environment. Needs to happen after installing packages
     - include: '{{ playbook_base }}/tasks/kludges.yml'
-    - include: '{{ playbook_base }}/tasks/paths.yml'
     - include: '{{ playbook_base }}/tasks/bootloader.yml'
     - include: '{{ playbook_base }}/tasks/services.yml'
     - include: '{{ playbook_base }}/tasks/users.yml'
diff --git a/guests/playbooks/update/tasks/paths.yml b/guests/playbooks/update/tasks/paths.yml
deleted file mode 100644
index c204462..0000000
--- a/guests/playbooks/update/tasks/paths.yml
+++ /dev/null
@@ -1,71 +0,0 @@
----
-- set_fact:
-    commands: {}
-    files: {}
-
-- name: 'Look for commands'
-  shell: 'which {{ item }} 2>/dev/null || true'
-  register: which
-  with_items:
-    - bash
-    - ccache
-    - gmake
-    - java
-    - make
-    - ninja
-    - ninja-build
-    - python3
-    - su
-
-- name: 'Look for files'
-  shell: '(find /usr/local/etc -name {{ item }}; find /etc -name {{ item }}) 2>/dev/null'
-  register: find
-  with_items:
-    - sudoers
-
-- set_fact:
-    commands: '{{ commands|combine({ item.item: item.stdout }) }}'
-  no_log: True
-  with_items:
-    '{{ which.results }}'
-
-- set_fact:
-    files: '{{ files|combine({ item.item: item.stdout }) }}'
-  no_log: True
-  with_items:
-    '{{ find.results }}'
-
-- name: 'Export paths'
-  set_fact:
-    bash: '{{ commands["bash"] }}'
-    ccache: '{{ commands["ccache"] }}'
-    java: '{{ commands["java"] }}'
-    python: '{{ commands["python3"] }}'
-    su: '{{ commands["su"] }}'
-    sudoers: '{{ files["sudoers"] }}'
-
-# Prefer gmake, fall back to make
-- name: 'Export paths'
-  set_fact:
-    make: '{{ commands["gmake"] }}'
-  when:
-    - commands["gmake"] != ''
-
-- name: 'Export paths'
-  set_fact:
-    make: '{{ commands["make"] }}'
-  when:
-    - make is undefined
-
-# Prefer ninja, fall back to ninja-build
-- name: 'Export paths'
-  set_fact:
-    ninja: '{{ commands["ninja"] }}'
-  when:
-    - commands["ninja"] != ''
-
-- name: 'Export paths'
-  set_fact:
-    ninja: '{{ commands["ninja-build"] }}'
-  when:
-    - ninja is undefined
-- 
2.25.1




More information about the libvir-list mailing list