[libvirt] [jenkins-ci PATCH 1/7] guests: Store OS details as strings

Andrea Bolognani abologna at redhat.com
Fri Dec 21 08:36:21 UTC 2018


When parsing something like

  os_version: 9

YAML will, somewhat reasonably, decide that os_version is
a number rather than a string.

This would be okay if not for the fact that in-development
versions of operating systems are usually assigned a
codename rather than a version number (Debian sid, Fedora
Rawhide, FreeBSD -CURRENT) and so we will always need to
convert the version to string before comparison to account
for that.

Quote all details about the OS to make sure they're stored
as strings to begin with.

Signed-off-by: Andrea Bolognani <abologna at redhat.com>
---
 guests/host_vars/libvirt-centos-7/main.yml        | 6 +++---
 guests/host_vars/libvirt-debian-8/main.yml        | 6 +++---
 guests/host_vars/libvirt-debian-9/main.yml        | 6 +++---
 guests/host_vars/libvirt-debian-sid/main.yml      | 6 +++---
 guests/host_vars/libvirt-fedora-28/main.yml       | 6 +++---
 guests/host_vars/libvirt-fedora-29/main.yml       | 6 +++---
 guests/host_vars/libvirt-fedora-rawhide/main.yml  | 6 +++---
 guests/host_vars/libvirt-freebsd-11/main.yml      | 6 +++---
 guests/host_vars/libvirt-freebsd-12/main.yml      | 6 +++---
 guests/host_vars/libvirt-freebsd-current/main.yml | 6 +++---
 guests/host_vars/libvirt-ubuntu-16/main.yml       | 6 +++---
 guests/host_vars/libvirt-ubuntu-18/main.yml       | 6 +++---
 12 files changed, 36 insertions(+), 36 deletions(-)

diff --git a/guests/host_vars/libvirt-centos-7/main.yml b/guests/host_vars/libvirt-centos-7/main.yml
index 8be54b6..4e3692f 100644
--- a/guests/host_vars/libvirt-centos-7/main.yml
+++ b/guests/host_vars/libvirt-centos-7/main.yml
@@ -14,6 +14,6 @@ projects:
   - osinfo-db-tools
   - virt-viewer
 
-package_format: rpm
-os_name: CentOS
-os_version: 7
+package_format: 'rpm'
+os_name: 'CentOS'
+os_version: '7'
diff --git a/guests/host_vars/libvirt-debian-8/main.yml b/guests/host_vars/libvirt-debian-8/main.yml
index fb37205..0ac5130 100644
--- a/guests/host_vars/libvirt-debian-8/main.yml
+++ b/guests/host_vars/libvirt-debian-8/main.yml
@@ -13,6 +13,6 @@ projects:
   - osinfo-db-tools
   - virt-viewer
 
-package_format: deb
-os_name: Debian
-os_version: 8
+package_format: 'deb'
+os_name: 'Debian'
+os_version: '8'
diff --git a/guests/host_vars/libvirt-debian-9/main.yml b/guests/host_vars/libvirt-debian-9/main.yml
index 33476ad..4a1b197 100644
--- a/guests/host_vars/libvirt-debian-9/main.yml
+++ b/guests/host_vars/libvirt-debian-9/main.yml
@@ -16,6 +16,6 @@ projects:
   - virt-manager
   - virt-viewer
 
-package_format: deb
-os_name: Debian
-os_version: 9
+package_format: 'deb'
+os_name: 'Debian'
+os_version: '9'
diff --git a/guests/host_vars/libvirt-debian-sid/main.yml b/guests/host_vars/libvirt-debian-sid/main.yml
index f2594e7..911b491 100644
--- a/guests/host_vars/libvirt-debian-sid/main.yml
+++ b/guests/host_vars/libvirt-debian-sid/main.yml
@@ -16,6 +16,6 @@ projects:
   - virt-manager
   - virt-viewer
 
-package_format: deb
-os_name: Debian
-os_version: Sid
+package_format: 'deb'
+os_name: 'Debian'
+os_version: 'Sid'
diff --git a/guests/host_vars/libvirt-fedora-28/main.yml b/guests/host_vars/libvirt-fedora-28/main.yml
index 13555af..dc1b3f3 100644
--- a/guests/host_vars/libvirt-fedora-28/main.yml
+++ b/guests/host_vars/libvirt-fedora-28/main.yml
@@ -17,6 +17,6 @@ projects:
   - virt-manager
   - virt-viewer
 
