Merge pull request edestecd#70 from flinkflonk/master

PR for issue edestecd#69 - compatibility with RHEL/Centos 8.x

Author: edestecd

manifests/freshclam.pp

@@ -44,7 +44,7 @@
     $service_subscribe = File['freshclam.conf']
   }
 
-  # NOTE: RedHat comes with /etc/cron.daily/freshclam instead of a service
+  # NOTE: RedHat <8 comes with /etc/cron.daily/freshclam instead of a service
   if $clamav::freshclam_service {
     service { 'freshclam':
       ensure     => $clamav::freshclam_service_ensure,

manifests/params.pp

@@ -23,7 +23,6 @@
     $manage_repo    = true
     $clamav_package = 'clamav'
     $clamav_version = 'installed'
-    $freshclam_service = undef
 
     if versioncmp($::operatingsystemmajrelease, '7') >= 0 {
       # ### user vars ####
@@ -58,6 +57,13 @@
       $freshclam_sysconfig = '/etc/sysconfig/freshclam'
       $freshclam_delay     = undef
 
+      # ### RHEL8/Centos8 actually do have a service
+      if versioncmp($::operatingsystemmajrelease, '8') >= 0 {
+        $freshclam_service = 'clamav-freshclam'
+      } else {
+        $freshclam_service = undef
+      }
+
       # ### clamav_milter vars ####
       $clamav_milter_package     = 'clamav-milter-systemd'
       $clamav_milter_version     = 'installed'