diff options
author | spisarski <s.pisarski@cablelabs.com> | 2018-03-07 13:01:37 -0700 |
---|---|---|
committer | spisarski <s.pisarski@cablelabs.com> | 2018-03-08 14:02:05 -0700 |
commit | f77540b440d3e5224eb1648339dc8c945f29bbe2 (patch) | |
tree | 84856cebdfb36f9c758d43bb127d1b91b88cc7d6 /snaps/config/tests | |
parent | e4563aec1eb8f98e5e42fe513918432eddadcd37 (diff) |
Ensure the project for volumes are handled properly.
Although this patch has added the project_name member to
VolumeConfig, it does not work properly due to a cinder bug
https://bugs.launchpad.net/cinder/+bug/1641982. However, the
code will now allow for volumes with the same name to be
created on the same pod just as long as they do not exist
within the same project/tenant.
JIRA: SNAPS-270
Change-Id: Ia6f98e9b66793a69c0205d2d8e38bb3d03f3333e
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
Diffstat (limited to 'snaps/config/tests')
-rw-r--r-- | snaps/config/tests/volume_tests.py | 18 |
1 files changed, 13 insertions, 5 deletions
diff --git a/snaps/config/tests/volume_tests.py b/snaps/config/tests/volume_tests.py index b4b54bd..21adffa 100644 --- a/snaps/config/tests/volume_tests.py +++ b/snaps/config/tests/volume_tests.py @@ -33,6 +33,7 @@ class VolumeConfigUnitTests(unittest.TestCase): def test_name_only(self): settings = VolumeConfig(name='foo') self.assertEqual('foo', settings.name) + self.assertIsNone(settings.project_name) self.assertIsNone(settings.description) self.assertEquals(1, settings.size) self.assertIsNone(settings.image_name) @@ -43,6 +44,7 @@ class VolumeConfigUnitTests(unittest.TestCase): def test_config_with_name_only(self): settings = VolumeConfig(**{'name': 'foo'}) self.assertEqual('foo', settings.name) + self.assertIsNone(settings.project_name) self.assertIsNone(settings.description) self.assertEquals(1, settings.size) self.assertIsNone(settings.image_name) @@ -52,10 +54,12 @@ class VolumeConfigUnitTests(unittest.TestCase): def test_all_strings(self): settings = VolumeConfig( - name='foo', description='desc', size='2', image_name='image', - type_name='type', availability_zone='zone1', multi_attach='true') + name='foo', project_name='proj-foo', description='desc', size='2', + image_name='image', type_name='type', availability_zone='zone1', + multi_attach='true') self.assertEqual('foo', settings.name) + self.assertEqual('proj-foo', settings.project_name) self.assertEqual('desc', settings.description) self.assertEqual(2, settings.size) self.assertEqual('image', settings.image_name) @@ -65,10 +69,12 @@ class VolumeConfigUnitTests(unittest.TestCase): def test_all_correct_type(self): settings = VolumeConfig( - name='foo', description='desc', size=2, image_name='image', - type_name='bar', availability_zone='zone1', multi_attach=True) + name='foo', project_name='proj-foo', description='desc', size=2, + image_name='image', type_name='bar', availability_zone='zone1', + multi_attach=True) self.assertEqual('foo', settings.name) + self.assertEqual('proj-foo', settings.project_name) self.assertEqual('desc', settings.description) self.assertEqual(2, settings.size) self.assertEqual('image', settings.image_name) @@ -78,11 +84,13 @@ class VolumeConfigUnitTests(unittest.TestCase): def test_config_all(self): settings = VolumeConfig( - **{'name': 'foo', 'description': 'desc', 'size': '2', + **{'name': 'foo', 'project_name': 'proj-foo', + 'description': 'desc', 'size': '2', 'image_name': 'foo', 'type_name': 'bar', 'availability_zone': 'zone1', 'multi_attach': 'true'}) self.assertEqual('foo', settings.name) + self.assertEqual('proj-foo', settings.project_name) self.assertEqual('desc', settings.description) self.assertEqual(2, settings.size) self.assertEqual('foo', settings.image_name) |