]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Enhance docstring for iscsi_helper
authorTom Fifield <tom@openstack.org>
Tue, 24 Jun 2014 13:59:20 +0000 (21:59 +0800)
committerTom Fifield <tom@openstack.org>
Tue, 8 Jul 2014 11:39:16 +0000 (19:39 +0800)
The helptext for the iscsi_helper option did not mention the
potential to use lioadm (or fake). This patch simply amends
the text so this functionality is more discoverable.

Change-Id: I4bf4a6f77191a7f42d5b38f71759944e4997ccc6

cinder/volume/driver.py
etc/cinder/cinder.conf.sample

index d55f7d64f0cb433c22fb125f57d96e84174eeb95..a8c12c36e0600528a75c3fdaf199821c40495d17 100644 (file)
@@ -80,7 +80,9 @@ volume_opts = [
                     'for example "-c3" for idle only priority.'),
     cfg.StrOpt('iscsi_helper',
                default='tgtadm',
-               help='iSCSI target user-land tool to use'),
+               help='iSCSI target user-land tool to use. tgtadm is default, '
+                    'use lioadm for LIO iSCSI support, iseradm for the ISER '
+                    'protocol, or fake for testing.'),
     cfg.StrOpt('volumes_dir',
                default='$state_path/volumes',
                help='Volume configuration file storage '
index 363de3d721d849b716eacd2b642f05509e3b02f3..fa6664af2110663bae79b1c6ff3db7785d0f190f 100644 (file)
 # "-c3" for idle only priority. (string value)
 #volume_clear_ionice=<None>
 
-# iSCSI target user-land tool to use (string value)
+# iSCSI target user-land tool to use. tgtadm is default, use
+# lioadm for LIO iSCSI support, iseradm for the ISER protocol,
+# or fake for testing. (string value)
 #iscsi_helper=tgtadm
 
 # Volume configuration file storage directory (string value)