]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Replace FLAGS with cfg.CONF in tests
authorSergey Vilgelm <svilgelm@mirantis.com>
Mon, 10 Jun 2013 08:28:44 +0000 (12:28 +0400)
committerSergey Vilgelm <svilgelm@mirantis.com>
Mon, 10 Jun 2013 14:33:08 +0000 (18:33 +0400)
Replace all the FLAGS with cfg.CONF in cinder/tests
Large commit was split into several parts

Change-Id: I20a188c20f440e4ba3f5167bd65346ec9fb6e90b
Fixes: bug #1182037
41 files changed:
cinder/tests/__init__.py
cinder/tests/api/contrib/test_extended_snapshot_attributes.py
cinder/tests/api/contrib/test_hosts.py
cinder/tests/api/contrib/test_volume_actions.py
cinder/tests/api/middleware/test_sizelimit.py
cinder/tests/api/test_extensions.py
cinder/tests/api/test_router.py
cinder/tests/api/v1/test_snapshots.py
cinder/tests/api/v1/test_volumes.py
cinder/tests/api/v2/test_snapshots.py
cinder/tests/api/v2/test_volumes.py
cinder/tests/declare_flags.py
cinder/tests/image/fake.py
cinder/tests/image/test_glance.py
cinder/tests/integrated/test_extensions.py
cinder/tests/runtime_flags.py
cinder/tests/scheduler/test_host_manager.py
cinder/tests/scheduler/test_rpcapi.py
cinder/tests/scheduler/test_scheduler.py
cinder/tests/test_backup.py
cinder/tests/test_backup_swift.py
cinder/tests/test_db_api.py
cinder/tests/test_drivers_compatibility.py
cinder/tests/test_emc.py
cinder/tests/test_glusterfs.py
cinder/tests/test_nexenta.py
cinder/tests/test_policy.py
cinder/tests/test_quota.py
cinder/tests/test_service.py
cinder/tests/test_utils.py
cinder/tests/test_volume.py
cinder/tests/test_volume_configuration.py
cinder/tests/test_volume_rpcapi.py
cinder/tests/test_volume_transfer.py
cinder/tests/test_volume_types.py
cinder/tests/test_volume_utils.py
cinder/tests/test_windows.py
cinder/tests/test_xenapi_sm.py
cinder/tests/test_xiv.py
cinder/tests/utils.py
cinder/tests/windows/windowsutils.py

