Refactoring of QoSSettings to extend QoSConfig 33/47433/3
authorspisarski <s.pisarski@cablelabs.com>
Fri, 17 Nov 2017 20:11:09 +0000 (13:11 -0700)
committerspisarski <s.pisarski@cablelabs.com>
Mon, 20 Nov 2017 16:50:55 +0000 (09:50 -0700)
QoSSettings and cinder_utils have a runtime cyclical
dependency. This patch reduces this dependency and
deprecates the QoSSettings class.

JIRA: SNAPS-222

Change-Id: I6385717b78db413c496b15b8c4b76ffabe9797c1
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
docs/how-to-use/LibraryUsage.rst
docs/how-to-use/UnitTests.rst
examples/launch.py
snaps/config/qos.py [new file with mode: 0644]
snaps/config/tests/qos_tests.py [new file with mode: 0644]
snaps/openstack/create_qos.py
snaps/openstack/tests/create_qos_tests.py
snaps/openstack/tests/create_volume_type_tests.py
snaps/openstack/utils/tests/cinder_utils_tests.py
snaps/openstack/utils/tests/settings_utils_tests.py
snaps/test_suite_builder.py

index 7c82387..d7f3177 100644 (file)
@@ -425,7 +425,7 @@ Create QoS Spec
 
 -  Volume Type - snaps.openstack.create\_qos.OpenStackQoS
 
-   -  snaps.openstack.create\_qos.QoSSettings
+   -  snaps.openstack.qos.QoSConfig
 
       -  name - the volume type's name (required)
       -  consumer - the qos's consumer type of the enum type Consumer (required)
@@ -433,9 +433,10 @@ Create QoS Spec
 
 .. code:: python
 
-    from snaps.openstack.create_qos import QoSSettings, OpenStackQoS
+    from snaps.openstack.qos import QoSConfig
+    from snaps.openstack.create_qos import OpenStackQoS
 
-    qos_settings = QoSSettings(name='stack-name', consumer=Consumer.front-end)
+    qos_settings = QoSConfig(name='stack-name', consumer=Consumer.front-end)
     qos_creator = OpenStackQoS(os_creds, vol_type_settings)
     qos_creator.create()
 
index f46d9ee..bc5515b 100644 (file)
@@ -294,11 +294,17 @@ VolumeTypeEncryptionObjectTests
 Ensures that all required members are included when constructing a
 VolumeTypeEncryption domain object (for Cinder)
 
+QoSConfigUnitTests
+------------------
+
+Ensures that all required members are included when constructing a
+QoSConfig object
+
 QoSSettingsUnitTests
 --------------------
 
 Ensures that all required members are included when constructing a
-QoSSettings object
+deprecated QoSSettings object
 
 QoSSpecDomainObjectTests
 ------------------------
index dfdf57f..b0e87ba 100644 (file)
@@ -30,6 +30,7 @@ from snaps.config.flavor import FlavorConfig
 from snaps.config.image import ImageConfig
 from snaps.config.keypair import KeypairConfig
 from snaps.config.project import ProjectConfig
+from snaps.config.qos import QoSConfig
 from snaps.config.router import RouterConfig
 from snaps.config.user import UserConfig
 from snaps.openstack.create_flavor import OpenStackFlavor
@@ -39,7 +40,7 @@ from snaps.openstack.create_keypairs import OpenStackKeypair
 from snaps.openstack.create_network import (
     PortSettings, NetworkSettings, OpenStackNetwork)
 from snaps.openstack.create_project import OpenStackProject
-from snaps.openstack.create_qos import QoSSettings, OpenStackQoS
+from snaps.openstack.create_qos import OpenStackQoS
 from snaps.openstack.create_router import OpenStackRouter
 from snaps.openstack.create_security_group import (
     OpenStackSecurityGroup, SecurityGroupSettings)
