diff options
-rw-r--r-- | manifests/brick.pp | 13 | ||||
-rw-r--r-- | manifests/simple.pp | 19 |
2 files changed, 22 insertions, 10 deletions
diff --git a/manifests/brick.pp b/manifests/brick.pp index 4ebdcb1..ee308f2 100644 --- a/manifests/brick.pp +++ b/manifests/brick.pp @@ -524,11 +524,22 @@ define gluster::brick( } elsif ((type($dev) == 'boolean') and (! $dev)) and ("${fqdn}" == "${host}") { # ensure the full path exists! + # TODO: is the mkdir needed ? exec { "/bin/mkdir -p '${valid_path}'": creates => "${valid_path}", logoutput => on_failure, noop => $exec_noop, - alias => "gluster-brick-mkdir ${name}", + alias => "gluster-brick-mkdir-${name}", + before => File["${valid_path}"], + } + + # avoid any possible purging of data! + file { "${valid_path}": + ensure => directory, # make sure this is a directory + recurse => false, # don't recurse into directory + purge => false, # don't purge unmanaged files + force => false, # don't purge subdirs and links + require => Exec["gluster-brick-mkfs-${name}"], } } } diff --git a/manifests/simple.pp b/manifests/simple.pp index 1b72df3..45d3bd1 100644 --- a/manifests/simple.pp +++ b/manifests/simple.pp @@ -43,15 +43,16 @@ class gluster::simple( #$vardir = $::gluster::vardir::module_vardir # with trailing slash $vardir = regsubst($::gluster::vardir::module_vardir, '\/$', '') - if "${path}" == '' { - file { "${vardir}/data/": - ensure => directory, # make sure this is a directory - recurse => false, # don't recurse into directory - purge => false, # don't purge unmanaged files - force => false, # don't purge subdirs and links - require => File["${vardir}/"], - } - } + # NOTE: this is done in brick.pp now + #if "${path}" == '' { + # file { "${vardir}/data/": + # ensure => directory, # make sure this is a directory + # recurse => false, # don't recurse into directory + # purge => false, # don't purge unmanaged files + # force => false, # don't purge subdirs and links + # require => File["${vardir}/"], + # } + #} $chosen_path = "${path}" ? { '' => "${vardir}/data/", |