index cb265b4aac7649b519e7dfb78acf931b80d3963d..270711744a832965d325051cebf91f733210844a 100644 (file)
@@ -31,6 +31,7 @@
 """
 
 import eventlet
+
 eventlet.monkey_patch()
 
 # See http://code.google.com/p/python-nose/issues/detail?id=373
@@ -40,23 +41,26 @@ setattr(__builtin__, '_', lambda x: x)
 import os
 import shutil
 
+from oslo.config import cfg
+
 from cinder.db.sqlalchemy.api import get_engine
-from cinder import flags
 
-FLAGS = flags.FLAGS
 
 _DB = None
 
+CONF = cfg.CONF
+
 
 def reset_db():
-    if FLAGS.database.connection == "sqlite://":
+    if CONF.database.connection == "sqlite://":
         engine = get_engine()
         engine.dispose()
         conn = engine.connect()
         conn.connection.executescript(_DB)
     else:
-        shutil.copyfile(os.path.join(FLAGS.state_path, FLAGS.sqlite_clean_db),
-                        os.path.join(FLAGS.state_path, FLAGS.sqlite_db))
+        shutil.copyfile(os.path.join(CONF.state_path,
+                                     CONF.sqlite_clean_db),
+                        os.path.join(CONF.state_path, CONF.sqlite_db))
 
 
 def setup():
@@ -64,22 +68,22 @@ def setup():
 
     from cinder.db import migration
     from cinder.tests import fake_flags
-    fake_flags.set_defaults(FLAGS)
+    fake_flags.set_defaults(CONF)
 
-    if FLAGS.database.connection == "sqlite://":
+    if CONF.database.connection == "sqlite://":
         if migration.db_version() > 1:
             return
     else:
-        testdb = os.path.join(FLAGS.state_path, FLAGS.sqlite_db)
+        testdb = os.path.join(CONF.state_path, CONF.sqlite_db)
         if os.path.exists(testdb):
             return
     migration.db_sync()
 
-    if FLAGS.database.connection == "sqlite://":
+    if CONF.database.connection == "sqlite://":
         global _DB
         engine = get_engine()
         conn = engine.connect()
         _DB = "".join(line for line in conn.connection.iterdump())
     else:
-        cleandb = os.path.join(FLAGS.state_path, FLAGS.sqlite_clean_db)
+        cleandb = os.path.join(CONF.state_path, CONF.sqlite_clean_db)
         shutil.copyfile(testdb, cleandb)
index 5937ac9628944705f0e89b25e58b12ed1b9c7b67..28236d01c6582db7eca44bfc9d4f5ecee176c01f 100644 (file)
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
+
 from lxml import etree
 import webob
 
 from cinder.api.contrib import extended_snapshot_attributes
 from cinder import exception
-from cinder import flags
 from cinder.openstack.common import jsonutils
 from cinder import test
 from cinder.tests.api import fakes
 from cinder import volume
 
 
-FLAGS = flags.FLAGS
-
-
 UUID1 = '00000000-0000-0000-0000-000000000001'
 UUID2 = '00000000-0000-0000-0000-000000000002'
 
index 525f96305133a720f89058524341aca8b56ba459..3f9112736c99cb8547090fd6f55928d176d52b52 100644 (file)
@@ -23,13 +23,11 @@ import webob.exc
 from cinder.api.contrib import hosts as os_hosts
 from cinder import context
 from cinder import db
-from cinder import flags
 from cinder.openstack.common import log as logging
 from cinder.openstack.common import timeutils
 from cinder import test
 
 
-FLAGS = flags.FLAGS
 LOG = logging.getLogger(__name__)
 created_time = datetime.datetime(2012, 11, 14, 1, 20, 41, 95099)
 curr_time = timeutils.utcnow()
index aed906985d2c579e9aaba42bd9d7196cce29e6fe..38b747f75e0141e4b1ceacf88d91c000f71b55aa 100644 (file)
@@ -20,7 +20,6 @@ import webob
 
 from cinder.api.contrib import volume_actions
 from cinder import exception
-from cinder import flags
 from cinder.openstack.common import jsonutils
 from cinder.openstack.common.rpc import common as rpc_common
 from cinder import test
@@ -30,9 +29,6 @@ from cinder import volume
 from cinder.volume import api as volume_api
 
 
-FLAGS = flags.FLAGS
-
-
 def fake_volume_api(*args, **kwargs):
     return True
 
index 3b87a2cd963c8e35cf63cedefaefa1f92c290239..c955d386e151d29077b7af1e5e9c7a9a12fcadcc 100644 (file)
 #    under the License.
 
 import StringIO
+
+from oslo.config import cfg
 import webob
 
 from cinder.api.middleware import sizelimit
-from cinder import flags
 from cinder import test
 
-FLAGS = flags.FLAGS
-MAX_REQUEST_BODY_SIZE = FLAGS.osapi_max_request_body_size
+
+CONF = cfg.CONF
+
+MAX_REQUEST_BODY_SIZE = CONF.osapi_max_request_body_size
 
 
 class TestLimitingReader(test.TestCase):
index 9ad19f18eda6ff4aa5c5ac07aa94cc6c4e687736..d46dbcad50e684685aab2ca530810105f1238b8c 100644 (file)
 
 import iso8601
 from lxml import etree
+from oslo.config import cfg
 import webob
 
 from cinder.api.v1 import router
 from cinder.api import xmlutil
-from cinder import flags
 from cinder.openstack.common import jsonutils
 from cinder import test
 
-FLAGS = flags.FLAGS
+
 NS = "{http://docs.openstack.org/common/api/v1.0}"
 
 
+CONF = cfg.CONF
+
+
 class ExtensionTestCase(test.TestCase):
     def setUp(self):
         super(ExtensionTestCase, self).setUp()
-        ext_list = FLAGS.osapi_volume_extension[:]
+        ext_list = CONF.osapi_volume_extension[:]
         fox = ('cinder.tests.api.extensions.foxinsocks.Foxinsocks')
         if fox not in ext_list:
             ext_list.append(fox)
index 9a3f3da2a86aaafd7e34e9cd3ac104196b4b9317..f63c287f3943546bd78f240f45915454eaa0d461 100644 (file)
@@ -19,12 +19,10 @@ from cinder.api.v1 import router
 from cinder.api.v1 import snapshots
 from cinder.api.v1 import volumes
 from cinder.api import versions
-from cinder import flags
 from cinder.openstack.common import log as logging
 from cinder import test
 from cinder.tests.api import fakes
 
-FLAGS = flags.FLAGS
 
 LOG = logging.getLogger(__name__)
 
index 6a13179ac3a1942ca3a198eedf00c0e0c1aecaa3..a0c0fa0b5fbbd738a03a22ee434eb063cbce622a 100644 (file)
@@ -21,7 +21,6 @@ import webob
 from cinder.api.v1 import snapshots
 from cinder import db
 from cinder import exception
-from cinder import flags
 from cinder.openstack.common import log as logging
 from cinder import test
 from cinder.tests.api import fakes
@@ -29,7 +28,6 @@ from cinder.tests.api.v1 import stubs
 from cinder import volume
 
 
-FLAGS = flags.FLAGS
 LOG = logging.getLogger(__name__)
 
 UUID = '00000000-0000-0000-0000-000000000001'
index b199ca40e84489036e69c71d426163119d5c14d1..3b0df4e711a7896acb13c35b09080c5ed0be5f4e 100644 (file)
@@ -16,6 +16,7 @@
 import datetime
 
 from lxml import etree
+from oslo.config import cfg
 import webob
 
 from cinder.api import extensions
@@ -23,7 +24,6 @@ from cinder.api.v1 import volumes
 from cinder import context
 from cinder import db
 from cinder import exception
-from cinder import flags
 from cinder import test
 from cinder.tests.api import fakes
 from cinder.tests.api.v2 import stubs
@@ -31,11 +31,12 @@ from cinder.tests.image import fake as fake_image
 from cinder.volume import api as volume_api
 
 
-FLAGS = flags.FLAGS
 NS = '{http://docs.openstack.org/volume/api/v1}'
 
 TEST_SNAPSHOT_UUID = '00000000-0000-0000-0000-000000000001'
 
+CONF = cfg.CONF
+
 
 def stub_snapshot_get(self, context, snapshot_id):
     if snapshot_id != TEST_SNAPSHOT_UUID:
@@ -94,7 +95,7 @@ class VolumeApiTest(test.TestCase):
         self.assertEqual(res_dict, expected)
 
     def test_volume_create_with_type(self):
-        vol_type = FLAGS.default_volume_type
+        vol_type = CONF.default_volume_type
         db.volume_type_create(context.get_admin_context(),
                               dict(name=vol_type, extra_specs={}))
         db_vol_type = db.volume_type_get_by_name(context.get_admin_context(),
index f003cb0b3d1431907bb77a2ad344e61a0c2c054a..5404ab102ef905fc25b8682281e828d91a95e184 100644 (file)
@@ -21,7 +21,6 @@ import webob
 from cinder.api.v2 import snapshots
 from cinder import db
 from cinder import exception
-from cinder import flags
 from cinder.openstack.common import log as logging
 from cinder import test
 from cinder.tests.api import fakes
@@ -29,7 +28,6 @@ from cinder.tests.api.v2 import stubs
 from cinder import volume
 
 
-FLAGS = flags.FLAGS
 LOG = logging.getLogger(__name__)
 
 UUID = '00000000-0000-0000-0000-000000000001'
index 7264c30a4977e62c537c2cf2036ad2beae825b00..377fb1ec57cf99c1498c40040ebfad53df915bbe 100644 (file)
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
+
 import datetime
 
 from lxml import etree
+from oslo.config import cfg
 import webob
 
 from cinder.api import extensions
@@ -23,7 +25,6 @@ from cinder.api.v2 import volumes
 from cinder import context
 from cinder import db
 from cinder import exception
-from cinder import flags
 from cinder import test
 from cinder.tests.api import fakes
 from cinder.tests.api.v2 import stubs
@@ -31,7 +32,8 @@ from cinder.tests.image import fake as fake_image
 from cinder.volume import api as volume_api
 
 
-FLAGS = flags.FLAGS
+CONF = cfg.CONF
+
 NS = '{http://docs.openstack.org/api/openstack-volume/2.0/content}'
 
 TEST_SNAPSHOT_UUID = '00000000-0000-0000-0000-000000000001'
@@ -98,9 +100,10 @@ class VolumeApiTest(test.TestCase):
         self.assertEqual(res_dict, expected)
 
     def test_volume_create_with_type(self):
-        vol_type = db.volume_type_create(context.get_admin_context(),
-                                         dict(name=FLAGS.default_volume_type,
-                                              extra_specs={}))
+        vol_type = db.volume_type_create(
+            context.get_admin_context(),
+            dict(name=CONF.default_volume_type, extra_specs={})
+        )
 
         db_vol_type = db.volume_type_get(context.get_admin_context(),
                                          vol_type.id)
index 4f569c0266d1aca371332d32508cdcd50976c473..0eacd85d9669fb504121944984b73a44fdb7f1a9 100644 (file)
@@ -16,9 +16,9 @@
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
+
 from oslo.config import cfg
 
-from cinder import flags
 
-FLAGS = flags.FLAGS
-FLAGS.register_opt(cfg.IntOpt('answer', default=42, help='test flag'))
+CONF = cfg.CONF
+CONF.register_opt(cfg.IntOpt('answer', default=42, help='test flag'))
index e8feb2433d718f116fdee346b5c3e877b27ef31f..814b3bd82f5ff213da93853a3f23a4fbbd3a341b 100644 (file)
@@ -23,7 +23,6 @@ import datetime
 import uuid
 
 from cinder import exception
-from cinder import flags
 import cinder.image.glance
 from cinder.openstack.common import log as logging
 
@@ -31,9 +30,6 @@ from cinder.openstack.common import log as logging
 LOG = logging.getLogger(__name__)
 
 
-FLAGS = flags.FLAGS
-
-
 class _FakeImageService(object):
     """Mock (fake) image service for unit testing."""
 
index 745f8b654a087cff85b2a3467bcf818c7677c7a4..9248d0f312326dd978a2320f7117dc37f25d157f 100644 (file)
 import datetime
 
 import glanceclient.exc
+from glanceclient.v2.client import Client as glanceclient_v2
+from oslo.config import cfg
 
 from cinder import context
 from cinder import exception
-from cinder import flags
 from cinder.image import glance
 from cinder import test
 from cinder.tests.glance import stubs as glance_stubs
-from glanceclient.v2.client import Client as glanceclient_v2
 
 
-FLAGS = flags.FLAGS
+CONF = cfg.CONF
 
 
 class NullWriter(object):
@@ -575,7 +575,7 @@ class TestGlanceClientVersion(test.TestCase):
                                                        9292)
         self.assertEquals(client_wrapper_v2.client.__module__,
                           'glanceclient.v2.client')
-        FLAGS.reset()
+        CONF.reset()
 
     def test_glance_version_by_arg(self):
         """Test glance version set by arg to GlanceClientWrapper"""
index 363baed66ba16be5c3790e2aef84084433a86abc..039ecbe13e52154663105927c3b8e4b86e44f0b9 100644 (file)
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
-from cinder import flags
+
+from oslo.config import cfg
+
 from cinder.openstack.common import log as logging
 from cinder.tests.integrated import integrated_helpers
 
 
-FLAGS = flags.FLAGS
+CONF = cfg.CONF
+
 LOG = logging.getLogger(__name__)
 
 
 class ExtensionsTest(integrated_helpers._IntegratedTestBase):
     def _get_flags(self):
         f = super(ExtensionsTest, self)._get_flags()
-        f['osapi_volume_extension'] = FLAGS.osapi_volume_extension[:]
+        f['osapi_volume_extension'] = CONF.osapi_volume_extension[:]
         f['osapi_volume_extension'].append(
             'cinder.tests.api.extensions.foxinsocks.Foxinsocks')
         return f
index ad2994cde0cf13147ade7d60e5d3469309c07342..ca90e269490ce0f434e7191ec49ec25fc01fcf8a 100644 (file)
@@ -16,9 +16,9 @@
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
+
 from oslo.config import cfg
 
-from cinder import flags
 
-FLAGS = flags.FLAGS
-FLAGS.register_opt(cfg.IntOpt('runtime_answer', default=54, help='test flag'))
+CONF = cfg.CONF
+CONF.register_opt(cfg.IntOpt('runtime_answer', default=54, help='test flag'))
index ca3a9f521c9c6156c9c7ed018244ce1a5226afac..960d59bd61fee6330f6604eb7a2a015e3b771f5c 100644 (file)
 Tests For HostManager
 """
 
