From 05576cd410bb2616ce72e3e9e24850f911cbb6dc Mon Sep 17 00:00:00 2001 From: James Shubin Date: Wed, 12 Mar 2014 11:34:52 -0400 Subject: Setting volume properties should be race-free. This isn't essential, as ensuring this is race-free is really up to glusterfs, but with this patch you reduce the likelihood to ~0% that you'll see a: "volume set: failed: Another transaction is in progress." error. The error isn't harmful, but now we'll see less unnecessary red. --- manifests/simple.pp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'manifests/simple.pp') diff --git a/manifests/simple.pp b/manifests/simple.pp index 0fa75da..6bc6c11 100644 --- a/manifests/simple.pp +++ b/manifests/simple.pp @@ -176,7 +176,8 @@ class gluster::simple( if "${setgroup}" != '' { $setgroup_yaml = inline_template("<%= @valid_volumes.inject(Hash.new) { |h,i| {i+'#'+@setgroup => {}}.merge(h) }.to_yaml %>") $setgroup_hash = parseyaml($setgroup_yaml) - create_resources('gluster::volume::property::group', $setgroup_hash) + $setgroup_defaults = {'vip' => "${vip}"} + create_resources('gluster::volume::property::group', $setgroup_hash, $setgroup_defaults) } } -- cgit