From 58aae6a64d5f69fbd681058f2bc00255e44c9476 Mon Sep 17 00:00:00 2001 From: James Shubin Date: Wed, 18 Sep 2013 06:13:26 -0400 Subject: Added short comment. --- manifests/volume.pp | 7 +++++++ 1 file changed, 7 insertions(+) (limited to 'manifests') 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(' ') %>") -- cgit