+from oslo.config import cfg
 
 from cinder import db
 from cinder import exception
-from cinder import flags
 from cinder.openstack.common.scheduler import filters
 from cinder.openstack.common import timeutils
 from cinder.scheduler import host_manager
@@ -27,7 +27,7 @@ from cinder import test
 from cinder.tests.scheduler import fakes
 
 
-FLAGS = flags.FLAGS
+CONF = cfg.CONF
 
 
 class FakeFilterClass1(filters.BaseHostFilter):
@@ -137,7 +137,7 @@ class HostManagerTestCase(test.TestCase):
 
     def test_get_all_host_states(self):
         context = 'fake_context'
-        topic = FLAGS.volume_topic
+        topic = CONF.volume_topic
 
         self.mox.StubOutWithMock(db, 'service_get_all_by_topic')
         self.mox.StubOutWithMock(host_manager.LOG, 'warn')
index 52fbb18aa503b092054f428c5e85211e618d0331..ce2509c0138a4120c93c434cc89122c82fe04f53 100644 (file)
 Unit Tests for cinder.scheduler.rpcapi
 """
 
+
+from oslo.config import cfg
+
 from cinder import context
-from cinder import flags
 from cinder.openstack.common import rpc
 from cinder.scheduler import rpcapi as scheduler_rpcapi
 from cinder import test
 
 
-FLAGS = flags.FLAGS
+CONF = cfg.CONF
 
 
 class SchedulerRpcAPITestCase(test.TestCase):
@@ -58,7 +60,7 @@ class SchedulerRpcAPITestCase(test.TestCase):
         retval = getattr(rpcapi, method)(ctxt, **kwargs)
 
         self.assertEqual(retval, expected_retval)
-        expected_args = [ctxt, FLAGS.scheduler_topic, expected_msg]
+        expected_args = [ctxt, CONF.scheduler_topic, expected_msg]
         for arg, expected_arg in zip(self.fake_args, expected_args):
             self.assertEqual(arg, expected_arg)
 
index 5a954ad016fbb6ed831b266637ad114313a95b02..7cf279675fac90e96b142326625dfff32c700d2c 100644 (file)
 Tests For Scheduler
 """
 