-package_format: rpm
-os_name: Fedora
-os_version: 28
+package_format: 'rpm'
+os_name: 'Fedora'
+os_version: '28'
diff --git a/guests/host_vars/libvirt-fedora-29/main.yml b/guests/host_vars/libvirt-fedora-29/main.yml
index b6140ed..aba292d 100644
--- a/guests/host_vars/libvirt-fedora-29/main.yml
+++ b/guests/host_vars/libvirt-fedora-29/main.yml
@@ -17,6 +17,6 @@ projects:
   - virt-manager
   - virt-viewer
 
-package_format: rpm
-os_name: Fedora
-os_version: 29
+package_format: 'rpm'
+os_name: 'Fedora'
+os_version: '29'
diff --git a/guests/host_vars/libvirt-fedora-rawhide/main.yml b/guests/host_vars/libvirt-fedora-rawhide/main.yml
index 604f57f..d8d5dbc 100644
--- a/guests/host_vars/libvirt-fedora-rawhide/main.yml
+++ b/guests/host_vars/libvirt-fedora-rawhide/main.yml
@@ -27,6 +27,6 @@ projects:
   - virt-viewer+mingw32
   - virt-viewer+mingw64
 
-package_format: rpm
-os_name: Fedora
-os_version: Rawhide
+package_format: 'rpm'
+os_name: 'Fedora'
+os_version: 'Rawhide'
diff --git a/guests/host_vars/libvirt-freebsd-11/main.yml b/guests/host_vars/libvirt-freebsd-11/main.yml
index 887c9e6..ca32329 100644
--- a/guests/host_vars/libvirt-freebsd-11/main.yml
+++ b/guests/host_vars/libvirt-freebsd-11/main.yml
@@ -17,6 +17,6 @@ projects:
   - virt-manager
   - virt-viewer
 
-package_format: pkg
-os_name: FreeBSD
-os_version: 11
+package_format: 'pkg'
+os_name: 'FreeBSD'
+os_version: '11'
diff --git a/guests/host_vars/libvirt-freebsd-12/main.yml b/guests/host_vars/libvirt-freebsd-12/main.yml
index ac50271..e8d6a49 100644
--- a/guests/host_vars/libvirt-freebsd-12/main.yml
+++ b/guests/host_vars/libvirt-freebsd-12/main.yml
@@ -17,6 +17,6 @@ projects:
   - virt-manager
   - virt-viewer
 
-package_format: pkg
-os_name: FreeBSD
-os_version: 12
+package_format: 'pkg'
+os_name: 'FreeBSD'
+os_version: '12'
diff --git a/guests/host_vars/libvirt-freebsd-current/main.yml b/guests/host_vars/libvirt-freebsd-current/main.yml
index 1f3e911..3609ddb 100644
--- a/guests/host_vars/libvirt-freebsd-current/main.yml
+++ b/guests/host_vars/libvirt-freebsd-current/main.yml
@@ -17,6 +17,6 @@ projects:
   - virt-manager
   - virt-viewer
 
-package_format: pkg
-os_name: FreeBSD
-os_version: Current
+package_format: 'pkg'
+os_name: 'FreeBSD'
+os_version: 'Current'
diff --git a/guests/host_vars/libvirt-ubuntu-16/main.yml b/guests/host_vars/libvirt-ubuntu-16/main.yml
index 6ec1df8..11cd095 100644
--- a/guests/host_vars/libvirt-ubuntu-16/main.yml
+++ b/guests/host_vars/libvirt-ubuntu-16/main.yml
@@ -15,6 +15,6 @@ projects:
   - osinfo-db-tools
   - virt-viewer
 
-package_format: deb
-os_name: Ubuntu
-os_version: 16
+package_format: 'deb'
+os_name: 'Ubuntu'
+os_version: '16'
diff --git a/guests/host_vars/libvirt-ubuntu-18/main.yml b/guests/host_vars/libvirt-ubuntu-18/main.yml
index 61c7384..166d0e3 100644
--- a/guests/host_vars/libvirt-ubuntu-18/main.yml
+++ b/guests/host_vars/libvirt-ubuntu-18/main.yml
@@ -16,6 +16,6 @@ projects:
   - virt-manager
   - virt-viewer
 
-package_format: deb
-os_name: Ubuntu
-os_version: 18
+package_format: 'deb'
+os_name: 'Ubuntu'
+os_version: '18'
-- 
2.19.2




More information about the libvir-list mailing list