@@ -646,7 +647,7 @@ def main(arguments):
 
                 # Create QoS specs
                 qos_dict = __create_instances(
-                    os_creds_dict, OpenStackQoS, QoSSettings,
+                    os_creds_dict, OpenStackQoS, QoSConfig,
                     os_config.get('qos_specs'), 'qos_spec', clean, users_dict)
                 creators.append(qos_dict)
 
diff --git a/snaps/config/qos.py b/snaps/config/qos.py
new file mode 100644 (file)
index 0000000..e507d2d
--- /dev/null
@@ -0,0 +1,92 @@
+# Copyright (c) 2017 Cable Television Laboratories, Inc. ("CableLabs")
+#                    and others.  All rights reserved.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at:
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+import enum
+
+
+class Consumer(enum.Enum):
+    """
+    QoS Specification consumer types
+    """
+    front_end = 'front-end'
+    back_end = 'back-end'
+    both = 'both'
+
+
+class QoSConfig(object):
+    def __init__(self, **kwargs):
+        """
+        Constructor
+        :param name: the qos's name (required)
+        :param consumer: the qos's consumer type of the enum type Consumer
+                         (required)
+        :param specs: dict of key/values
+        """
+
+        self.name = kwargs.get('name')
+
+        if kwargs.get('consumer'):
+            self.consumer = map_consumer(kwargs['consumer'])
+        else:
+            self.consumer = None
+
+        self.specs = kwargs.get('specs')
+        if not self.specs:
+            self.specs = dict()
+
+        if not self.name or not self.consumer:
+            raise QoSConfigError(
+                "The attributes name and consumer are required")
+
+
+def map_consumer(consumer):
+    """
+    Takes a the protocol value maps it to the Consumer enum. When None return
+    None
+    :param consumer: the value to map to the Enum
+    :return: the Protocol enum object
+    :raise: Exception if value is invalid
+    """
+    if not consumer:
+        return None
+    elif isinstance(consumer, Consumer):
+        return consumer
+    elif isinstance(consumer, str):
+        proto_str = str(consumer)
+        if proto_str == 'front-end':
+            return Consumer.front_end
+        elif proto_str == 'back-end':
+            return Consumer.back_end
+        elif proto_str == 'both':
+            return Consumer.both
+        else:
+            raise QoSConfigError('Invalid Consumer - ' + proto_str)
+    else:
+        if consumer.value == 'front-end':
+            return Consumer.front_end
+        elif consumer.value == 'back-end':
+            return Consumer.back_end
+        elif consumer.value == 'both':
+            return Consumer.both
+        else:
+            raise QoSConfigError('Invalid Consumer - ' + consumer.value)
+
+
+class QoSConfigError(Exception):
+    """
+    Exception to be thrown when an qos settings are incorrect
+    """
+
+    def __init__(self, message):
+        Exception.__init__(self, message)
diff --git a/snaps/config/tests/qos_tests.py b/snaps/config/tests/qos_tests.py
new file mode 100644 (file)
index 0000000..7314c5b
--- /dev/null
@@ -0,0 +1,91 @@
+# Copyright (c) 2017 Cable Television Laboratories, Inc. ("CableLabs")
+#                    and others.  All rights reserved.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at:
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+import unittest
+
+from snaps.config.qos import QoSConfig, QoSConfigError, Consumer
+
+
+class QoSConfigUnitTests(unittest.TestCase):
+    """
+    Tests the construction of the QoSConfig class
+    """
+
+    def test_no_params(self):
+        with self.assertRaises(QoSConfigError):
+            QoSConfig()
+
+    def test_empty_config(self):
+        with self.assertRaises(QoSConfigError):
+            QoSConfig(**dict())
+
+    def test_name_only(self):
+        with self.assertRaises(QoSConfigError):
+            QoSConfig(name='foo')
+
+    def test_config_with_name_only(self):
+        with self.assertRaises(QoSConfigError):
+            QoSConfig(**{'name': 'foo'})
+
+    def test_invalid_consumer(self):
+        with self.assertRaises(QoSConfigError):
+            QoSConfig(name='foo', consumer='bar')
+
+    def test_config_with_invalid_consumer(self):
+        with self.assertRaises(QoSConfigError):
+            QoSConfig(**{'name': 'foo', 'consumer': 'bar'})
+
+    def test_name_consumer(self):
+        settings = QoSConfig(name='foo', consumer=Consumer.front_end)
+
+        self.assertEqual('foo', settings.name)
+        self.assertEqual(Consumer.front_end, settings.consumer)
+        self.assertEqual(dict(), settings.specs)
+
+    def test_name_consumer_front_end_strings(self):
+        settings = QoSConfig(name='foo', consumer='front-end')
+
+        self.assertEqual('foo', settings.name)
+        self.assertEqual(Consumer.front_end, settings.consumer)
+        self.assertEqual(dict(), settings.specs)
+
+    def test_name_consumer_back_end_strings(self):
+        settings = QoSConfig(name='foo', consumer='back-end')
+
+        self.assertEqual('foo', settings.name)
+        self.assertEqual(Consumer.back_end, settings.consumer)
+        self.assertEqual(dict(), settings.specs)
+
+    def test_name_consumer_both_strings(self):
+        settings = QoSConfig(name='foo', consumer='both')
+
+        self.assertEqual('foo', settings.name)
+        self.assertEqual(Consumer.both, settings.consumer)
+        self.assertEqual(dict(), settings.specs)
+
+    def test_all(self):
+        specs = {'spec1': 'val1', 'spec2': 'val2'}
+        settings = QoSConfig(name='foo', consumer=Consumer.both, specs=specs)
+
+        self.assertEqual('foo', settings.name)
+        self.assertEqual(Consumer.both, settings.consumer)
+        self.assertEqual(specs, settings.specs)
+
+    def test_config_all(self):
+        settings = QoSConfig(
+            **{'name': 'foo', 'consumer': 'both', 'specs': {'spec1': 'val1'}})
+
+        self.assertEqual('foo', settings.name)
+        self.assertEqual(Consumer.both, settings.consumer)
+        self.assertEqual({'spec1': 'val1'}, settings.specs)
index 96f8c9b..44e35a3 100644 (file)
@@ -18,6 +18,7 @@ import logging
 import enum
 from cinderclient.exceptions import NotFound
 