-
 from cinder import context
 from cinder import db
 from cinder import exception
-from cinder import flags
 from cinder.openstack.common import timeutils
 from cinder.scheduler import driver
 from cinder.scheduler import manager
@@ -31,9 +29,6 @@ from cinder import test
 from cinder import utils
 
 
-FLAGS = flags.FLAGS
-
-
 class SchedulerManagerTestCase(test.TestCase):
     """Test case for scheduler manager."""
 
index bffddb36c86701e847162a1151bca49bf44b6e86..9679ea8aea7052c5f092c3a7594b2d93b9c19d12 100644 (file)
@@ -19,16 +19,19 @@ Tests for Backup code.
 
 import tempfile
 
+from oslo.config import cfg
+
 from cinder import context
 from cinder import db
 from cinder import exception
-from cinder import flags
 from cinder.openstack.common import importutils
 from cinder.openstack.common import log as logging
 from cinder.openstack.common import timeutils
 from cinder import test
 
-FLAGS = flags.FLAGS
+
+CONF = cfg.CONF
+
 LOG = logging.getLogger(__name__)
 
 
@@ -45,7 +48,7 @@ class BackupTestCase(test.TestCase):
         self.flags(connection_type='fake',
                    volumes_dir=vol_tmpdir)
         self.backup_mgr = \
-            importutils.import_object(FLAGS.backup_manager)
+            importutils.import_object(CONF.backup_manager)
         self.backup_mgr.host = 'testhost'
         self.ctxt = context.get_admin_context()
 
@@ -74,7 +77,7 @@ class BackupTestCase(test.TestCase):
         backup['container'] = container
         backup['status'] = status
         backup['fail_reason'] = ''
-        backup['service'] = FLAGS.backup_service
+        backup['service'] = CONF.backup_service
         backup['size'] = size
         backup['object_count'] = object_count
         return db.backup_create(self.ctxt, backup)['id']
index 013641729a9f31026fef5d022bdef53a20d6ad1d..2c5f97f97853a7c829b518be018a4de9f1af3a59 100644 (file)
@@ -23,18 +23,17 @@ import os
 import tempfile
 import zlib
 
+from swiftclient import client as swift
+
 from cinder.backup.services.swift import SwiftBackupService
 from cinder import context
 from cinder import db
 from cinder import exception
-from cinder import flags
 from cinder.openstack.common import log as logging
 from cinder import test
 from cinder.tests.backup.fake_swift_client import FakeSwiftClient
-from swiftclient import client as swift
 
 
-FLAGS = flags.FLAGS
 LOG = logging.getLogger(__name__)
 
 
index ef12e572dcc2f044ba4ee29c3e85dac9667d208c..92ca9ef984aa47a5b913ca43101fcd123a306941 100644 (file)
 """Unit tests for cinder.db.api."""
 
 
+import datetime
+
+from oslo.config import cfg
+
 from cinder import context
 from cinder import db
 from cinder import exception
-from cinder import flags
 from cinder.openstack.common import uuidutils
 from cinder.quota import ReservableResource
 from cinder import test
-from datetime import datetime
-from datetime import timedelta
 
-FLAGS = flags.FLAGS
+
+CONF = cfg.CONF
 
 
 def _quota_reserve(context, project_id):
@@ -49,9 +51,11 @@ def _quota_reserve(context, project_id):
             resource,
             get_sync(resource, i), 'quota_res_%d' % i)
         deltas[resource] = i
-    return db.quota_reserve(context, resources, quotas, deltas,
-                            datetime.utcnow(), datetime.utcnow(),
-                            timedelta(days=1), project_id)
+    return db.quota_reserve(
+        context, resources, quotas, deltas,
+        datetime.datetime.utcnow(), datetime.datetime.utcnow(),
+        datetime.timedelta(days=1), project_id
+    )
 
 
 class ModelsObjectComparatorMixin(object):
@@ -224,9 +228,12 @@ class DBAPIServiceTestCase(BaseTest):
 
     def test_service_get_all_volume_sorted(self):
         values = [
-            ({'host': 'h1', 'binary': 'a', 'topic': FLAGS.volume_topic}, 100),
-            ({'host': 'h2', 'binary': 'b', 'topic': FLAGS.volume_topic}, 200),
-            ({'host': 'h3', 'binary': 'b', 'topic': FLAGS.volume_topic}, 300)]
+            ({'host': 'h1', 'binary': 'a', 'topic': CONF.volume_topic},
+             100),
+            ({'host': 'h2', 'binary': 'b', 'topic': CONF.volume_topic},
+             200),
+            ({'host': 'h3', 'binary': 'b', 'topic': CONF.volume_topic},
+             300)]
         services = []
         for vals, size in values:
             services.append(self._create_service(vals))
@@ -386,12 +393,15 @@ class DBAPIReservationTestCase(BaseTest):
 
     def setUp(self):
         super(DBAPIReservationTestCase, self).setUp()
-        self.values = {'uuid': 'sample-uuid',
-                       'project_id': 'project1',
-                       'resource': 'resource',
-                       'delta': 42,
-                       'expire': datetime.utcnow() + timedelta(days=1),
-                       'usage': {'id': 1}}
+        self.values = {
+            'uuid': 'sample-uuid',
+            'project_id': 'project1',
+            'resource': 'resource',
+            'delta': 42,
+            'expire': (datetime.datetime.utcnow() +
+                       datetime.timedelta(days=1)),
+            'usage': {'id': 1}
+        }
 
     def test_reservation_create(self):
         reservation = db.reservation_create(self.ctxt, **self.values)
@@ -463,7 +473,8 @@ class DBAPIReservationTestCase(BaseTest):
                              'project1'))
 
     def test_reservation_expire(self):
-        self.values['expire'] = datetime.utcnow() + timedelta(days=1)
+        self.values['expire'] = datetime.datetime.utcnow() + \
+            datetime.timedelta(days=1)
         reservations = _quota_reserve(self.ctxt, 'project1')
         db.reservation_expire(self.ctxt)
 
index f69eb65ab27f074b31b6726a8f5898b8e0c2fd85..7dc6157de658dead492ff18928359224d7d6d4ec 100644 (file)
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
+
+from oslo.config import cfg
+
 from cinder import context
