fix: modern way to get os family

costowell · costowell · commit d1eb40dc9a8f · 2025-10-14T21:22:22.000-04:00
diff --git a/manifests/params.pp b/manifests/params.pp
@@ -36,7 +36,7 @@
   # RHEL 6.6 includes automatic sudo configuration
   # RHEL 7.0 requires manual confifuration
   # RHEL >=7.1 includes automatic sudo configuration
-  case $::osfamily {
+  case $facts['os']['family'] {
     'RedHat': {
       case $::operatingsystem {
         'Fedora': {
@@ -74,7 +74,7 @@
       }
     }
     default: {
-      fail("This module does not support operatingsystem ${::operatingsystem}")
+      fail("This module does not support os family " + facts['os']['family'])
     }
   }
 

Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@`
`36`	`36`	`# RHEL 6.6 includes automatic sudo configuration`
`37`	`37`	`# RHEL 7.0 requires manual confifuration`
`38`	`38`	`# RHEL >=7.1 includes automatic sudo configuration`
`39`		`- case $::osfamily {`
	`39`	`+ case $facts['os']['family'] {`
`40`	`40`	`'RedHat': {`
`41`	`41`	`case $::operatingsystem {`
`42`	`42`	`'Fedora': {`
`@@ -74,7 +74,7 @@`
`74`	`74`	`}`
`75`	`75`	`}`
`76`	`76`	`default: {`
`77`		`- fail("This module does not support operatingsystem ${::operatingsystem}")`
	`77`	`+ fail("This module does not support os family " + facts['os']['family'])`
`78`	`78`	`}`
`79`	`79`	`}`
`80`	`80`