Sohonet fixes

lib/puppet/type/volume_group.rb

@@ -10,6 +10,13 @@
     desc "The list of physical volumes to be included in the volume group; this
          will automatically set these as dependencies, but they must be defined elsewhere
          using the physical_volume resource type."
+        munge do |value|
+          if value.is_a?(Hash) then
+            value.keys.sort
+          else
+            value
+          end
+        end
 
     def insync?(is)
       if @resource.parameter(:followsymlinks).value == :true

manifests/logical_volume.pp

@@ -14,6 +14,7 @@
   Boolean $mountpath_require         = false,
   Boolean $mounted                   = true,
   Boolean $createfs                  = true,
+  $mountpath_fs_permissions          = undef,
   $extents                           = undef,
   $stripes                           = undef,
   $stripesize                        = undef,
@@ -120,5 +121,15 @@
       dump    => $fixed_dump,
       atboot  => true,
     }
+
+    if $mountpath_fs_permissions {
+      exec { "ensure permissions for '${fixed_mountpath}' are correct":
+        path        => [ '/bin', '/usr/bin' ],
+        require     => Mount[$mount_title],
+        subscribe   => Mount[$mount_title],
+        refreshonly => true,
+        command     => "chown ${mountpath_fs_permissions}:${mountpath_fs_permissions} ${fixed_mountpath}",
+      }
+    }
   }
 }