Add yes argument when creating/extending a LV

Author: bmlit

README.md

@@ -201,6 +201,7 @@ resources out yourself.
 * stripesize (Parameter) - The stripesize to use for the new logical volume.
 * thinpool (Parameter) - Default value: `false` - Set to true to create a thin pool or to pool name to create thin volume
 * volume_group (Parameter) - The volume group name associated with this logical volume. This will automatically set this volume group as a dependency, but it must be defined elsewhere using the volume_group resource type.
+* yes (Parameter) - Default value: `false` - Do not prompt for confirmation interactively but always assume the answer yes.
 
 ### physical_volume
 

lib/puppet/provider/logical_volume/lvm.rb

@@ -118,6 +118,10 @@ def create
     else
       args << @resource[:volume_group]
     end
+
+    if @resource[:yes]
+      args.push('--yes')
+    end
     lvcreate(*args)
   end
 
@@ -183,7 +187,11 @@ def size=(new_size)
     end
 
     if resizeable
-      lvextend('-L', new_size, path) || raise("Cannot extend to size #{new_size} because lvextend failed.")
+      args = []
+      if @resource[:yes]
+        args.push('--yes')
+      end
+      lvextend('-L', new_size, path, *args) || raise("Cannot extend to size #{new_size} because lvextend failed.")
 
       unless @resource[:resize_fs] == :false || @resource[:resize_fs] == false || @resource[:resize_fs] == 'false'
         begin

lib/puppet/type/logical_volume.rb

@@ -180,6 +180,11 @@ def insync?(is)
     end
   end
 
+  newparam(:yes) do
+    desc "Do not prompt for confirmation interactively but always assume the answer yes."
+    defaultto false
+  end
+
   autorequire(:volume_group) do
     @parameters[:volume_group].value
   end

manifests/logical_volume.pp

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

tasks/ensure_lv.json

@@ -80,6 +80,10 @@
     "region_size": {
       "description": "A mirror is divided into regions of this size (in MB), the mirror log uses this granularity to track which regions are in sync. CAN NOT BE CHANGED on already mirrored volume. Take your mirror size in terabytes and round up that number to the next power of 2, using that number as the -R argument.",
       "type": "Optional[Integer]"
-    } 
+    },
+    "yes": {
+      "description": "Do not prompt for confirmation interactively but always assume the answer yes.",
+      "type": "Optional[Boolean]"
+    }
   }
-}
+}

tasks/ensure_lv.rb

@@ -50,6 +50,7 @@
 logical_volume[:alloc]            = params['alloc']            if params['alloc']
 logical_volume[:no_sync]          = params['no_sync']          if params['no_sync']
 logical_volume[:region_size]      = params['region_size']      if params['region_size']
+logical_volume[:yes]              = params['yes']              if params['yes']
 
 # Save the result
 _resource, report = Puppet::Resource.indirection.save(logical_volume)