Allow for lvs to be created on specific pv

Author: Elbandi

REFERENCE.md

@@ -799,6 +799,7 @@ The following parameters are available in the `logical_volume` type.
 * [`name`](#-logical_volume--name)
 * [`no_sync`](#-logical_volume--no_sync)
 * [`persistent`](#-logical_volume--persistent)
+* [`physical_volume`](#-logical_volume--physical_volume)
 * [`poolmetadatasize`](#-logical_volume--poolmetadatasize)
 * [`provider`](#-logical_volume--provider)
 * [`range`](#-logical_volume--range)
@@ -844,6 +845,10 @@ An optimization in lvcreate, at least on Linux.
 
 Set to true to make the block device persistent
 
+##### <a name="-logical_volume--physical_volume"></a>`physical_volume`
+
+Create this logical volume on the specified physical volume
+
 ##### <a name="-logical_volume--poolmetadatasize"></a>`poolmetadatasize`
 
 Change the size of logical volume pool metadata

lib/puppet/provider/logical_volume/lvm.rb

@@ -120,6 +120,7 @@ def create
     end
 
     args.push('--yes') if @resource[:yes_flag]
+    args.push(@resource[:physical_volume]) if @resource[:physical_volume]
     lvcreate(*args)
   end
 

lib/puppet/type/logical_volume.rb

@@ -119,6 +119,24 @@ def insync?(is)
     end
   end
 
+  newparam(:physical_volume) do
+    desc "The name of the physical volume on which this logical volume will be created"
+    validate do |val|
+      unless val.is_a?(::String) || val.is_a?(::Array)
+        raise ArgumentError, "physical_volume should be String or Array: #{val}"
+      end
+    end
+
+    munge do |val|
+      case val
+      when ::String
+        [val]
+      else
+        val
+      end
+    end
+  end
+
   newproperty(:stripes) do
     desc 'The number of stripes to allocate for the new logical volume.'
     validate do |value|

manifests/logical_volume.pp

@@ -91,6 +91,7 @@
   Optional[Variant[String[1], Integer]] $region_size                            = undef,
   Optional[Enum['anywhere', 'contiguous', 'cling', 'inherit', 'normal']] $alloc = undef,
   Boolean $yes_flag                                                             = false,
+  Optional[Variant[Array[String],String]] $physical_volume                      = undef,
 ) {
   $lvm_device_path = "/dev/${volume_group}/${name}"
 
@@ -153,6 +154,7 @@
     region_size      => $region_size,
     alloc            => $alloc,
     yes_flag         => $yes_flag,
+    physical_volume  => $physical_volume,
   }
 
   if $createfs {