]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Two choices for iscsi_helper is missing
authornikeshmahalka <Nikesh.Mahalka@emulex.com>
Wed, 4 Mar 2015 09:04:42 +0000 (14:34 +0530)
committernikeshmahalka <Nikesh.Mahalka@emulex.com>
Thu, 5 Mar 2015 04:11:17 +0000 (09:41 +0530)
In the cinder/volume/driver.py, scstadmin and ietadm were missing from
the choices for iscsi_helper. This will cause SCST and IET target helper
functionality to not work properly.

Change-Id: Ia455e61384d283a47f796466615139361ac9e83e
Closes-Bug: #1428034

cinder/volume/driver.py

index b697c087e1bd0db4a01dddb06e017590bae3543a..a92d147349ea0b557956f0b27959f53bfa450ea4 100644 (file)
@@ -95,11 +95,13 @@ volume_opts = [
                     'for example "-c3" for idle only priority.'),
     cfg.StrOpt('iscsi_helper',
                default='tgtadm',
-               choices=['tgtadm', 'lioadm', 'iseradm', 'iscsictl', 'fake'],
+               choices=['tgtadm', 'lioadm', 'scstadmin', 'iseradm', 'iscsictl',
+                        'ietadm', 'fake'],
                help='iSCSI target user-land tool to use. tgtadm is default, '
-                    'use lioadm for LIO iSCSI support, iseradm for the ISER '
-                    'protocol, iscsictl for Chelsio iSCSI Target or fake for '
-                    'testing.'),
+                    'use lioadm for LIO iSCSI support, scstadmin for SCST '
+                    'target support, iseradm for the ISER protocol, ietadm '
+                    'for iSCSI Enterprise Target, iscsictl for Chelsio iSCSI '
+                    'Target or fake for testing.'),
     cfg.StrOpt('volumes_dir',
                default='$state_path/volumes',
                help='Volume configuration file storage '