]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Cleanup unused conf variables
authorMichael Still <mikal@stillhq.com>
Tue, 16 Feb 2016 00:24:43 +0000 (11:24 +1100)
committerMichael Still <mikal@stillhq.com>
Tue, 16 Feb 2016 00:26:06 +0000 (11:26 +1100)
I noticed in nova that we had a lot of unused conf variables. I
wondered if this was a thing in other projects as well. Turns out
it is.

Change-Id: I2424e32dd4ad61881d9b6a0449e12937ea8151e0

20 files changed:
cinder/api/contrib/volume_manage.py
cinder/objects/service.py
cinder/objects/volume_attachment.py
cinder/objects/volume_type.py
cinder/replication/api.py
cinder/tests/unit/api/contrib/test_volume_actions.py
cinder/tests/unit/api/v1/test_snapshot_metadata.py
cinder/tests/unit/api/v2/test_snapshot_metadata.py
cinder/tests/unit/backup/drivers/test_backup_glusterfs.py
cinder/tests/unit/backup/test_rpcapi.py
cinder/tests/unit/db/test_volume_type.py
cinder/tests/unit/scheduler/test_rpcapi.py
cinder/tests/unit/test_db_api.py
cinder/tests/unit/test_hpe_xp_fc.py
cinder/tests/unit/test_ibm_xiv_ds8k.py
cinder/tests/unit/test_netapp_nfs.py
cinder/tests/unit/test_volume_throttling.py
cinder/tests/unit/volume/flows/test_create_volume_flow.py
cinder/volume/qos_specs.py
cinder/wsgi/common.py

index 819e5fad1da1b986aaf9d133a3e1caf642285077..098da3a4261da07650f4d2f7456397d14cb24e8b 100644 (file)
@@ -12,7 +12,6 @@
 #   License for the specific language governing permissions and limitations
 #   under the License.
 
-from oslo_config import cfg
 from oslo_log import log as logging
 from oslo_utils import uuidutils
 from webob import exc
@@ -28,7 +27,6 @@ from cinder import volume as cinder_volume
 from cinder.volume import volume_types
 
 LOG = logging.getLogger(__name__)
-CONF = cfg.CONF
 authorize = extensions.extension_authorizer('volume', 'volume_manage')
 
 
index 8f28bfa674604466908df91a688b19acafdd7d85..f7f4c0ff295a83d2c0b7066139c4919beb8a4b9c 100644 (file)
@@ -12,7 +12,6 @@
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
-from oslo_config import cfg
 from oslo_log import log as logging
 from oslo_utils import versionutils
 from oslo_versionedobjects import fields
@@ -24,7 +23,6 @@ from cinder import objects
 from cinder.objects import base
 from cinder import utils
 
-CONF = cfg.CONF
 LOG = logging.getLogger(__name__)
 
 
index fcb82ccf450c8160aa63308e30e9625f92416ed0..958fbf79dff3cd0c2a6520fe95ca1741b5840748 100644 (file)
@@ -12,7 +12,6 @@
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
-from oslo_config import cfg
 from oslo_log import log as logging
 from oslo_versionedobjects import fields
 
@@ -20,7 +19,6 @@ from cinder import db
 from cinder import objects
 from cinder.objects import base
 
-CONF = cfg.CONF
 LOG = logging.getLogger(__name__)
 
 
index 1f6fcf0d494d47309317cda63e7ec7972725ca00..646796722a213516d5649c2873a97deef980f0f3 100644 (file)
@@ -12,7 +12,6 @@
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
-from oslo_config import cfg
 from oslo_log import log as logging
 from oslo_versionedobjects import fields
 
@@ -22,7 +21,6 @@ from cinder import objects
 from cinder.objects import base
 from cinder.volume import volume_types
 
-CONF = cfg.CONF
 OPTIONAL_FIELDS = ['extra_specs', 'projects']
 LOG = logging.getLogger(__name__)
 
