From a9cd134c9e43ae2b828206b1d8e069d579862075 Mon Sep 17 00:00:00 2001
From: sylvainfaivre <sylvainfaivre@gmail.com>
Date: Wed, 13 Dec 2023 15:11:12 +0100
Subject: [PATCH] fix: correct package names for debian 11, drop support for
 debian 9

---
 .gitlab-ci.yml                                     |  4 ++--
 .pre-commit-config.yaml                            |  2 +-
 libvirt/parameters/defaults.yaml                   |  2 +-
 libvirt/parameters/os/AlmaLinux.yaml               |  1 -
 libvirt/parameters/os/CentOS.yaml                  |  1 -
 libvirt/parameters/os/Fedora.yaml                  |  1 -
 libvirt/parameters/os/Rocky.yaml                   |  1 -
 libvirt/parameters/os_family/Debian.yaml           |  4 +---
 .../osfinger/{Ubuntu-16.04.yaml => Debian-9.yaml}  |  7 ++++---
 .../{Ubuntu-22.04.yaml => Ubuntu-18.04.yaml}       |  4 ++--
 libvirt/parameters/osfinger/Ubuntu-20.04.yaml      | 14 ++++++++++++++
 .../share/libraries/libvirt_packages.rb            |  5 ++---
 12 files changed, 27 insertions(+), 19 deletions(-)
 rename libvirt/parameters/osfinger/{Ubuntu-16.04.yaml => Debian-9.yaml} (83%)
 rename libvirt/parameters/osfinger/{Ubuntu-22.04.yaml => Ubuntu-18.04.yaml} (89%)
 create mode 100644 libvirt/parameters/osfinger/Ubuntu-20.04.yaml

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 70381df..a7df0e6 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -144,10 +144,10 @@ rubocop:
 # default-oraclelinux-7-tiamat-py3: {extends: '.test_instance'}
 # default-almalinux-8-tiamat-py3: {extends: '.test_instance'}
 # default-rockylinux-8-tiamat-py3: {extends: '.test_instance'}
-# default-debian-11-master-py3: {extends: '.test_instance'}
+default-debian-11-master-py3: {extends: '.test_instance'}
 default-debian-10-master-py3: {extends: '.test_instance'}
 # clean-debian-10-master-py3: {extends: '.test_instance'}
-default-debian-9-master-py3: {extends: '.test_instance'}
+# default-debian-9-master-py3: {extends: '.test_instance'}
 default-ubuntu-2204-master-py3: {extends: '.test_instance_failure_permitted'}
 default-ubuntu-2004-master-py3: {extends: '.test_instance'}
 default-ubuntu-1804-master-py3: {extends: '.test_instance'}
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index 1299a84..0abb695 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -53,7 +53,7 @@ repos:
         always_run: true
         pass_filenames: false
   - repo: https://github.com/warpnet/salt-lint
-    rev: v0.8.0
+    rev: v0.9.2
     hooks:
       - id: salt-lint
         name: Check Salt files using salt-lint
diff --git a/libvirt/parameters/defaults.yaml b/libvirt/parameters/defaults.yaml
index 3edf8c7..265d448 100644
--- a/libvirt/parameters/defaults.yaml
+++ b/libvirt/parameters/defaults.yaml
@@ -15,7 +15,7 @@ values:
   libvirt_pkg: libvirt
   qemu_pkg: qemu
   python2_pkg: libvirt-python
-  python3_pkg: libvirt-python3
+  python3_pkg: python3-libvirt
   libvirt_service: libvirtd
   libvirtd_config: /etc/libvirt/libvirtd.conf
   daemon_config_path: {}
diff --git a/libvirt/parameters/os/AlmaLinux.yaml b/libvirt/parameters/os/AlmaLinux.yaml
index cf4e916..932cb1c 100644
--- a/libvirt/parameters/os/AlmaLinux.yaml
+++ b/libvirt/parameters/os/AlmaLinux.yaml
@@ -11,5 +11,4 @@
 ---
 values:
   python2_pkg: ~
-  python3_pkg: python3-libvirt
 ...
diff --git a/libvirt/parameters/os/CentOS.yaml b/libvirt/parameters/os/CentOS.yaml
index cf4e916..932cb1c 100644
--- a/libvirt/parameters/os/CentOS.yaml
+++ b/libvirt/parameters/os/CentOS.yaml
@@ -11,5 +11,4 @@
 ---
 values:
   python2_pkg: ~
-  python3_pkg: python3-libvirt
 ...
diff --git a/libvirt/parameters/os/Fedora.yaml b/libvirt/parameters/os/Fedora.yaml
index 99df8a1..77b0c73 100644
--- a/libvirt/parameters/os/Fedora.yaml
+++ b/libvirt/parameters/os/Fedora.yaml
@@ -11,5 +11,4 @@
 ---
 values:
   python2_pkg: python2-libvirt
-  python3_pkg: python3-libvirt
 ...
diff --git a/libvirt/parameters/os/Rocky.yaml b/libvirt/parameters/os/Rocky.yaml
index cf4e916..932cb1c 100644
--- a/libvirt/parameters/os/Rocky.yaml
+++ b/libvirt/parameters/os/Rocky.yaml
@@ -11,5 +11,4 @@
 ---
 values:
   python2_pkg: ~
