Remove lvm-thin pool_size config option

The pool_size config option when using thin LVM
is never actually checked/honored.  It's also unlikely
that users will want to do something other than use the
entire VG for their thin pool.

Rather than have the unused option going forward let's
remove it from the config and just go with the default
behavior of utilizing the entire Cinder VG.

For folks that would like to only use a portion of the
VG for some reason, they can do an lvcreate pool on their
own (just like we have them do for the VG itself) and get
the same flexibility that way.

This is done simply by:
lvcreate -T -L ${size} ${vg-name}/${vg_name}-pool

DocImpact

Change-Id: I3ef2cde433b8dd775b1873a0d6df165dd1945d8c

diff --git a/cinder/volume/drivers/lvm.py b/cinder/volume/drivers/lvm.py
index bb3b71dd4b34758c45dc2796593d13c746926080..e1cb9fe0e62a9a9885505926a77918144d560228 100644 (file)
--- a/cinder/volume/drivers/lvm.py
+++ b/cinder/volume/drivers/lvm.py
@@ -44,10 +44,6 @@ volume_opts = [
     cfg.StrOpt('volume_group',
                default='cinder-volumes',
                help='Name for the VG that will contain exported volumes'),
-    cfg.StrOpt('pool_size',
-               default=None,
-               help='Size of thin provisioning pool '
-                    '(None uses entire cinder VG)'),
     cfg.IntOpt('lvm_mirrors',
                default=0,
                help='If set, create lvms with multiple mirrors. Note that '

diff --git a/etc/cinder/cinder.conf.sample b/etc/cinder/cinder.conf.sample
index 8ab46e8e02104c99836f5cc47278c1f8e5fdcce4..c501bf784475e46d26ac37a3e9458ce8f95ea38e 100644 (file)
--- a/etc/cinder/cinder.conf.sample
+++ b/etc/cinder/cinder.conf.sample
@@ -1241,10 +1241,6 @@
 # value)
 #volume_group=cinder-volumes
 
-# Size of thin provisioning pool (None uses entire cinder VG)
-# (string value)
-#pool_size=<None>
-
 # If set, create lvms with multiple mirrors. Note that this
 # requires lvm_mirrors + 2 pvs with available space (integer
 # value)