+from snaps.config.qos import QoSConfig
 from snaps.openstack.openstack_creator import OpenStackVolumeObject
 from snaps.openstack.utils import cinder_utils
 
@@ -101,69 +102,25 @@ class OpenStackQoS(OpenStackVolumeObject):
 class Consumer(enum.Enum):
     """
     QoS Specification consumer types
+    deprecated - use snaps.config.qos.Consumer
     """
     front_end = 'front-end'
     back_end = 'back-end'
     both = 'both'
 
 
-class QoSSettings:
-    def __init__(self, **kwargs):
-        """
-        Constructor
-        :param name: the qos's name (required)
-        :param consumer: the qos's consumer type of the enum type Consumer
-                         (required)
-        :param specs: dict of key/values
-        """
-
-        self.name = kwargs.get('name')
-
-        if kwargs.get('consumer'):
-            self.consumer = map_consumer(kwargs['consumer'])
-        else:
-            self.consumer = None
-
-        self.specs = kwargs.get('specs')
-        if not self.specs:
-            self.specs = dict()
-
-        if not self.name or not self.consumer:
-            raise QoSSettingsError(
-                "The attributes name and consumer are required")
-
-
-def map_consumer(consumer):
-    """
-    Takes a the protocol value maps it to the Consumer enum. When None return
-    None
-    :param consumer: the value to map to the Enum
-    :return: the Protocol enum object
-    :raise: Exception if value is invalid
-    """
-    if not consumer:
-        return None
-    elif isinstance(consumer, Consumer):
-        return consumer
-    else:
-        proto_str = str(consumer)
-        if proto_str == 'front-end':
-            return Consumer.front_end
-        elif proto_str == 'back-end':
-            return Consumer.back_end
-        elif proto_str == 'both':
-            return Consumer.both
-        else:
-            raise QoSSettingsError('Invalid Consumer - ' + proto_str)
-
-
-class QoSSettingsError(Exception):
+class QoSSettings(QoSConfig):
     """
-    Exception to be thrown when an qos settings are incorrect
+    Class to hold the configuration settings required for creating OpenStack
+    QoS objects
+    deprecated
     """
 