-  python3_pkg: python3-libvirt
 ...
diff --git a/libvirt/parameters/os_family/Debian.yaml b/libvirt/parameters/os_family/Debian.yaml
index b594d72..bcfa828 100644
--- a/libvirt/parameters/os_family/Debian.yaml
+++ b/libvirt/parameters/os_family/Debian.yaml
@@ -12,13 +12,11 @@
 values:
   libvirt_pkg: libvirt-daemon-system
   libvirt_service: libvirtd
-  qemu_pkg: qemu-kvm
+  qemu_pkg: qemu-system-x86
   python2_pkg: python-libvirt
-  python3_pkg: python3-libvirt
   extra_pkgs:
     - libguestfs0
     - libguestfs-tools
     - gnutls-bin
-    - virt-top
   daemon_config_path: /etc/default
 ...
diff --git a/libvirt/parameters/osfinger/Ubuntu-16.04.yaml b/libvirt/parameters/osfinger/Debian-9.yaml
similarity index 83%
rename from libvirt/parameters/osfinger/Ubuntu-16.04.yaml
rename to libvirt/parameters/osfinger/Debian-9.yaml
index 0ea5750..977da64 100644
--- a/libvirt/parameters/osfinger/Ubuntu-16.04.yaml
+++ b/libvirt/parameters/osfinger/Debian-9.yaml
@@ -1,7 +1,7 @@
 # -*- coding: utf-8 -*-
 # vim: ft=yaml
 #
-# Setup variables specific to salt['config.get']('osfinger') == Ubuntu-16.04.
+# Setup variables specific to salt['config.get']('osfinger') == Debian-9.
 # You just need to add the key:values for this `osfinger` that differ
 # from `defaults.yaml`.
 #
@@ -10,6 +10,7 @@
 # values: {}
 ---
 values:
-  libvirt_pkg: libvirt-bin
-  libvirt_service: libvirt-bin
+  extra_pkgs:
+    - gnutls-bin
+  qemu_pkg: qemu-kvm
 ...
diff --git a/libvirt/parameters/osfinger/Ubuntu-22.04.yaml b/libvirt/parameters/osfinger/Ubuntu-18.04.yaml
similarity index 89%
rename from libvirt/parameters/osfinger/Ubuntu-22.04.yaml
rename to libvirt/parameters/osfinger/Ubuntu-18.04.yaml
index fd70503..83c9146 100644
--- a/libvirt/parameters/osfinger/Ubuntu-22.04.yaml
+++ b/libvirt/parameters/osfinger/Ubuntu-18.04.yaml
@@ -1,7 +1,7 @@
 # -*- coding: utf-8 -*-
 # vim: ft=yaml
 #
-# Setup variables specific to salt['config.get']('osfinger') == Ubuntu-22.04.
+# Setup variables specific to salt['config.get']('osfinger') == Ubuntu-18.04.
 # You just need to add the key:values for this `osfinger` that differ
 # from `defaults.yaml`.
 #
@@ -10,5 +10,5 @@
 # values: {}
 ---
 values:
-  qemu_pkg: qemu-system-x86
+  qemu_pkg: qemu-kvm
 ...
diff --git a/libvirt/parameters/osfinger/Ubuntu-20.04.yaml b/libvirt/parameters/osfinger/Ubuntu-20.04.yaml
new file mode 100644
index 0000000..de494c1
--- /dev/null
+++ b/libvirt/parameters/osfinger/Ubuntu-20.04.yaml
@@ -0,0 +1,14 @@
+# -*- coding: utf-8 -*-
+# vim: ft=yaml
+#
+# Setup variables specific to salt['config.get']('osfinger') == Ubuntu-20.04.
+# You just need to add the key:values for this `osfinger` that differ
+# from `defaults.yaml`.
+#
+# If you do not need to provide defaults via the `osfinger` config,
+# you can remove this file or provide at least an empty dict, e.g.
+# values: {}
+---
+values:
+  qemu_pkg: qemu-kvm
+...
diff --git a/test/integration/share/libraries/libvirt_packages.rb b/test/integration/share/libraries/libvirt_packages.rb
index c43ab42..879f107 100644
--- a/test/integration/share/libraries/libvirt_packages.rb
+++ b/test/integration/share/libraries/libvirt_packages.rb
@@ -78,7 +78,8 @@ def build_default_packages
   def build_debian_packages
     {
       'libvirt' => ['libvirt-daemon-system'],
-      'extra' => %w[libguestfs0 libguestfs-tools gnutls-bin virt-top],
+      'qemu' => ['qemu-system-x86'],
+      'extra' => %w[libguestfs0 libguestfs-tools gnutls-bin],
       'python' => if inspec.salt_minion.python3?
                     ['python3-libvirt']
                   else
@@ -134,8 +135,6 @@ def build_centos_packages
 
   def build_ubuntu_packages
     case inspec.system.platform[:release]
-    when /^22/
-      { 'qemu' => ['qemu-system-x86'] }
     when /^16/
       { 'libvirt' => ['libvirt-bin'] }
     else