-from cinder import flags
 from cinder.openstack.common import importutils
 from cinder import test
 from cinder.volume.drivers.solidfire import SolidFire
 
-FLAGS = flags.FLAGS
+
+CONF = cfg.CONF
 
 RBD_MODULE = "cinder.volume.drivers.rbd.RBDDriver"
 SHEEPDOG_MODULE = "cinder.volume.drivers.sheepdog.SheepdogDriver"
@@ -46,7 +49,7 @@ class VolumeDriverCompatibility(test.TestCase):
 
     def setUp(self):
         super(VolumeDriverCompatibility, self).setUp()
-        self.manager = importutils.import_object(FLAGS.volume_manager)
+        self.manager = importutils.import_object(CONF.volume_manager)
         self.context = context.get_admin_context()
 
     def tearDown(self):
index e12a1a94355577d903e22da5859958994128981a..1184349028a70f6829618179e6142d02f7da16e2 100644 (file)
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
-import mox
 import os
 import shutil
 import tempfile
 from xml.dom.minidom import Document
 
+import mox
+
 from cinder import exception
-from cinder import flags
 from cinder.openstack.common import log as logging
 from cinder import test
 from cinder.volume import configuration as conf
 from cinder.volume.drivers.emc.emc_smis_common import EMCSMISCommon
 from cinder.volume.drivers.emc.emc_smis_iscsi import EMCSMISISCSIDriver
 
+
 CINDER_EMC_CONFIG_FILE = '/etc/cinder/cinder_emc_config.xml'
 LOG = logging.getLogger(__name__)
 
index 5b3790f5d58760667d647209ad05322adc21178c..762b4a4cc9d17635e6b8dc66532aad39f16010da 100644 (file)
@@ -16,7 +16,7 @@
 #    under the License.
 """Unit tests for the GlusterFS driver module."""
 
-import __builtin__
+
 import errno
 import os
 
@@ -29,7 +29,6 @@ from cinder import context
 from cinder import exception
 from cinder.exception import ProcessExecutionError
 from cinder import test
-
 from cinder.volume import configuration as conf
 from cinder.volume.drivers import glusterfs
 
index aadefc1ffbcf09727f1b6341de36de7eddf99ec4..08f9e1d0a87b61f2f8c318243634a0b2158c7787 100644 (file)
@@ -22,13 +22,15 @@ Unit tests for OpenStack Cinder volume driver
 import base64
 import urllib2
 
-import cinder.flags
+from oslo.config import cfg
+
 from cinder import test
 from cinder.volume.drivers import nexenta
 from cinder.volume.drivers.nexenta import jsonrpc
 from cinder.volume.drivers.nexenta import volume
 
-FLAGS = cinder.flags.FLAGS
+
+CONF = cfg.CONF
 
 
 class TestNexentaDriver(test.TestCase):
@@ -148,9 +150,9 @@ class TestNexentaDriver(test.TestCase):
         self.assertEquals(
             retval,
             {'provider_location':
-                '%s:%s,1 %s%s 0' % (FLAGS.nexenta_host,
-                                    FLAGS.nexenta_iscsi_target_portal_port,
-                                    FLAGS.nexenta_target_prefix,
+                '%s:%s,1 %s%s 0' % (CONF.nexenta_host,
+                                    CONF.nexenta_iscsi_target_portal_port,
+                                    CONF.nexenta_target_prefix,
                                     self.TEST_VOLUME_NAME)})
 
     def __get_test(i):
@@ -203,7 +205,7 @@ class TestNexentaDriver(test.TestCase):
                  'available': '5368709120G',
                  'health': 'ONLINE'}
         self.nms_mock.volume.get_child_props(
-            FLAGS.nexenta_volume,
+            CONF.nexenta_volume,
             'health|size|used|available').AndReturn(stats)
         self.mox.ReplayAll()
         stats = self.drv.get_volume_stats(True)
index 5d05273dcd4e603909e02f1222307f1794267306..a65fc66ad95a1fd8dd7b94d7712e7418d709a781 100644 (file)
@@ -21,16 +21,18 @@ import os.path
 import StringIO
 import urllib2
 
+from oslo.config import cfg
+
 from cinder import context
 from cinder import exception
-from cinder import flags
 import cinder.openstack.common.policy
 from cinder.openstack.common import policy as common_policy
 from cinder import policy
 from cinder import test
 from cinder import utils
 
-FLAGS = flags.FLAGS
+
+CONF = cfg.CONF
 
 
 class PolicyFileTestCase(test.TestCase):
@@ -209,7 +211,7 @@ class ContextIsAdminPolicyTestCase(test.TestCase):
         rules = {
             'context_is_admin': [["role:administrator"], ["role:johnny-admin"]]
         }
-        brain = common_policy.Brain(rules, FLAGS.policy_default_rule)
+        brain = common_policy.Brain(rules, CONF.policy_default_rule)
         common_policy.set_brain(brain)
         ctx = context.RequestContext('fake', 'fake', roles=['johnny-admin'])
         self.assert_(ctx.is_admin)
@@ -224,7 +226,7 @@ class ContextIsAdminPolicyTestCase(test.TestCase):
             "admin_or_owner": [["role:admin"], ["project_id:%(project_id)s"]],
             "default": [["rule:admin_or_owner"]],
         }
-        brain = common_policy.Brain(rules, FLAGS.policy_default_rule)
+        brain = common_policy.Brain(rules, CONF.policy_default_rule)
         common_policy.set_brain(brain)
         ctx = context.RequestContext('fake', 'fake')
         self.assertFalse(ctx.is_admin)
index c31ed52c25ad631d74b49462a92976e5efd7dc84..6639cd37838f6a022c1c938a0329821f669a1c4c 100644 (file)
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
+
 import datetime
 
+from oslo.config import cfg
+
 from cinder import context
 from cinder import db
 from cinder.db.sqlalchemy import api as sqa_api
 from cinder.db.sqlalchemy import models as sqa_models
 from cinder import exception
-from cinder import flags
 from cinder.openstack.common import rpc
 from cinder.openstack.common import timeutils
 from cinder import quota
@@ -32,7 +34,7 @@ import cinder.tests.image.fake
 from cinder import volume
 
 
-FLAGS = flags.FLAGS
+CONF = cfg.CONF
 
 
 class QuotaIntegrationTestCase(test.TestCase):
