Merge pull request #75 from mattt416/remove_pool_size_config_option

Merged automatically by jenkins after successful build test

README.md

@@ -119,7 +119,12 @@ Default (LVM)
 
 By default, the 'lvm' provider is chosen.  No special configuration is
 necessary on the chef side, but the cinder volume node must have an
-lvm volume group named cinder volumes.
+lvm volume group named cinder-volumes. The attributes below can be
+adjusted for non-standard LVM configurations:
+
+node["cinder"]["storage"]["lvm"]["volume_group"] = "cinder-volumes"
+node["cinder"]["storage"]["lvm"]["volume_clear"] = "zero"
+node["cinder"]["storage"]["lvm"]["config"] = "/etc/lvm/lvm.conf"
 
 EMC
 ---

attributes/default.rb

@@ -44,7 +44,6 @@
 # LVM Settings
 default["cinder"]["storage"]["lvm"]["volume_group"] = "cinder-volumes"     # name from volume group
 default["cinder"]["storage"]["lvm"]["volume_clear"] = "zero"               # none, zero, shred
-default["cinder"]["storage"]["lvm"]["pool_size"] = "None"                  # Size of thin provisioning pool
 default["cinder"]["storage"]["lvm"]["config"] = "/etc/lvm/lvm.conf"        # Path of LVM config file
 
 # solidfire settings - set these if you are using solidfire

providers/conf.rb

@@ -61,7 +61,6 @@
   when "lvm"
     storage_options["volume_group"] = node["cinder"]["storage"]["lvm"]["volume_group"]
     storage_options["volume_clear"] = node["cinder"]["storage"]["lvm"]["volume_clear"]
-    storage_options["volume_pool_size"] = node["cinder"]["storage"]["lvm"]["pool_size"]
   else
     msg = "#{storage_provider}, is not currently supported by these cookbooks. Please change the storage provider attribute in your environment to one of lvm, emc, solidfire, netappiscsi, netappnfsdirect."
     Chef::Application.fatal! msg