-    def __init__(self, message):
-        Exception.__init__(self, message)
+    def __init__(self, **kwargs):
+        from warnings import warn
+        warn('Use snaps.config.qos.QoSConfig instead',
+             DeprecationWarning)
+        super(self.__class__, self).__init__(**kwargs)
 
 
 class QoSCreationError(Exception):
index 6c0a056..68737f8 100644 (file)
@@ -12,6 +12,8 @@
 # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 # See the License for the specific language governing permissions and
 # limitations under the License.
+from snaps.config.qos import Consumer, QoSConfigError, QoSConfig
+from snaps.openstack.create_qos import Consumer as Consumer_old
 
 try:
     from urllib.request import URLError
@@ -23,8 +25,7 @@ import unittest
 import uuid
 
 from snaps.openstack import create_qos
-from snaps.openstack.create_qos import (QoSSettings, QoSSettingsError,
-                                        Consumer)
+from snaps.openstack.create_qos import QoSSettings
 from snaps.openstack.tests.os_source_file_test import OSIntegrationTestCase
 from snaps.openstack.utils import cinder_utils
 
@@ -39,64 +40,64 @@ class QoSSettingsUnitTests(unittest.TestCase):
     """
 
     def test_no_params(self):
-        with self.assertRaises(QoSSettingsError):
+        with self.assertRaises(QoSConfigError):
             QoSSettings()
 
     def test_empty_config(self):
-        with self.assertRaises(QoSSettingsError):
+        with self.assertRaises(QoSConfigError):
             QoSSettings(**dict())
 
     def test_name_only(self):
-        with self.assertRaises(QoSSettingsError):
+        with self.assertRaises(QoSConfigError):
             QoSSettings(name='foo')
 
     def test_config_with_name_only(self):
-        with self.assertRaises(QoSSettingsError):
+        with self.assertRaises(QoSConfigError):
             QoSSettings(**{'name': 'foo'})
 
     def test_invalid_consumer(self):
-        with self.assertRaises(QoSSettingsError):
+        with self.assertRaises(QoSConfigError):
             QoSSettings(name='foo', consumer='bar')
 
     def test_config_with_invalid_consumer(self):
-        with self.assertRaises(QoSSettingsError):
+        with self.assertRaises(QoSConfigError):
             QoSSettings(**{'name': 'foo', 'consumer': 'bar'})
 
     def test_name_consumer(self):
-        settings = QoSSettings(name='foo', consumer=Consumer.front_end)
+        settings = QoSSettings(name='foo', consumer=Consumer_old.front_end)
 
         self.assertEqual('foo', settings.name)
-        self.assertEqual(Consumer.front_end, settings.consumer)
+        self.assertEqual(Consumer_old.front_end.value, settings.consumer.value)
         self.assertEqual(dict(), settings.specs)
 
     def test_name_consumer_front_end_strings(self):
         settings = QoSSettings(name='foo', consumer='front-end')
 
         self.assertEqual('foo', settings.name)
-        self.assertEqual(Consumer.front_end, settings.consumer)
+        self.assertEqual(Consumer_old.front_end.value, settings.consumer.value)
         self.assertEqual(dict(), settings.specs)
 
     def test_name_consumer_back_end_strings(self):
         settings = QoSSettings(name='foo', consumer='back-end')
 
         self.assertEqual('foo', settings.name)
-        self.assertEqual(Consumer.back_end, settings.consumer)
+        self.assertEqual(Consumer_old.back_end.value, settings.consumer.value)
         self.assertEqual(dict(), settings.specs)
 
     def test_name_consumer_both_strings(self):
         settings = QoSSettings(name='foo', consumer='both')
 
         self.assertEqual('foo', settings.name)
-        self.assertEqual(Consumer.both, settings.consumer)
+        self.assertEqual(Consumer_old.both.value, settings.consumer.value)
         self.assertEqual(dict(), settings.specs)
 
     def test_all(self):
         specs = {'spec1': 'val1', 'spec2': 'val2'}
-        settings = QoSSettings(name='foo', consumer=Consumer.both,
+        settings = QoSSettings(name='foo', consumer=Consumer_old.both,
                                specs=specs)
 
         self.assertEqual('foo', settings.name)
-        self.assertEqual(Consumer.both, settings.consumer)
+        self.assertEqual(Consumer_old.both.value, settings.consumer.value)
         self.assertEqual(specs, settings.specs)
 
     def test_config_all(self):
@@ -121,7 +122,7 @@ class CreateQoSTests(OSIntegrationTestCase):
         super(self.__class__, self).__start__()
 
         guid = uuid.uuid4()
-        self.qos_settings = QoSSettings(
+        self.qos_settings = QoSConfig(
             name=self.__class__.__name__ + '-' + str(guid),
             consumer=Consumer.both)
 
index 93e9351..8bff91f 100644 (file)
@@ -12,7 +12,8 @@
 # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 # See the License for the specific language governing permissions and
 # limitations under the License.
-from snaps.openstack.create_qos import QoSSettings, Consumer, OpenStackQoS
+from snaps.config.qos import QoSConfig, Consumer
+from snaps.openstack.create_qos import OpenStackQoS
 
 try:
     from urllib.request import URLError
@@ -23,10 +24,9 @@ import logging
 import unittest
 import uuid
 
-from snaps.openstack import create_volume_type
 from snaps.openstack.create_volume_type import (
     VolumeTypeSettings, VolumeTypeSettingsError, VolumeTypeEncryptionSettings,
-    ControlLocation)
+    ControlLocation, OpenStackVolumeType)
 from snaps.openstack.tests.os_source_file_test import OSIntegrationTestCase
 from snaps.openstack.utils import cinder_utils
 
@@ -109,7 +109,7 @@ class VolumeTypeSettingsUnitTests(unittest.TestCase):
 
 class CreateSimpleVolumeTypeSuccessTests(OSIntegrationTestCase):
     """
