summaryrefslogtreecommitdiffstats
path: root/manifests
diff options
context:
space:
mode:
authorJames Shubin <james@shubin.ca>2013-09-18 06:13:26 -0400
committerJames Shubin <james@shubin.ca>2013-09-18 06:13:26 -0400
commit58aae6a64d5f69fbd681058f2bc00255e44c9476 (patch)
treed0a3cf2e027a2b0514355bb02fa73e0986aad94c /manifests
parent99b5c5c16751a2b459ea31d7fe98aa68be9f0eb7 (diff)
downloadpuppet-gluster-58aae6a64d5f69fbd681058f2bc00255e44c9476.tar.gz
puppet-gluster-58aae6a64d5f69fbd681058f2bc00255e44c9476.tar.xz
puppet-gluster-58aae6a64d5f69fbd681058f2bc00255e44c9476.zip
Added short comment.
Diffstat (limited to 'manifests')
-rw-r--r--manifests/volume.pp7
1 files changed, 7 insertions, 0 deletions
diff --git a/manifests/volume.pp b/manifests/volume.pp
index 8623897..8aab709 100644
--- a/manifests/volume.pp
+++ b/manifests/volume.pp
@@ -72,6 +72,13 @@ define gluster::volume(
# add /${name} to the end of each: brick:/path entry
$brick_spec = inline_template("<%= bricks.collect {|x| ''+x.chomp('/')+'/${name}' }.join(' ') %>")
+ # if volume creation fails for a stupid reason, in many cases, glusterd
+ # already did some of the work and left us with volume name directories
+ # on all bricks. the problem is that the future volume create commands,
+ # will error if they see that volume directory already present, so when
+ # we error we should rmdir any empty volume dirs to keep it pristine...
+ # TODO: this should be a gluster bug... we must hope it doesn't happen!
+
# get the list of bricks fqdn's that don't have our fqdn
$others = inline_template("<%= bricks.find_all{|x| x.split(':')[0] != '${fqdn}' }.collect {|y| y.split(':')[0] }.join(' ') %>")