Merge pull request Puppet-Finland#59 from flichtenheld/u24+arm64

Improved compatibility (Ubuntu 24, arm64)

Author: mattock

manifests/install/client/debian.pp

@@ -4,32 +4,20 @@
 # This code is needed as the --mkhomedir parameter passed to ipa-client-install does
 # not configure PAM even though it does install the required packages.
 #
-# Currently Ubuntu 14.04/16.04 and Debian 8/9 are supported.
-#
 class easy_ipa::install::client::debian {
-  case $facts['os']['distro']['codename'] {
-    /^(xenial|stretch|bionic|focal|buster|bullseye|jammy|bookworm)$/: {
-      # Ensure that required packages are present even if they do not get pulled
-      # in as freeipa-client package dependencies
-      stdlib::ensure_packages(['oddjob','oddjob-mkhomedir'], { 'ensure' => 'present' })
+  # Ensure that required packages are present even if they do not get pulled
+  # in as freeipa-client package dependencies
+  stdlib::ensure_packages(['oddjob','oddjob-mkhomedir'], { 'ensure' => 'present' })
 
-      # This should preferably be in a separate Puppet module
-      service { 'oddjobd':
-        ensure => 'running',
-        enable => true,
-        name   => 'oddjobd',
-      }
-      $mkhomedir_line = 'session optional /lib/x86_64-linux-gnu/security/pam_oddjob_mkhomedir.so'
-      $notify = Service['oddjobd']
-    }
-    /^(trusty|jessie)$/: {
-      $mkhomedir_line = 'session required pam_mkhomedir.so skel=/etc/skel/ umask=0022'
-      $notify = undef
-    }
-    default: {
-      fail('ERROR: unsupported Debian/Ubuntu version!')
-    }
+  # This should preferably be in a separate Puppet module
+  service { 'oddjobd':
+    ensure => 'running',
+    enable => true,
+    name   => 'oddjobd',
   }
+  $arch = $facts['os']['hardware']
+  $mkhomedir_line = "session optional /lib/${arch}-linux-gnu/security/pam_oddjob_mkhomedir.so"
+  $notify = Service['oddjobd']
 
   file_line { 'mkhomedir':
     ensure => 'present',

manifests/params.pp

@@ -15,11 +15,7 @@
       $named_conf_d = '/etc/named/conf.d'
     }
     'Debian': {
-      case $facts['os']['distro']['codename'] {
-        /(trusty|xenial|bionic|buster|focal|bullseye|jammy|bookworm)/: { $ipa_client_package_ensure = 'present' }
-        /(stretch)/:                                          { $ipa_client_package_ensure = 'absent' }
-        default:                                              { fail('ERROR: unsupported operating system') }
-      }
+      $ipa_client_package_ensure = 'present'
       $ldaputils_package_name = 'ldap-utils'
       $ipa_client_package_name = 'freeipa-client'
     }

metadata.json

@@ -48,15 +48,17 @@
       "operatingsystem": "Debian",
       "operatingsystemrelease": [
         "10",
-        "11"
+        "11",
+        "12"
       ]
     },
     {
       "operatingsystem": "Ubuntu",
       "operatingsystemrelease": [
         "18.04",
         "20.04",
-        "22.04"
+        "22.04",
+        "24.04"
       ]
     }
   ],