Rename the yes parameter to yes_arg

Author: bmlit

README.md

@@ -201,7 +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.
+* yes_flag (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

@@ -119,7 +119,7 @@ def create
       args << @resource[:volume_group]
     end
 
-    if @resource[:yes]
+    if @resource[:yes_flag]
       args.push('--yes')
     end
     lvcreate(*args)
@@ -188,7 +188,7 @@ def size=(new_size)
 
     if resizeable
       args = []
-      if @resource[:yes]
+      if @resource[:yes_flag]
         args.push('--yes')
       end
       lvextend('-L', new_size, path, *args) || raise("Cannot extend to size #{new_size} because lvextend failed.")

lib/puppet/type/logical_volume.rb

@@ -180,7 +180,7 @@ def insync?(is)
     end
   end
 
-  newparam(:yes) do
+  newparam(:yes_flag) do
     desc "Do not prompt for confirmation interactively but always assume the answer yes."
     defaultto false
   end

manifests/logical_volume.pp

@@ -140,7 +140,7 @@
     no_sync          => $no_sync,
     region_size      => $region_size,
     alloc            => $alloc,
-    yes              => $yes,
+    yes_flag         => $yes_flag,
   }
 
   if $createfs {

tasks/ensure_lv.json

@@ -81,7 +81,7 @@
       "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": {
+    "yes_flag": {
       "description": "Do not prompt for confirmation interactively but always assume the answer yes.",
       "type": "Optional[Boolean]"
     }

tasks/ensure_lv.rb

@@ -50,7 +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']
+logical_volume[:yes_flag]         = params['yes_flag']         if params['yes_flag']
 
 # Save the result
 _resource, report = Puppet::Resource.indirection.save(logical_volume)