From 324e0a13a9cb499086b2910bb51337259a456241 Mon Sep 17 00:00:00 2001
From: Klas Arvidsson <klas.arvidsson@liu.se>
Date: Tue, 3 Oct 2023 21:13:28 +0200
Subject: [PATCH] Adaptions for RHEL9, corrections

---
 manifests/aes_sw.pp       | 4 ++--
 manifests/broker.pp       | 6 +++---
 manifests/init.pp         | 6 +++---
 manifests/squid_filter.pp | 4 ++--
 4 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/manifests/aes_sw.pp b/manifests/aes_sw.pp
index f2ab5ee..129d446 100644
--- a/manifests/aes_sw.pp
+++ b/manifests/aes_sw.pp
@@ -8,7 +8,7 @@ class aes::aes_sw {
   $examadm_group = $examadm_user
   $examadm_home = "/home/${examadm_user}"
 
-  case fact('os.family') {
+  case fact('os.name') {
     'RedHat': {
       firewalld_custom_service { 'aes-server':
         description => 'Authentic Examination System server',
@@ -52,7 +52,7 @@ class aes::aes_sw {
       }
     }
     default: {
-      fail("${module_name} - Not supported for family ${fact('os.family')}.")
+      fail("${module_name} - Not supported for family ${fact('os.name')}.")
     }
   }
 
diff --git a/manifests/broker.pp b/manifests/broker.pp
index f5b02c1..c52ab67 100644
--- a/manifests/broker.pp
+++ b/manifests/broker.pp
@@ -11,7 +11,7 @@ class aes::broker {
   $broker_home = "/srv/${broker_user}"
   $broker_service = 'aes_broker'
 
-  case fact('os.family') {
+  case fact('os.name') {
     'RedHat': {
       package {
         [
@@ -22,7 +22,7 @@ class aes::broker {
       }
 
       firewalld_custom_service { 'aes-broker':
-        description => 'Authentic Examination System server',
+        description => 'Authentic Examination System communication broker',
         ports       => [
           { port => '31337',  protocol => 'tcp' },
         ],
@@ -67,7 +67,7 @@ class aes::broker {
       }
     }
     default: {
-      fail("${module_name} - Not supported for family ${fact('os.family')}.")
+      fail("${module_name} - Not supported for family ${fact('os.name')}.")
     }
   }
 
diff --git a/manifests/init.pp b/manifests/init.pp
index 4a4027d..0799d53 100644
--- a/manifests/init.pp
+++ b/manifests/init.pp
@@ -15,8 +15,9 @@ class aes {
   include aes::auth
   include aes::auth_keydb
 
-  case fact('os.family') {
+  case fact('os.name') {
     'RedHat': {
+      # TODO: liurepo::centos_sclo_rh ???
       # TODO: Move this to the subclass that actually require it?
       package {
         [
@@ -27,7 +28,6 @@ class aes {
           'python3',
         ]:
           ensure  => installed,
-          require => Class['liurepo::centos_sclo_rh'],
       }
     }
     'CentOS': {
@@ -46,7 +46,7 @@ class aes {
       }
     }
     default: {
-      fail("${module_name} - Not supported for family ${fact('os.family')}.")
+      fail("${module_name} - Not supported for family ${fact('os.name')}.")
     }
   }
 
diff --git a/manifests/squid_filter.pp b/manifests/squid_filter.pp
index eb5dc55..46e71a0 100644
--- a/manifests/squid_filter.pp
+++ b/manifests/squid_filter.pp
@@ -5,7 +5,7 @@
 #
 #
 class aes::squid_filter {
-  case fact('os.family') {
+  case fact('os.name') {
     'RedHat': {
       firewalld_custom_service { 'squid':
         description => 'Squid proxy for filtered internet access',
@@ -43,7 +43,7 @@ class aes::squid_filter {
       }
     }
     default: {
-      fail("${module_name} - Not supported for family ${fact('os.family')}.")
+      fail("${module_name} - Not supported for family ${fact('os.name')}.")
     }
   }
 
-- 
GitLab