@@ -82,7 +84,7 @@ class QuotaIntegrationTestCase(test.TestCase):
 
     def test_too_many_volumes(self):
         volume_ids = []
-        for i in range(FLAGS.quota_volumes):
+        for i in range(CONF.quota_volumes):
             vol_ref = self._create_volume()
             volume_ids.append(vol_ref['id'])
         self.assertRaises(exception.QuotaError,
@@ -130,7 +132,7 @@ class QuotaIntegrationTestCase(test.TestCase):
 
         # Make sure the snapshot volume_size isn't included in usage.
         vol_type = db.volume_type_create(self.context,
-                                         dict(name=FLAGS.default_volume_type))
+                                         dict(name=CONF.default_volume_type))
         vol_ref2 = volume.API().create(self.context, 10, '', '')
         usages = db.quota_usage_get_all_by_project(self.context,
                                                    self.project_id)
index 38bdff33e54a3102d6141946990b8453a0ef92a5..1af498385b50034fb5774c129d799f340b5b70cf 100644 (file)
 Unit Tests for remote procedure calls using queue
 """
 
+
 import mox
 from oslo.config import cfg
 
 from cinder import context
 from cinder import db
 from cinder import exception
-from cinder import flags
 from cinder import manager
 from cinder import service
 from cinder import test
 from cinder import wsgi
 
+
 test_service_opts = [
     cfg.StrOpt("fake_manager",
                default="cinder.tests.test_service.FakeManager",
@@ -43,7 +44,8 @@ test_service_opts = [
                default=0,
                help="Port number to bind test service to"), ]
 
-flags.FLAGS.register_opts(test_service_opts)
+CONF = cfg.CONF
+CONF.register_opts(test_service_opts)
 
 
 class FakeManager(manager.Manager):
index 2e7b08629a3f90e789023d6ad0bae57b232457ff..2c807ed19598ae46e29a1fd8781ca286c9985759 100644 (file)
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
+
 import __builtin__
 import datetime
 import hashlib
 import os
-import os.path
 import paramiko
 import StringIO
 import tempfile
 import uuid
 
 import mox
+from oslo.config import cfg
 
 import cinder
 from cinder import exception
-from cinder import flags
-from cinder.openstack.common import strutils
 from cinder.openstack.common import timeutils
 from cinder import test
 from cinder import utils
 
 
-FLAGS = flags.FLAGS
+CONF = cfg.CONF
 
 
 class ExecuteTestCase(test.TestCase):
@@ -302,7 +301,8 @@ class GenericUtilsTestCase(test.TestCase):
 
     def test_generate_glance_url(self):
         generated_url = utils.generate_glance_url()
-        actual_url = "http://%s:%d" % (FLAGS.glance_host, FLAGS.glance_port)
+        actual_url = "http://%s:%d" % (CONF.glance_host,
+                                       CONF.glance_port)
         self.assertEqual(generated_url, actual_url)
 
     def test_read_cached_file(self):
index 39874c81d6a7577e23a99fa4d61eb12cb58c484c..64761e83100be3e2e3cd94fd7977c2606d9c11b3 100644 (file)
@@ -22,16 +22,16 @@ Tests for Volume Code.
 
 import datetime
 import os
-
-import mox
 import shutil
 import tempfile
 
+import mox
+from oslo.config import cfg
+
 from cinder.brick.iscsi import iscsi
 from cinder import context
 from cinder import db
 from cinder import exception
-from cinder import flags
 from cinder.image import image_utils
 from cinder.openstack.common import importutils
 from cinder.openstack.common.notifier import api as notifier_api
@@ -45,8 +45,10 @@ from cinder.tests.image import fake as fake_image
 from cinder.volume import configuration as conf
 from cinder.volume import driver
 
+
 QUOTAS = quota.QUOTAS
-FLAGS = flags.FLAGS
+
+CONF = cfg.CONF
 
 
 class VolumeTestCase(test.TestCase):
@@ -58,7 +60,7 @@ class VolumeTestCase(test.TestCase):
         self.flags(connection_type='fake',
                    volumes_dir=vol_tmpdir,
                    notification_driver=[test_notifier.__name__])
-        self.volume = importutils.import_object(FLAGS.volume_manager)
+        self.volume = importutils.import_object(CONF.volume_manager)
         self.context = context.get_admin_context()
         self.stubs.Set(iscsi.TgtAdm, '_get_target', self.fake_get_target)
         fake_image.stub_out_image_service(self.stubs)
@@ -66,7 +68,7 @@ class VolumeTestCase(test.TestCase):
 
     def tearDown(self):
         try:
-            shutil.rmtree(FLAGS.volumes_dir)
+            shutil.rmtree(CONF.volumes_dir)
         except OSError:
             pass
         notifier_api._reset_drivers()
@@ -85,10 +87,10 @@ class VolumeTestCase(test.TestCase):
         vol['image_id'] = image_id
         vol['user_id'] = 'fake'
         vol['project_id'] = 'fake'
-        vol['availability_zone'] = FLAGS.storage_availability_zone
+        vol['availability_zone'] = CONF.storage_availability_zone
         vol['status'] = status
         vol['attach_status'] = "detached"
-        vol['host'] = FLAGS.host
+        vol['host'] = CONF.host
         if metadata is not None:
             vol['metadata'] = metadata
         return db.volume_create(context.get_admin_context(), vol)
@@ -408,7 +410,7 @@ class VolumeTestCase(test.TestCase):
             self.assert_(iscsi_target not in targets)
             targets.append(iscsi_target)
 
-        total_slots = FLAGS.iscsi_num_targets
+        total_slots = CONF.iscsi_num_targets
         for _index in xrange(total_slots):
             self._create_volume()
         for volume_id in volume_ids:
@@ -1184,7 +1186,7 @@ class DriverTestCase(test.TestCase):
         vol_tmpdir = tempfile.mkdtemp()
         self.flags(volume_driver=self.driver_name,
                    volumes_dir=vol_tmpdir)
-        self.volume = importutils.import_object(FLAGS.volume_manager)
+        self.volume = importutils.import_object(CONF.volume_manager)
         self.context = context.get_admin_context()
         self.output = ""
         self.stubs.Set(iscsi.TgtAdm, '_get_target', self.fake_get_target)
@@ -1196,7 +1198,7 @@ class DriverTestCase(test.TestCase):
 
     def tearDown(self):
         try:
-            shutil.rmtree(FLAGS.volumes_dir)
+            shutil.rmtree(CONF.volumes_dir)
         except OSError:
             pass
         super(DriverTestCase, self).tearDown()
@@ -1267,7 +1269,7 @@ class ISCSITestCase(DriverTestCase):
 
         iscsi_driver = driver.ISCSIDriver(configuration=configuration)
         iscsi_driver._execute = lambda *a, **kw: \
-            ("%s dummy" % FLAGS.iscsi_ip_address, '')
+            ("%s dummy" % CONF.iscsi_ip_address, '')
         volume = {"name": "dummy",
                   "host": "0.0.0.0"}
         iscsi_driver._do_iscsi_discovery(volume)
index d2751c1d98d2e2b192805419255df9859843e40e..07da34b7d493027fb2674cff02c349029bfc18fa 100644 (file)
 
 """Tests for the configuration wrapper in volume drivers."""
 
+
 from oslo.config import cfg
 
-from cinder import flags
 from cinder.openstack.common import log as logging
 from cinder import test
 from cinder.volume import configuration
-from cinder.volume import driver
 
 
 LOG = logging.getLogger(__name__)
-FLAGS = flags.FLAGS
 
 
 volume_opts = [
@@ -39,8 +37,9 @@ more_volume_opts = [
     cfg.IntOpt('int_opt', default=1),
 ]
 
-FLAGS.register_opts(volume_opts)
-FLAGS.register_opts(more_volume_opts)
+CONF = cfg.CONF
+CONF.register_opts(volume_opts)
+CONF.register_opts(more_volume_opts)
 
 
 class VolumeConfigurationTest(test.TestCase):
@@ -52,20 +51,20 @@ class VolumeConfigurationTest(test.TestCase):
 
     def test_group_grafts_opts(self):
         c = configuration.Configuration(volume_opts, config_group='foo')
-        self.assertEquals(c.str_opt, FLAGS.foo.str_opt)
-        self.assertEquals(c.bool_opt, FLAGS.foo.bool_opt)
+        self.assertEquals(c.str_opt, CONF.foo.str_opt)
+        self.assertEquals(c.bool_opt, CONF.foo.bool_opt)
 
     def test_opts_no_group(self):
         c = configuration.Configuration(volume_opts)
-        self.assertEquals(c.str_opt, FLAGS.str_opt)
-        self.assertEquals(c.bool_opt, FLAGS.bool_opt)
+        self.assertEquals(c.str_opt, CONF.str_opt)
+        self.assertEquals(c.bool_opt, CONF.bool_opt)
 
     def test_grafting_multiple_opts(self):
         c = configuration.Configuration(volume_opts, config_group='foo')
         c.append_config_values(more_volume_opts)
-        self.assertEquals(c.str_opt, FLAGS.foo.str_opt)
-        self.assertEquals(c.bool_opt, FLAGS.foo.bool_opt)
-        self.assertEquals(c.int_opt, FLAGS.foo.int_opt)
+        self.assertEquals(c.str_opt, CONF.foo.str_opt)
+        self.assertEquals(c.bool_opt, CONF.foo.bool_opt)
+        self.assertEquals(c.int_opt, CONF.foo.int_opt)
 
     def test_safe_get(self):
         c = configuration.Configuration(volume_opts, config_group='foo')
index 70735243d7136a5295ceec64178d541e640620f3..8bf68c7c3fc3eddb9559ab80869a043380f161a8 100644 (file)
@@ -19,16 +19,17 @@ Unit Tests for cinder.volume.rpcapi
 """
 
 
