]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Remove deprecated options
authorRongze Zhu <zrzhit@gmail.com>
Tue, 1 Sep 2015 02:32:39 +0000 (10:32 +0800)
committerRongze Zhu <zrzhit@gmail.com>
Tue, 1 Sep 2015 04:39:11 +0000 (12:39 +0800)
These options are deprecated and unused, this commit remove them.
  - iscsi_num_targets
  - iser_num_targets

Change-Id: I46597a827f33fc338a11c31434bb0d967b35b003

cinder/volume/driver.py
cinder/volume/targets/iscsi.py

index 36111157e668e527dd1cff7d7f2ad42d99d73af8..c681b24ab730898df614f496b19dfa109b74782b 100644 (file)
@@ -48,10 +48,6 @@ volume_opts = [
                default=0,
                min=0, max=100,
                help='The percentage of backend capacity is reserved'),
-    cfg.IntOpt('iscsi_num_targets',
-               default=None,
-               help='This option is deprecated and unused. '
-                    'It will be removed in the Liberty release.'),
     cfg.StrOpt('iscsi_target_prefix',
                default='iqn.2010-10.org.openstack:',
                help='Prefix for iSCSI volumes'),
@@ -257,10 +253,6 @@ iser_opts = [
                default=3,
                help='The maximum number of times to rescan iSER target'
                     'to find volume'),
-    cfg.IntOpt('iser_num_targets',
-               default=None,
-               help='This option is deprecated and unused. '
-                    'It will be removed in the Liberty release.'),
     cfg.StrOpt('iser_target_prefix',
                default='iqn.2010-10.org.openstack:',
                help='Prefix for iSER volumes'),
index b37535a1b87eae0902809d72b270c4db506b2c4a..792c06eec669a08e69c4d94c68a9c79347c7bcb9 100644 (file)
@@ -14,7 +14,6 @@ import abc
 
 from oslo_concurrency import processutils
 from oslo_log import log as logging
-from oslo_log import versionutils
 
 from cinder import exception
 from cinder.i18n import _, _LI, _LW, _LE
@@ -43,13 +42,6 @@ class ISCSITarget(driver.Target):
         self.protocol = 'iSCSI'
         self.volumes_dir = self.configuration.safe_get('volumes_dir')
 
-        # If any of the deprecated options are set, we'll warn the operator.
-        msg = _LW("The option %s has been deprecated and no longer has "
-                  "any effect. It will be removed in the Liberty release.")
-        for opt in ('iscsi_num_targets', 'iser_num_targets'):
-            if self.configuration.safe_get(opt) is not None:
-                versionutils.report_deprecated_feature(LOG, msg, opt)
-
     def _get_iscsi_properties(self, volume, multipath=False):
         """Gets iscsi configuration