index 199dbd0e5596723945be2a20b56385922f6a17ae..39022d098d78bfbce19cbeba2200c12e301a48ce 100644 (file)
@@ -17,7 +17,6 @@ Handles all requests relating to volume replication.
 """
 import functools
 
-from oslo_config import cfg
 from oslo_log import log as logging
 
 from cinder.db import base
@@ -28,7 +27,6 @@ from cinder import volume as cinder_volume
 from cinder.volume import rpcapi as volume_rpcapi
 from cinder.volume import utils as volume_utils
 
-CONF = cfg.CONF
 
 LOG = logging.getLogger(__name__)
 
index 3fc1b2f98905e2afc9dc60677318c998b19e8a38..d662bce45696c8c85f32dbfa9d087deb225394d4 100644 (file)
@@ -18,7 +18,6 @@ import json
 import uuid
 
 import mock
-from oslo_config import cfg
 import oslo_messaging as messaging
 from oslo_serialization import jsonutils
 import webob
@@ -36,8 +35,6 @@ from cinder import volume
 from cinder.volume import api as volume_api
 from cinder.volume import rpcapi as volume_rpcapi
 
-CONF = cfg.CONF
-
 
 class VolumeActionsTest(test.TestCase):
 
index cfcbb39b8f94cb9310cb352d96e7c8650ba1ee61..9b0a39413369607c9c554798d1825086c7edb9e9 100644 (file)
@@ -16,7 +16,6 @@
 import uuid
 
 import mock
-from oslo_config import cfg
 from oslo_serialization import jsonutils
 import webob
 
@@ -33,9 +32,6 @@ from cinder.tests.unit import fake_volume
 from cinder import volume
 
 
-CONF = cfg.CONF
-
-
 def return_create_snapshot_metadata(context, snapshot_id, metadata, delete):
     return stub_snapshot_metadata()
 
index c3676c0f93b1a35a015105050b64e5182a0ab851..605efdea490b8b4cf0a80a5379e41aa6e3fe3b56 100644 (file)
@@ -16,7 +16,6 @@
 import uuid
 
 import mock
-from oslo_config import cfg
 from oslo_serialization import jsonutils
 import webob
 
@@ -33,9 +32,6 @@ from cinder.tests.unit import fake_volume
 from cinder import volume
 
 
-CONF = cfg.CONF
-
-
 def return_create_snapshot_metadata(context, snapshot_id, metadata, delete):
     return stub_snapshot_metadata()
 
index a60b06f9d6fd20f88270e7f55b0061d436cdbc60..68d05297b7d6ec71200e11041302ce22361ec1f0 100644 (file)
@@ -20,7 +20,6 @@ import os
 
 import mock
 from os_brick.remotefs import remotefs as remotefs_brick
-from oslo_config import cfg
 
 from cinder.backup.drivers import glusterfs
 from cinder import context
@@ -28,8 +27,6 @@ from cinder import exception
 from cinder import test
 from cinder import utils
 
-CONF = cfg.CONF
-
 FAKE_BACKUP_MOUNT_POINT_BASE = '/fake/mount-point-base'
 FAKE_HOST = 'fake_host'
 FAKE_VOL_NAME = 'backup_vol'
index 4354a8194194053d3b250258fa6f5fae902dc4c3..d3c347ce2125fde9d6c519e29a9dc09e24d2c5cf 100644 (file)
@@ -19,7 +19,6 @@ Unit Tests for cinder.backup.rpcapi
 import copy
 
 import mock
-from oslo_config import cfg
 
 from cinder.backup import rpcapi as backup_rpcapi
 from cinder import context
@@ -28,9 +27,6 @@ from cinder import test
 from cinder.tests.unit import fake_backup
 
 
-CONF = cfg.CONF
-
-
 class BackupRpcAPITestCase(test.TestCase):
     def setUp(self):
         super(BackupRpcAPITestCase, self).setUp()
index f87db1eeb2ddecc9aa98c8133e9ca3e5de7fcaf4..9d748cebd5bd334a31b21dd4f44556aa8538331c 100644 (file)
 
 """Tests for volume type."""
 
-from oslo_config import cfg
-
 from cinder import context
 from cinder import db
 from cinder import test
 from cinder.volume import volume_types
 
-CONF = cfg.CONF
-
 
 class VolumeTypeTestCase(test.TestCase):
     """Test cases for volume type."""
index dd2f875a5aa5ac109e56e6392ce0f13bd9c0ff0f..370d617bfeb223ac134c1b7a060365198767406e 100644 (file)
@@ -20,16 +20,12 @@ Unit Tests for cinder.scheduler.rpcapi
 import copy
 
 import mock
-from oslo_config import cfg
 
 from cinder import context
 from cinder.scheduler import rpcapi as scheduler_rpcapi
 from cinder import test
 
 
-CONF = cfg.CONF
-
-
 class SchedulerRpcAPITestCase(test.TestCase):
 
     def setUp(self):
index ed321537856ac8f753d7c07163c73b0c392f3cf0..0307ad350b4e2ac0e6b86060edacca4a58e65869 100644 (file)
@@ -18,7 +18,6 @@ import datetime
 
 import enum
 import mock
-from oslo_config import cfg
 from oslo_utils import uuidutils
 import six
 
@@ -30,8 +29,6 @@ from cinder import exception
 from cinder import quota
 from cinder import test
 
-CONF = cfg.CONF
-
 THREE = 3
 THREE_HUNDREDS = 300
 ONE_HUNDREDS = 100
index 97635accc2b0539ac0cf48f7ef320878e29addb1..248effba73f6f3d4c30c7e4c5cc58c38c71bde2f 100644 (file)
@@ -25,9 +25,7 @@ from cinder.volume import driver
 from cinder.volume.drivers.hpe import hpe_xp_fc
 from cinder.volume.drivers.hpe import hpe_xp_opts
 
-from oslo_config import cfg
 from oslo_utils import importutils
-CONF = cfg.CONF
 
 NORMAL_LDEV_TYPE = 'Normal'
 POOL_INFO = {'30': {'total_gb': 'infinite', 'free_gb': 'infinite'}}
index 3e6be8a9e9a255ae0d1f5942b966cc1f2d2214c5..df73e620779bb0c6d3c2296baea9852f394b743a 100644 (file)
@@ -22,7 +22,6 @@
 import copy
 
 from mox3 import mox
-from oslo_config import cfg
 
 from cinder import context
 from cinder import exception
@@ -82,8 +81,6 @@ CG_SNAPSHOT = {'id': CG_SNAPSHOT_ID,
 
 CONNECTOR = {'initiator': "iqn.2012-07.org.fake:01:948f189c4695", }
 
-CONF = cfg.CONF
-
 
 class XIVDS8KFakeProxyDriver(object):
     """Fake IBM XIV and DS8K Proxy Driver."""
index 19e0136072c3b4bddcd24766333d78f6e362d54f..2f050457298785324e83579a375982cb91a430b3 100644 (file)
@@ -42,10 +42,6 @@ from cinder.volume.drivers.netapp.dataontap import nfs_base
 from cinder.volume.drivers.netapp.dataontap import ssc_cmode
 from cinder.volume.drivers.netapp import utils
 
-from oslo_config import cfg
-
-
-CONF = cfg.CONF
 
 CONNECTION_INFO = {
     'hostname': 'fake_host',
index 77f15437599ec39f174e1cedb856fa6856c5e294..82e264557fa95c06e073ff7ceecf3639898c214f 100644 (file)
 """Tests for volume copy throttling helpers."""
 
 import mock
-from oslo_config import cfg
 
 from cinder import test
 from cinder import utils
 from cinder.volume import throttling
 
 
-CONF = cfg.CONF
-
-
 class ThrottleTestCase(test.TestCase):
 
     def test_NoThrottle(self):
index d4b9f03790f57db2a30401f538444db1d19b87a0..4302611b15fe534476ac5408022e4684d74ddd3a 100644 (file)
@@ -17,8 +17,6 @@
 import ddt
 import mock
 
-from oslo_config import cfg
-
 from cinder import context
 from cinder import exception
 from cinder.openstack.common import imageutils
@@ -33,8 +31,6 @@ from cinder.tests.unit.volume.flows import fake_volume_api
 from cinder.volume.flows.api import create_volume
 from cinder.volume.flows.manager import create_volume as create_volume_manager
 
-CONF = cfg.CONF
-
 
 @ddt.ddt
 class CreateVolumeFlowTestCase(test.TestCase):
index 529be993faaab5413d11d31ea1c0af3f87c7c3c3..52527d78e23519c73d001ed11145c62329e67a27 100644 (file)
@@ -16,7 +16,6 @@
 """The QoS Specs Implementation"""
 
 
-from oslo_config import cfg
 from oslo_db import exception as db_exc
 from oslo_log import log as logging
 
@@ -27,7 +26,6 @@ from cinder.i18n import _, _LE, _LW
 from cinder.volume import volume_types
 
 
-CONF = cfg.CONF
 LOG = logging.getLogger(__name__)
 
 CONTROL_LOCATION = ['front-end', 'back-end', 'both']
index 4da4ffa0bcc4d02feb3c07f5185d466294e529c5..0eb47e1a0f6959693dbf7f82a23ea82e8e7ee2ea 100644 (file)
 
 """Utility methods for working with WSGI servers."""
 
-from oslo_config import cfg
 from oslo_log import log as logging
 import webob.dec
 import webob.exc
 
 from cinder.i18n import _
 
-CONF = cfg.CONF
 LOG = logging.getLogger(__name__)