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/common.yaml

@@ -0,0 +1 @@
+---

data/os/RedHat/AlmaLinux/10.yaml

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

data/os/RedHat/CentOS/10.yaml

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

data/os/RedHat/OracleLinux/10.yaml

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

data/os/RedHat/RedHat/10.yaml

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

data/os/RedHat/Rocky/10.yaml

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

hiera.yaml

@@ -0,0 +1,12 @@
+---
+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"
+    path: "os/%{facts.os.family}.yaml"
+  - name: "common"
+    path: "common.yaml"

manifests/database/postgresql.pp

@@ -98,10 +98,23 @@
   $port = scanf($database_port, '%i')[0]
 
   if $manage_server {
-    class { 'postgresql::globals':
-      manage_package_repo => $manage_package_repo,
-      version             => $postgres_version,
+    # Workaround for EL10 support
+    if $facts['os']['family'] == 'RedHat' and $facts['os']['release']['major'] == '10' {
+      class { 'postgresql::globals':
+        manage_package_repo => $manage_package_repo,
+        manage_dnf_module   => false,
+        version             => $postgres_version,
+        datadir             => '/var/lib/pgsql/data',
+        confdir             => '/var/lib/pgsql/data',
+        service_name        => 'postgresql',
+      }
+    } else {
+      class { 'postgresql::globals':
+        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/unit/classes/init_spec.rb

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