(MODULES-11666) validate LV present to prevent creating empty moountpaths

skyamgarp · skyamgarp · commit 424d9d9c4185 · 2026-03-27T16:14:04.000+05:30
(MODULES-11666) Fix failing tests
diff --git a/manifests/logical_volume.pp b/manifests/logical_volume.pp
@@ -166,9 +166,10 @@
   if $createfs or $ensure != 'present' {
     if $fs_type != 'swap' {
       exec { "ensure mountpoint '${fixed_mountpath}' exists":
-        path    => ['/bin', '/usr/bin'],
+        path    => ['/bin', '/usr/bin', '/sbin'],
         command => "mkdir -p ${fixed_mountpath}",
         unless  => "test -d ${fixed_mountpath}",
+        onlyif  => "test -b /dev/${volume_group}/${name} > /dev/null 2>&1",
         before  => Mount[$mount_title],
       }
     }

Original file line number	Diff line number	Diff line change
`@@ -166,9 +166,10 @@`
`166`	`166`	`if $createfs or $ensure != 'present' {`
`167`	`167`	`if $fs_type != 'swap' {`
`168`	`168`	`exec { "ensure mountpoint '${fixed_mountpath}' exists":`
`169`		`- path => ['/bin', '/usr/bin'],`
	`169`	`+ path => ['/bin', '/usr/bin', '/sbin'],`
`170`	`170`	`command => "mkdir -p ${fixed_mountpath}",`
`171`	`171`	`unless => "test -d ${fixed_mountpath}",`
	`172`	`+ onlyif => "test -b /dev/${volume_group}/${name} > /dev/null 2>&1",`
`172`	`173`	`before => Mount[$mount_title],`
`173`	`174`	`}`
`174`	`175`	`}`