From e71fb3b68d7d7cebc7d0f9f03a38c3ae7f2a2a80 Mon Sep 17 00:00:00 2001 From: James Shubin Date: Sat, 26 Oct 2013 01:19:29 -0400 Subject: Small fixes for gluster::simple. --- manifests/simple.pp | 5 +++-- manifests/volume.pp | 4 +++- 2 files changed, 6 insertions(+), 3 deletions(-) (limited to 'manifests') diff --git a/manifests/simple.pp b/manifests/simple.pp index 4f34cb9..edd80ac 100644 --- a/manifests/simple.pp +++ b/manifests/simple.pp @@ -20,7 +20,8 @@ class gluster::simple( $volume = 'puppet', # NOTE: this can be a list... $replica = 1, $stripe = 1, # TODO: not fully implemented in puppet-gluster - $vip = '' # strongly recommended + $vip = '', # strongly recommended + $shorewall = true ) { include gluster::vardir @@ -58,7 +59,7 @@ class gluster::simple( class { '::gluster::server': vip => "${vip}", #zone => 'net', # defaults to net - shorewall => true, + shorewall => $shorewall, } @@gluster::host { "${::fqdn}": diff --git a/manifests/volume.pp b/manifests/volume.pp index 5e8b7e1..d16db55 100644 --- a/manifests/volume.pp +++ b/manifests/volume.pp @@ -71,7 +71,8 @@ define gluster::volume( } } - $collected_bricks = split(getvar("gluster_brick_group_${group}"), ',') + $gluster_brick_group_fact = getvar("gluster_brick_group_${group}") + $collected_bricks = split($gluster_brick_group_fact, ',') $valid_bricks = type($bricks) ? { 'boolean' => $bricks ? { true => $collected_bricks, # an array... @@ -318,6 +319,7 @@ define gluster::volume( source => "${zone}", # use our source zone before => Service['glusterd'], } + } # fsm variables and boilerplate $statefile = "${vardir}/volume/fsm/${name}/state" -- cgit