+from oslo.config import cfg
+
 from cinder import context
 from cinder import db
-from cinder import flags
 from cinder.openstack.common import jsonutils
 from cinder.openstack.common import rpc
 from cinder import test
 from cinder.volume import rpcapi as volume_rpcapi
 
 
-FLAGS = flags.FLAGS
+CONF = cfg.CONF
 
 
 class VolumeRpcAPITestCase(test.TestCase):
@@ -37,7 +38,7 @@ class VolumeRpcAPITestCase(test.TestCase):
         self.context = context.get_admin_context()
         vol = {}
         vol['host'] = 'fake_host'
-        vol['availability_zone'] = FLAGS.storage_availability_zone
+        vol['availability_zone'] = CONF.storage_availability_zone
         vol['status'] = "available"
         vol['attach_status'] = "detached"
         volume = db.volume_create(self.context, vol)
@@ -87,7 +88,7 @@ class VolumeRpcAPITestCase(test.TestCase):
             host = kwargs['host']
         else:
             host = kwargs['volume']['host']
-        expected_topic = '%s.%s' % (FLAGS.volume_topic, host)
+        expected_topic = '%s.%s' % (CONF.volume_topic, host)
 
         self.fake_args = None
         self.fake_kwargs = None
index 1e091117f43cf83f35a5c5dbaa6f62a1a063e5dd..034fe51d68796ab28f10ead82e4b9163124af180 100644 (file)
 #    License for the specific language governing permissions and limitations
 #    under the License.
 """Unit Tests for volume transfers."""
+
+
 import datetime
 
 from cinder import context
 from cinder import db
 from cinder import exception
-from cinder import flags
 from cinder.openstack.common import log as logging
 from cinder import test
 from cinder.transfer import api as transfer_api
-from cinder.volume import api as cinder_api
+
 
 LOG = logging.getLogger(__name__)
 
index 0d162d80fb4a0a6954bf6e4edfdcd4e3d52b0546..b11fa12ef46d6548c2206b96109b18c6fa2a4bad 100644 (file)
 """
 Unit Tests for volume types code
 """
+
+
 import time
 
 from cinder import context
 from cinder.db.sqlalchemy import api as db_api
 from cinder.db.sqlalchemy import models
 from cinder import exception
-from cinder import flags
 from cinder.openstack.common import log as logging
 from cinder import test
 from cinder.tests import fake_flags
 from cinder.volume import volume_types
 
-FLAGS = flags.FLAGS
+
 LOG = logging.getLogger(__name__)
 
 
index 1521149ac06f226a7a60cc632407ad7075e7681f..929397e5ed3b65c6ba84c0352846cdab5d150a26 100644 (file)
 
 """Tests For miscellaneous util methods used with volume."""
 
+
+from oslo.config import cfg
+
 from cinder import context
 from cinder import db
