]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Correct lvm_mirrors help message
authorEric Harney <eharney@redhat.com>
Mon, 16 Jun 2014 19:44:09 +0000 (15:44 -0400)
committerEric Harney <eharney@redhat.com>
Thu, 3 Jul 2014 19:01:48 +0000 (15:01 -0400)
The description is not correct if someone sets
lvm_mirrors=0.  Also clarify LVs vs. lvms.

Change-Id: I7ec74fc8beaef5e638bc1a5dac3d7d97f1c474d3

cinder/volume/drivers/lvm.py
etc/cinder/cinder.conf.sample

index dc98a78703ca2579501d774f42dca257a45d49b7..bdfe12d2413c9b4d7808bddfabfc6e8af35fe2ae 100644 (file)
@@ -44,8 +44,8 @@ volume_opts = [
                help='Name for the VG that will contain exported volumes'),
     cfg.IntOpt('lvm_mirrors',
                default=0,
-               help='If set, create lvms with multiple mirrors. Note that '
-                    'this requires lvm_mirrors + 2 pvs with available space'),
+               help='If >0, create LVs with multiple mirrors. Note that '
+                    'this requires lvm_mirrors + 2 PVs with available space'),
     cfg.StrOpt('lvm_type',
                default='default',
                help='Type of LVM volumes to deploy; (default or thin)'),
index 018ea0a19307a62f4806649181b4cc205988ac8e..db893a3062778fd617c9a1108d14be15bcc036a5 100644 (file)
 # value)
 #volume_group=cinder-volumes
 
-# If set, create lvms with multiple mirrors. Note that this
-# requires lvm_mirrors + 2 pvs with available space (integer
+# If >0, create LVs with multiple mirrors. Note that this
+# requires lvm_mirrors + 2 PVs with available space (integer
 # value)
 #lvm_mirrors=0