From: Eric Harney <eharney@redhat.com>
Date: Mon, 16 Jun 2014 19:44:09 +0000 (-0400)
Subject: Correct lvm_mirrors help message
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=515b6d31659aa58ef0789d8dcc9abba541f59c6a;p=openstack-build%2Fcinder-build.git

Correct lvm_mirrors help message

The description is not correct if someone sets
lvm_mirrors=0.  Also clarify LVs vs. lvms.

Change-Id: I7ec74fc8beaef5e638bc1a5dac3d7d97f1c474d3
---

diff --git a/cinder/volume/drivers/lvm.py b/cinder/volume/drivers/lvm.py
index dc98a7870..bdfe12d24 100644
--- a/cinder/volume/drivers/lvm.py
+++ b/cinder/volume/drivers/lvm.py
@@ -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)'),
diff --git a/etc/cinder/cinder.conf.sample b/etc/cinder/cinder.conf.sample
index 018ea0a19..db893a306 100644
--- a/etc/cinder/cinder.conf.sample
+++ b/etc/cinder/cinder.conf.sample
@@ -1290,8 +1290,8 @@
 # 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