-from cinder import flags
 from cinder.openstack.common import importutils
 from cinder.openstack.common import log as logging
 from cinder.openstack.common.notifier import api as notifier_api
@@ -29,7 +31,8 @@ from cinder.volume import utils as volume_utils
 
 
 LOG = logging.getLogger(__name__)
-FLAGS = flags.FLAGS
+
+CONF = cfg.CONF
 
 
 class UsageInfoTestCase(test.TestCase):
@@ -45,7 +48,7 @@ class UsageInfoTestCase(test.TestCase):
         self.flags(connection_type='fake',
                    host='fake',
                    notification_driver=[test_notifier.__name__])
-        self.volume = importutils.import_object(FLAGS.volume_manager)
+        self.volume = importutils.import_object(CONF.volume_manager)
         self.user_id = 'fake'
         self.project_id = 'fake'
         self.snapshot_id = 'fake'
@@ -63,8 +66,8 @@ class UsageInfoTestCase(test.TestCase):
         vol['snapshot_id'] = self.snapshot_id
         vol['user_id'] = self.user_id
         vol['project_id'] = self.project_id
-        vol['host'] = FLAGS.host
-        vol['availability_zone'] = FLAGS.storage_availability_zone
+        vol['host'] = CONF.host
+        vol['availability_zone'] = CONF.storage_availability_zone
         vol['status'] = "creating"
         vol['attach_status'] = "detached"
         vol['size'] = self.volume_size
index 709e2f5f2b2e4da3b22b63cc49c9af0084743a92..05b1907aea82032d2c4a55d5befa37bc01365dd3 100644 (file)
@@ -18,6 +18,8 @@
 """
 Unit tests for Windows Server 2012 OpenStack Cinder volume driver
 """
+
+
 import sys
 
 from oslo.config import cfg
@@ -27,6 +29,7 @@ from cinder.tests.windows import db_fakes
 from cinder.tests.windows import windowsutils
 from cinder.volume.drivers import windows
 
+
 CONF = cfg.CONF
 
 
index 950bdca9d5fb1018ee6fe92e54a448c4a58d1f02..4253cf0b024b809d24ef3ca30fe75d985b4601d2 100644 (file)
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
+
 import contextlib
 import StringIO
 
 import mock
 import mox
-from oslo.config import cfg
 
 from cinder.db import api as db_api
 from cinder import exception
 from cinder import test
 from cinder.volume import configuration as conf
-from cinder.volume import driver as parent_driver
 from cinder.volume.drivers.xenapi import lib
 from cinder.volume.drivers.xenapi import sm as driver
 from cinder.volume.drivers.xenapi import tools
index 8db11a0cad52c206905104295e46fb9be11b2055..fbb45b14bfa38d42561563293d538f82f6f4a97c 100644 (file)
 #   Erik Zaadi <erikz@il.ibm.com>
 #   Avishay Traeger <avishay@il.ibm.com>
 
+
 import mox
+from oslo.config import cfg
 
 from cinder import exception
-from cinder import flags
 from cinder import test
 from cinder.volume import configuration as conf
 from cinder.volume.drivers import xiv
 
 
-FLAGS = flags.FLAGS
-
 FAKE = "fake"
 VOLUME = {'size': 16,
           'name': FAKE,
@@ -38,6 +37,8 @@ VOLUME = {'size': 16,
 
 CONNECTOR = {'initiator': "iqn.2012-07.org.fake:01:948f189c4695", }
 
+CONF = cfg.CONF
+
 
 class XIVFakeProxyDriver(object):
     """Fake XIV Proxy Driver."""
@@ -56,10 +57,10 @@ class XIVFakeProxyDriver(object):
         self.volumes = {}
 
     def setup(self, context):
-        if self.xiv_info['xiv_user'] != FLAGS.san_login:
+        if self.xiv_info['xiv_user'] != CONF.san_login:
             raise self.exception.NotAuthorized()
 
-        if self.xiv_info['xiv_address'] != FLAGS.san_ip:
+        if self.xiv_info['xiv_address'] != CONF.san_ip:
             raise self.exception.HostNotFound(host='fake')
 
     def create_volume(self, volume):
@@ -127,13 +128,13 @@ class XIVVolumeDriverTest(test.TestCase):
         """Test that the san flags are passed to the XIV proxy."""
 
         self.assertEquals(self.driver.xiv_proxy.xiv_info['xiv_user'],
-                          FLAGS.san_login)
+                          CONF.san_login)
         self.assertEquals(self.driver.xiv_proxy.xiv_info['xiv_pass'],
-                          FLAGS.san_password)
+                          CONF.san_password)
         self.assertEquals(self.driver.xiv_proxy.xiv_info['xiv_address'],
-                          FLAGS.san_ip)
+                          CONF.san_ip)
         self.assertEquals(self.driver.xiv_proxy.xiv_info['xiv_vol_pool'],
-                          FLAGS.san_clustername)
+                          CONF.san_clustername)
 
     def test_setup_should_fail_if_credentials_are_invalid(self):
         """Test that the xiv_proxy validates credentials."""
index 221e6b097b44450da8049dabf2ab782776a96e46..5b775f723aecad8a59eeceb5c9b1d248387538a2 100644 (file)
 #    License for the specific language governing permissions and limitations
 #
 
+
 import os
 
 import cinder.context
 
-FLAGS = cinder.flags.FLAGS
-
 
 def get_test_admin_context():
     return cinder.context.get_admin_context()
index 4ef3212ebb465f660343432e99db7418735b3111..4a35a5b4e5bff8b645f8ec53cb7593d431114a50 100644 (file)
@@ -19,15 +19,15 @@ Windows storage classes to be used in testing.
 """
 
 import os
-import sys
 
-from cinder import flags
+from oslo.config import cfg
 
 # Check needed for unit testing on Unix
 if os.name == 'nt':
     import wmi
 
-FLAGS = flags.FLAGS
+
+CONF = cfg.CONF
 
 
 class WindowsUtils(object):
@@ -89,7 +89,7 @@ class WindowsUtils(object):
     def _get_vhd_path(self, volume_name):
         '''Gets the path disk of the volume.'''
 
-        base_vhd_folder = FLAGS.windows_iscsi_lun_path
+        base_vhd_folder = CONF.windows_iscsi_lun_path
         return os.path.join(base_vhd_folder, volume_name + ".vhd")
 
     def delete_snapshot(self, name):