feat: EL10 support

Add an "unholy hack" (quote) to support EL10 while it's added to postgresql

Co-authored-by: Corporate Gadfly <corporate-gadfly@users.noreply.github.com>

Author: d1nuc0m

data/os/RedHat/10.yaml

@@ -0,0 +1,3 @@
+---
+# As of v10.6.1, puppetlabs/postgresql does not have EL10 support/data
+openvoxdb::postgres_version: '16'

hiera.yaml

@@ -0,0 +1,14 @@
+---
+version: 5
+
+hierarchy:
+  - name: "OS version"
+    path: "os/%{facts.os.family}/%{facts.os.name}/%{facts.os.release.major}.yaml"
+  - name: "OS variant"
+    path: "os/%{facts.os.family}/%{facts.os.name}.yaml"
+  - name: "OS family/major"
+    path: "os/%{facts.os.family}/%{facts.os.release.major}.yaml"
+  - name: "OS family"
+    path: "os/%{facts.os.family}.yaml"
+  - name: "common"
+    path: "common.yaml"

manifests/database/postgresql.pp

@@ -99,9 +99,11 @@
 
   if $manage_server {
     class { 'postgresql::globals':
+      manage_dnf_module   => false, # disable DNF module on EL8, EL9 and EL10
       manage_package_repo => $manage_package_repo,
       version             => $postgres_version,
     }
+
     # get the pg server up and running
     class { 'postgresql::server':
       ip_mask_allow_all_users => '0.0.0.0/0',

metadata.json

@@ -30,13 +30,15 @@
       "operatingsystem": "AlmaLinux",
       "operatingsystemrelease": [
         "8",
-        "9"
+        "9",
+        "10"
       ]
     },
     {
       "operatingsystem": "CentOS",
       "operatingsystemrelease": [
-        "9"
+        "9",
+        "10"
       ]
     },
     {
@@ -51,21 +53,24 @@
       "operatingsystem": "OracleLinux",
       "operatingsystemrelease": [
         "8",
-        "9"
+        "9",
+        "10"
       ]
     },
     {
       "operatingsystem": "RedHat",
       "operatingsystemrelease": [
         "8",
-        "9"
+        "9",
+        "10"
       ]
     },
     {
       "operatingsystem": "Rocky",
       "operatingsystemrelease": [
         "8",
-        "9"
+        "9",
+        "10"
       ]
     },
     {

spec/support/acceptance/shared/puppetserver.pp

@@ -12,17 +12,25 @@
   # TODO: backport to litmusimage, required for serverspec port tests
   package { 'iproute': ensure => installed }
 
-  # TODO: rework this hack, maybe not needed for newer version of postgresl module?
+  # EL 8 and 9 ship a built-in DNF module for postgresql that must be disabled
+  # before installing from the upstream PGDG repo. EL 10 dropped the module stream.
   if versioncmp($facts['os']['release']['major'], '8') >= 0 {
-    package { 'disable-builtin-dnf-postgresql-module':
-      ensure   => 'disabled',
-      name     => 'postgresql',
-      provider => 'dnfmodule',
-    }
+    if versioncmp($facts['os']['release']['major'], '10') < 0 {
+      # EL 8 and 9
+      package { 'disable-builtin-dnf-postgresql-module':
+        ensure   => 'disabled',
+        name     => 'postgresql',
+        provider => 'dnfmodule',
+      }
 
-    Yumrepo <| tag == 'postgresql::repo' |>
-    -> Package['disable-dnf-postgresql-module']
-    -> Package <| tag == 'postgresql' |>
+      Yumrepo <| tag == 'postgresql::repo' |>
+      -> Package['disable-builtin-dnf-postgresql-module']
+      -> Package <| tag == 'postgresql' |>
+    } else {
+      # EL 10
+      Yumrepo <| tag == 'postgresql::repo' |>
+      -> Package <| tag == 'postgresql' |>
+    }
   }
 }
 

spec/unit/classes/init_spec.rb

@@ -19,10 +19,19 @@
 
       describe 'without managed postgresql' do
         let :pre_condition do
-          <<-HEREDOC
-          class { 'postgresql::server':
-          }
-          HEREDOC
+          if facts.dig(:os, 'family') == 'RedHat' && facts.dig(:os, 'release', 'major') == '10'
+            <<-HEREDOC
+            class { 'postgresql::globals':
+              version => '16',
+            }
+            -> class { 'postgresql::server': }
+            HEREDOC
+          else
+            <<-HEREDOC
+            class { 'postgresql::server':
+            }
+            HEREDOC
+          end
         end
 
         let :params do