Fix physical_volumes provisioning with 'followsymlinks' enabled

Author: Siarhei Rasiukevich

lib/puppet/provider/volume_group/lvm.rb

@@ -58,14 +58,30 @@ def exists?
     end
 
     def physical_volumes=(new_volumes = [])
+        # Need to replace device path with real names, instead of symlink
+        if @resource.parameter(:followsymlinks).value == :true then
+            new_volume_real = []
+            new_volumes.each do |s|
+                if File.symlink?(s)
+                    device = File.expand_path(File.readlink(s), File.dirname(s))
+                    new_volume_real.push device
+                else
+                    real_should.push s
+                end
+            end
+        else
+            new_volume_real = new_volumes
+        end 
+
+
         # Only take action if createonly is false just to be safe
         #  this is really only here to enforce the createonly setting
         #  if something goes wrong in physical_volumes
         if @resource[:createonly].to_s == "false"
           existing_volumes = physical_volumes
-          extraneous = existing_volumes - new_volumes
+          extraneous = existing_volumes - new_volume_real 
           extraneous.each { |volume| reduce_with(volume) }
-          missing = new_volumes - existing_volumes
+          missing = new_volume_real - existing_volumes
           missing.each { |volume| extend_with(volume) }
         end
     end