-    Test for the OpenStackVolumeType class defined in create_volume_type.py
+    Test for the OpenStackVolumeType class defined in py
     without any QoS Specs or Encryption
     """
 
@@ -141,7 +141,7 @@ class CreateSimpleVolumeTypeSuccessTests(OSIntegrationTestCase):
         Tests the creation of an OpenStack volume.
         """
         # Create VolumeType
-        self.volume_type_creator = create_volume_type.OpenStackVolumeType(
+        self.volume_type_creator = OpenStackVolumeType(
             self.os_creds, self.volume_type_settings)
         created_volume_type = self.volume_type_creator.create()
         self.assertIsNotNone(created_volume_type)
@@ -163,7 +163,7 @@ class CreateSimpleVolumeTypeSuccessTests(OSIntegrationTestCase):
         clean() does not raise an Exception.
         """
         # Create VolumeType
-        self.volume_type_creator = create_volume_type.OpenStackVolumeType(
+        self.volume_type_creator = OpenStackVolumeType(
             self.os_creds, self.volume_type_settings)
         created_volume_type = self.volume_type_creator.create()
         self.assertIsNotNone(created_volume_type)
@@ -189,7 +189,7 @@ class CreateSimpleVolumeTypeSuccessTests(OSIntegrationTestCase):
         Tests the creation of an OpenStack volume_type when one already exists.
         """
         # Create VolumeType
-        self.volume_type_creator = create_volume_type.OpenStackVolumeType(
+        self.volume_type_creator = OpenStackVolumeType(
             self.os_creds, self.volume_type_settings)
         volume_type1 = self.volume_type_creator.create()
 
@@ -198,7 +198,7 @@ class CreateSimpleVolumeTypeSuccessTests(OSIntegrationTestCase):
         self.assertEqual(volume_type1, retrieved_volume_type)
 
         # Should be retrieving the instance data
-        os_volume_type_2 = create_volume_type.OpenStackVolumeType(
+        os_volume_type_2 = OpenStackVolumeType(
             self.os_creds, self.volume_type_settings)
         volume_type2 = os_volume_type_2.create()
         self.assertEqual(volume_type2, volume_type2)
@@ -220,7 +220,7 @@ class CreateVolumeTypeComplexTests(OSIntegrationTestCase):
         self.volume_type_name = guid + '-vol_type'
         self.volume_type_creator = None
 
-        qos_settings = QoSSettings(
+        qos_settings = QoSConfig(
             name=guid + '-qos-spec', consumer=Consumer.both)
         self.qos_creator = OpenStackQoS(self.os_creds, qos_settings)
         self.qos_creator.create()
@@ -238,7 +238,7 @@ class CreateVolumeTypeComplexTests(OSIntegrationTestCase):
         """
         Creates a Volume Type object with an associated QoS Spec
         """
-        self.volume_type_creator = create_volume_type.OpenStackVolumeType(
+        self.volume_type_creator = OpenStackVolumeType(
             self.os_creds,
             VolumeTypeSettings(
                 name=self.volume_type_name,
@@ -267,7 +267,7 @@ class CreateVolumeTypeComplexTests(OSIntegrationTestCase):
         encryption_settings = VolumeTypeEncryptionSettings(
             name='foo', provider_class='bar',
             control_location=ControlLocation.back_end)
-        self.volume_type_creator = create_volume_type.OpenStackVolumeType(
+        self.volume_type_creator = OpenStackVolumeType(
             self.os_creds,
             VolumeTypeSettings(
                 name=self.volume_type_name,
@@ -296,7 +296,7 @@ class CreateVolumeTypeComplexTests(OSIntegrationTestCase):
         encryption_settings = VolumeTypeEncryptionSettings(
             name='foo', provider_class='bar',
             control_location=ControlLocation.back_end)
-        self.volume_type_creator = create_volume_type.OpenStackVolumeType(
+        self.volume_type_creator = OpenStackVolumeType(
             self.os_creds,
             VolumeTypeSettings(
                 name=self.volume_type_name,
index 6fd92e3..547a6f9 100644 (file)
@@ -18,8 +18,8 @@ import uuid
 import time
 from cinderclient.exceptions import NotFound, BadRequest
 
+from snaps.config.qos import Consumer, QoSConfig
 from snaps.openstack import create_volume
-from snaps.openstack.create_qos import QoSSettings, Consumer
 from snaps.openstack.create_volume import VolumeSettings
 from snaps.openstack.create_volume_type import (
     VolumeTypeSettings, VolumeTypeEncryptionSettings, ControlLocation)
@@ -196,8 +196,8 @@ class CinderUtilsQoSTests(OSComponentTestCase):
         """
         Tests the cinder_utils.create_qos()
         """
-        qos_settings = QoSSettings(name=self.qos_name, specs=self.specs,
-                                   consumer=Consumer.both)
+        qos_settings = QoSConfig(
+            name=self.qos_name, specs=self.specs, consumer=Consumer.both)
         self.qos = cinder_utils.create_qos(self.cinder, qos_settings)
         self.assertIsNotNone(self.qos)
 
@@ -213,8 +213,8 @@ class CinderUtilsQoSTests(OSComponentTestCase):
         """
         Tests the cinder_utils.create_qos()
         """
-        qos_settings = QoSSettings(name=self.qos_name, specs=self.specs,
-                                   consumer=Consumer.front_end)
+        qos_settings = QoSConfig(
+            name=self.qos_name, specs=self.specs, consumer=Consumer.front_end)
         self.qos = cinder_utils.create_qos(self.cinder, qos_settings)
         self.assertIsNotNone(self.qos)
 
@@ -230,8 +230,8 @@ class CinderUtilsQoSTests(OSComponentTestCase):
         """
         Tests the cinder_utils.create_qos()
         """
-        qos_settings = QoSSettings(name=self.qos_name, specs=self.specs,
-                                   consumer=Consumer.back_end)
+        qos_settings = QoSConfig(
+            name=self.qos_name, specs=self.specs, consumer=Consumer.back_end)
         self.qos = cinder_utils.create_qos(self.cinder, qos_settings)
         self.assertIsNotNone(self.qos)
 
@@ -247,7 +247,7 @@ class CinderUtilsQoSTests(OSComponentTestCase):
         """
         Tests the cinder_utils.create_qos()
         """
-        qos_settings = QoSSettings(name=self.qos_name, consumer=Consumer.both)
+        qos_settings = QoSConfig(name=self.qos_name, consumer=Consumer.both)
         self.qos = cinder_utils.create_qos(self.cinder, qos_settings)
         self.assertIsNotNone(self.qos)
         self.assertEqual(self.qos_name, self.qos.name)
@@ -463,8 +463,8 @@ class CinderUtilsVolumeTypeCompleteTests(OSComponentTestCase):
         self.vol_type_name = self.__class__.__name__ + '-' + str(guid)
         self.specs = {'foo': 'bar'}
         self.cinder = cinder_utils.cinder_client(self.os_creds)
-        qos_settings = QoSSettings(name=self.qos_name, specs=self.specs,
-                                   consumer=Consumer.both)
+        qos_settings = QoSConfig(
+            name=self.qos_name, specs=self.specs, consumer=Consumer.both)
         self.qos = cinder_utils.create_qos(self.cinder, qos_settings)
         self.volume_type = None
 
index 3fba4fc..10b5114 100644 (file)
@@ -20,6 +20,7 @@ import uuid
 
 from snaps.config.flavor import FlavorConfig
 from snaps.config.keypair import KeypairConfig
+from snaps.config.qos import Consumer
 from snaps.domain.flavor import Flavor
 from snaps.domain.volume import (
     Volume, VolumeType, VolumeTypeEncryption, QoSSpec)
@@ -28,7 +29,6 @@ from snaps.openstack import (
     create_keypairs, create_instance)
 from snaps.openstack.create_network import (
     NetworkSettings, OpenStackNetwork, SubnetSettings)
-from snaps.openstack.create_qos import Consumer
 from snaps.openstack.create_security_group import (
     SecurityGroupRuleSettings,  Direction, Protocol, OpenStackSecurityGroup,
     SecurityGroupSettings)
index ffb27a8..1212019 100644 (file)
@@ -16,6 +16,7 @@
 import logging
 import unittest
 
+from snaps.config.tests.qos_tests import QoSConfigUnitTests
 from snaps.config.tests.stack_tests import StackConfigUnitTests
 from snaps.config.tests.router_tests import RouterConfigUnitTests
 from snaps.config.tests.user_tests import UserConfigUnitTests
@@ -221,6 +222,8 @@ def add_unit_tests(suite):
         VmInstDomainObjectTests))
     suite.addTest(unittest.TestLoader().loadTestsFromTestCase(
         FloatingIpDomainObjectTests))
+    suite.addTest(unittest.TestLoader().loadTestsFromTestCase(
+        QoSConfigUnitTests))
     suite.addTest(unittest.TestLoader().loadTestsFromTestCase(
         QoSSettingsUnitTests))
     suite.addTest(unittest.TestLoader().loadTestsFromTestCase(