Merge pull request #952 from Valantin/feat/zabbix-7

Add zabbix 7.0 support to CI

Author: Valantin

.github/workflows/ci.yml

@@ -20,4 +20,4 @@ jobs:
     name: Puppet
     uses: voxpupuli/gha-puppet/.github/workflows/beaker.yml@v2
     with:
-      beaker_facter: 'zabbix_version:Zabbix:5.0,6.0'
+      beaker_facter: 'zabbix_version:Zabbix:5.0,6.0,7.0'

manifests/repo.pp

@@ -31,7 +31,10 @@
     }
     case $facts['os']['family'] {
       'RedHat': {
-        if versioncmp(fact('os.release.major'), '9') >= 0 {
+        if (versioncmp(fact('os.release.major'), '7') >= 0 and $zabbix_version == '7.0') {
+          $gpgkey_zabbix = 'https://repo.zabbix.com/RPM-GPG-KEY-ZABBIX-B5333005'
+          $gpgkey_nonsupported = 'https://repo.zabbix.com/RPM-GPG-KEY-ZABBIX-B5333005'
+        } elsif versioncmp(fact('os.release.major'), '9') >= 0 {
           $gpgkey_zabbix = 'https://repo.zabbix.com/RPM-GPG-KEY-ZABBIX-08EFA7DD'
           $gpgkey_nonsupported = 'https://repo.zabbix.com/RPM-GPG-KEY-ZABBIX-08EFA7DD'
         } else {
@@ -139,6 +142,10 @@
           id     => 'A1848F5352D022B9471D83D0082AB56BA14FE591',
           source => 'https://repo.zabbix.com/zabbix-official-repo.key',
         }
+        apt::key { 'zabbix-4C3D6F2':
+          id     => '4C3D6F2CC75F5146754FC374D913219AB5333005',
+          source => 'https://repo.zabbix.com/zabbix-official-repo.key',
+        }
 
         # Debian 11 provides Zabbix 5.0 by default. This can cause problems for 4.0 versions
         $pinpriority = $facts['os']['release']['major'] ? {
@@ -153,6 +160,7 @@
           require  => [
             Apt_key['zabbix-FBABD5F'],
             Apt_key['zabbix-A1848F5'],
+            Apt_key['zabbix-4C3D6F2'],
           ],
         }
 

spec/acceptance/zabbix_host_spec.rb

@@ -6,6 +6,9 @@
 # rubocop:disable RSpec/LetBeforeExamples
 describe 'zabbix_host type' do
   supported_server_versions(default[:platform]).each do |zabbix_version|
+    # Zabbix 7.0 removed the deprecated params 'user' in favor to 'username'
+    next if zabbix_version >= '7.0'
+
     context "create zabbix_host resources with zabbix version #{zabbix_version}" do
       # This will deploy a running Zabbix setup (server, web, db) which we can
       # use for custom type tests

spec/acceptance/zabbix_hostgroup_spec.rb

@@ -5,6 +5,9 @@
 
 describe 'zabbix_hostgroup type' do
   supported_server_versions(default[:platform]).each do |zabbix_version|
+    # Zabbix 7.0 removed the deprecated params 'user' in favor to 'username'
+    next if zabbix_version >= '7.0'
+
     context "create zabbix_hostgroup resources with zabbix version #{zabbix_version}" do
       # This will deploy a running Zabbix setup (server, web, db) which we can
       # use for custom type tests

spec/acceptance/zabbix_proxy_spec.rb

@@ -6,6 +6,9 @@
 # rubocop:disable RSpec/LetBeforeExamples
 describe 'zabbix_proxy type' do
   supported_server_versions(default[:platform]).each do |zabbix_version|
+    # Zabbix 7.0 removed the deprecated params 'user' in favor to 'username'
+    next if zabbix_version >= '7.0'
+
     context "create zabbix_proxy resources with zabbix version #{zabbix_version}" do
       # This will deploy a running Zabbix setup (server, web, db) which we can
       # use for custom type tests

spec/acceptance/zabbix_template_host_spec.rb

@@ -6,7 +6,7 @@
 describe 'zabbix_template_host type' do
   supported_server_versions(default[:platform]).each do |zabbix_version|
     # Zabbix 6.0 removed the ability to attach templates directly to hosts.
-    next if zabbix_version == '6.0'
+    next if zabbix_version >= '6.0'
 
     context "create zabbix_template_host resources with zabbix version #{zabbix_version}" do
       template = case zabbix_version

spec/acceptance/zabbix_template_spec.rb

@@ -5,6 +5,9 @@
 
 describe 'zabbix_template type' do
   supported_server_versions(default[:platform]).each do |zabbix_version|
+    # Zabbix 7.0 removed the deprecated params 'user' in favor to 'username'
+    next if zabbix_version >= '7.0'
+
     context "create zabbix_template resources with zabbix version #{zabbix_version}" do
       # This will deploy a running Zabbix setup (server, web, db) which we can
       # use for custom type tests

spec/classes/database_mysql_spec.rb

@@ -29,8 +29,7 @@
                  '/usr/share/doc/zabbix-*-mysql*'
                end
 
-        sql_server = case zabbix_version
-                     when '6.0'
+        sql_server = if Puppet::Util::Package.versioncmp(zabbix_version, '6.0') >= 0
                        'server.sql'
                      else
                        'create.sql'

spec/classes/database_postgresql_spec.rb

@@ -42,8 +42,7 @@
                  '/usr/share/doc/zabbix-*-pgsql'
                end
 
-        sql_server = case zabbix_version
-                     when '6.0'
+        sql_server = if Puppet::Util::Package.versioncmp(zabbix_version, '6.0') >= 0
                        'server.sql'
                      else
                        'create.sql'

spec/setup_acceptance_node.pp

@@ -8,6 +8,9 @@
         target => '/bin/mkdir',
       }
     }
+    package { 'gnupg':
+      ensure => present,
+    }
   }
   'Ubuntu': {
     # The Ubuntu 18.04+ docker image has a dpkg config that won't install docs, to keep used space low