openstack
/
glance_store
Code Issues Proposed changes

Change location metadata key 'backend' to 'store' 

As a part of vocabulary correction, changed the location
metadata key name from 'backend' to 'store'. Modified
corresponding tests as well.

bp:multi-store-vocabulary-correction
Change-Id: Ida27628c54607b43e76389029f1f6f78ccc49f93
This commit is contained in:
Abhishek Kekane 2019-05-13 10:06:39 +00:00 committed by Brian Rosmaita
parent cd039c301e
commit 54744cc3a3
10 changed files with 36 additions and 36 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
glance_store/_drivers/cinder.py
View File

@ -769,7 +769,7 @@ class Store(glance_store.driver.Store):
image_metadata = {}
if self.backend_group:
image_metadata['backend'] = u"%s" % self.backend_group
image_metadata['store'] = u"%s" % self.backend_group
return ('cinder://%s' % volume.id,
bytes_written,

2
glance_store/_drivers/filesystem.py
View File

@ -758,7 +758,7 @@ class Store(glance_store.driver.Store):
# Add store backend information to location metadata
if self.backend_group:
metadata['backend'] = u"%s" % self.backend_group
metadata['store'] = u"%s" % self.backend_group
return ('file://%s' % filepath,
bytes_written,

2
glance_store/_drivers/rbd.py
View File

@ -571,7 +571,7 @@ class Store(driver.Store):
# Add store backend information to location metadata
metadata = {}
if self.backend_group:
metadata['backend'] = u"%s" % self.backend_group
metadata['store'] = u"%s" % self.backend_group
return (loc.get_uri(),
image_size,

2
glance_store/_drivers/swift/store.py
View File

@ -1060,7 +1060,7 @@ class BaseStore(driver.Store):
metadata = {}
if self.backend_group:
metadata['backend'] = u"%s" % self.backend_group
metadata['store'] = u"%s" % self.backend_group
return (location.get_uri(credentials_included=include_creds),
image_size, obj_etag, os_hash_value.hexdigest(),

2
glance_store/_drivers/vmware_datastore.py
View File

@ -644,7 +644,7 @@ class Store(glance_store.Store):
metadata = {}
if self.backend_group:
metadata['backend'] = u"%s" % self.backend_group
metadata['store'] = u"%s" % self.backend_group
return (loc.get_uri(),
image_file.size,

2
glance_store/tests/unit/test_multistore_cinder.py
View File

@ -345,7 +345,7 @@ class TestMultiCinderStore(base.MultiStoreBaseTest,
'glance_image_id': expected_image_id,
'image_size': str(expected_size)},
volume_type='some_type')
self.assertEqual(backend, metadata["backend"])
self.assertEqual(backend, metadata["store"])
def test_cinder_add(self):
fake_volume = mock.MagicMock(id=str(uuid.uuid4()),

30
glance_store/tests/unit/test_multistore_filesystem.py
View File

@ -106,7 +106,7 @@ class TestMultiStore(base.MultiStoreBaseTest,
loc, size, checksum, metadata = self.store.add(
image_id, image_file, len(file_contents))
# Check metadata contains 'file1' as a backend
self.assertEqual(u"file1", metadata['backend'])
self.assertEqual(u"file1", metadata['store'])
# Now read it back...
uri = "file:///%s/%s" % (self.test_dir, image_id)
@ -136,7 +136,7 @@ class TestMultiStore(base.MultiStoreBaseTest,
image_file,
len(file_contents))
# Check metadata contains 'file1' as a backend
self.assertEqual(u"file1", metadata['backend'])
self.assertEqual(u"file1", metadata['store'])
# Now read it back...
uri = "file:///%s/%s" % (self.test_dir, image_id)
@ -197,7 +197,7 @@ class TestMultiStore(base.MultiStoreBaseTest,
self.assertEqual(expected_location, loc)
self.assertEqual(expected_file_size, size)
self.assertEqual(expected_checksum, checksum)
self.assertEqual(u"file1", metadata['backend'])
self.assertEqual(u"file1", metadata['store'])
uri = "file:///%s/%s" % (self.test_dir, expected_image_id)
loc = location.get_location_from_uri_and_backend(
@ -237,7 +237,7 @@ class TestMultiStore(base.MultiStoreBaseTest,
self.assertEqual(expected_location, loc)
self.assertEqual(expected_file_size, size)
self.assertEqual(expected_checksum, checksum)
self.assertEqual(u"file2", metadata['backend'])
self.assertEqual(u"file2", metadata['store'])
uri = "file:///%s/%s" % (self.test_dir, expected_image_id)
loc = location.get_location_from_uri_and_backend(
@ -257,20 +257,20 @@ class TestMultiStore(base.MultiStoreBaseTest,
in_metadata = [{'id': 'abcdefg',
'mountpoint': '/xyz/images'}]
location, size, checksum, metadata = self._store_image(in_metadata)
self.assertEqual({'backend': u'file1'}, metadata)
self.assertEqual({'store': u'file1'}, metadata)
def test_add_check_metadata_list_with_invalid_mountpoint_locations(self):
in_metadata = [{'id': 'abcdefg', 'mountpoint': '/xyz/images'},
{'id': 'xyz1234', 'mountpoint': '/pqr/images'}]
location, size, checksum, metadata = self._store_image(in_metadata)
self.assertEqual({'backend': u'file1'}, metadata)
self.assertEqual({'store': u'file1'}, metadata)
def test_add_check_metadata_list_with_valid_mountpoint_locations(self):
in_metadata = [{'id': 'abcdefg', 'mountpoint': '/tmp'},
{'id': 'xyz1234', 'mountpoint': '/xyz'}]
location, size, checksum, metadata = self._store_image(in_metadata)
self.assertEqual(in_metadata[0], metadata)
self.assertEqual(u"file1", metadata["backend"])
self.assertEqual(u"file1", metadata["store"])
def test_add_check_metadata_bad_nosuch_file(self):
expected_image_id = str(uuid.uuid4())
@ -287,7 +287,7 @@ class TestMultiStore(base.MultiStoreBaseTest,
image_file,
expected_file_size)
self.assertEqual({'backend': u'file1'}, metadata)
self.assertEqual({'store': u'file1'}, metadata)
def test_add_already_existing(self):
"""
@ -303,7 +303,7 @@ class TestMultiStore(base.MultiStoreBaseTest,
location, size, checksum, metadata = self.store.add(image_id,
image_file,
file_size)
self.assertEqual(u"file1", metadata["backend"])
self.assertEqual(u"file1", metadata["store"])
image_file = six.BytesIO(b"nevergonnamakeit")
self.assertRaises(exceptions.Duplicate,
@ -388,7 +388,7 @@ class TestMultiStore(base.MultiStoreBaseTest,
loc, size, checksum, metadata = self.store.add(image_id,
image_file,
file_size)
self.assertEqual(u"file1", metadata["backend"])
self.assertEqual(u"file1", metadata["store"])
# Now check that we can delete it
uri = "file:///%s/%s" % (self.test_dir, image_id)
@ -417,7 +417,7 @@ class TestMultiStore(base.MultiStoreBaseTest,
loc, size, checksum, metadata = self.store.add(image_id,
image_file,
file_size)
self.assertEqual(u"file1", metadata["backend"])
self.assertEqual(u"file1", metadata["store"])
uri = "file:///%s/%s" % (self.test_dir, image_id)
loc = location.get_location_from_uri_and_backend(uri, "file1",
@ -587,7 +587,7 @@ class TestMultiStore(base.MultiStoreBaseTest,
loc, size, checksum, metadata = self.store.add(expected_image_id,
image_file,
expected_file_size)
self.assertEqual(u"file1", metadata["backend"])
self.assertEqual(u"file1", metadata["store"])
self.assertEqual(expected_location, loc)
self.assertEqual(expected_file_size, size)
@ -634,7 +634,7 @@ class TestMultiStore(base.MultiStoreBaseTest,
loc, size, checksum, metadata = self.store.add(expected_image_id,
image_file,
expected_file_size)
self.assertEqual(u"file1", metadata["backend"])
self.assertEqual(u"file1", metadata["store"])
self.assertEqual(expected_location, loc)
self.assertEqual(expected_file_size, size)
@ -745,7 +745,7 @@ class TestMultiStore(base.MultiStoreBaseTest,
location, size, checksum, metadata = self.store.add(expected_image_id,
image_file,
expected_file_size)
self.assertEqual(u"file1", metadata["backend"])
self.assertEqual(u"file1", metadata["store"])
self.assertEqual(expected_location, location)
self.assertEqual(expected_file_size, size)
@ -789,7 +789,7 @@ class TestMultiStore(base.MultiStoreBaseTest,
location, size, checksum, metadata = self.store.add(expected_image_id,
image_file,
expected_file_size)
self.assertEqual(u"file1", metadata["backend"])
self.assertEqual(u"file1", metadata["store"])
self.assertEqual(expected_location, location)
self.assertEqual(expected_file_size, size)

4
glance_store/tests/unit/test_multistore_rbd.py
View File

@ -222,7 +222,7 @@ class TestMultiStore(base.MultiStoreBaseTest,
self.assertTrue(resize.called)
self.assertTrue(write.called)
self.assertEqual(ret[1], self.data_len)
self.assertEqual("ceph1", ret[3]['backend'])
self.assertEqual("ceph1", ret[3]['store'])
def test_add_w_image_size_zero_to_different_backend(self):
"""Assert that correct size is returned even though 0 was provided."""
@ -246,7 +246,7 @@ class TestMultiStore(base.MultiStoreBaseTest,
self.assertTrue(resize.called)
self.assertTrue(write.called)
self.assertEqual(ret[1], self.data_len)
self.assertEqual("ceph2", ret[3]['backend'])
self.assertEqual("ceph2", ret[3]['store'])
@mock.patch.object(MockRBD.Image, '__enter__')
@mock.patch.object(rbd_store.Store, '_create_image')

8
glance_store/tests/unit/test_multistore_vmware.py
View File

@ -192,7 +192,7 @@ class TestMultiStore(base.MultiStoreBaseTest,
expected_image_id, image, expected_size)
_, kwargs = HttpConn.call_args
self.assertEqual(expected_headers, kwargs['headers'])
self.assertEqual("vmware1", metadata["backend"])
self.assertEqual("vmware1", metadata["store"])
self.assertEqual(utils.sort_url_by_qs_keys(expected_location),
utils.sort_url_by_qs_keys(location))
@ -227,7 +227,7 @@ class TestMultiStore(base.MultiStoreBaseTest,
HttpConn.return_value = utils.fake_response()
location, size, checksum, metadata = self.store.add(
expected_image_id, image, 0)
self.assertEqual("vmware1", metadata["backend"])
self.assertEqual("vmware1", metadata["store"])
self.assertEqual(utils.sort_url_by_qs_keys(expected_location),
utils.sort_url_by_qs_keys(location))
@ -247,7 +247,7 @@ class TestMultiStore(base.MultiStoreBaseTest,
HttpConn.return_value = utils.fake_response()
location, size, checksum, multihash, metadata = self.store.add(
image_id, image, size, self.hash_algo, verifier=verifier)
self.assertEqual("vmware1", metadata["backend"])
self.assertEqual("vmware1", metadata["store"])
fake_reader.assert_called_with(image, self.hash_algo, verifier)
@ -264,7 +264,7 @@ class TestMultiStore(base.MultiStoreBaseTest,
HttpConn.return_value = utils.fake_response()
location, size, checksum, multihash, metadata = self.store.add(
image_id, image, 0, self.hash_algo, verifier=verifier)
self.assertEqual("vmware1", metadata["backend"])
self.assertEqual("vmware1", metadata["store"])
fake_reader.assert_called_with(image, self.hash_algo, verifier)

18
glance_store/tests/unit/test_swift_store_multibackend.py
View File

@ -407,7 +407,7 @@ class SwiftTests(object):
loc, size, checksum, metadata = self.store.add(
expected_image_id, image_swift, expected_swift_size)
self.assertEqual("swift1", metadata["backend"])
self.assertEqual("swift1", metadata["store"])
self.assertEqual(expected_location, loc)
self.assertEqual(expected_swift_size, size)
self.assertEqual(expected_checksum, checksum)
@ -446,7 +446,7 @@ class SwiftTests(object):
location, size, checksum, arg = self.store.add(expected_image_id,
image_swift,
expected_swift_size)
self.assertEqual("swift1", arg['backend'])
self.assertEqual("swift1", arg['store'])
self.assertEqual(expected_location, location)
@mock.patch('glance_store._drivers.swift.utils'
@ -492,7 +492,7 @@ class SwiftTests(object):
expected_swift_size,
context=ctxt)
self.assertEqual("swift1", metadata['backend'])
self.assertEqual("swift1", metadata['store'])
# ensure that image add uses user's context
self.assertEqual(expected_location, loc)
@ -535,7 +535,7 @@ class SwiftTests(object):
image_swift,
expected_swift_size)
self.assertEqual("swift1", metadata['backend'])
self.assertEqual("swift1", metadata['store'])
self.assertEqual(expected_location, loc)
self.assertEqual(expected_swift_size, size)
self.assertEqual(expected_checksum, checksum)
@ -615,7 +615,7 @@ class SwiftTests(object):
image_swift,
expected_swift_size)
self.assertEqual("swift1", metadata['backend'])
self.assertEqual("swift1", metadata['store'])
self.assertEqual(expected_location, loc)
self.assertEqual(expected_swift_size, size)
self.assertEqual(expected_checksum, checksum)
@ -664,7 +664,7 @@ class SwiftTests(object):
image_swift,
expected_swift_size)
self.assertEqual("swift1", metadata['backend'])
self.assertEqual("swift1", metadata['store'])
self.assertEqual(expected_location, loc)
self.assertEqual(expected_swift_size, size)
self.assertEqual(expected_checksum, checksum)
@ -849,7 +849,7 @@ class SwiftTests(object):
expected_swift_size,
context=ctxt)
self.assertEqual("swift1", metadata['backend'])
self.assertEqual("swift1", metadata['store'])
self.assertEqual(expected_location, location)
@mock.patch('glance_store._drivers.swift.utils'
@ -887,7 +887,7 @@ class SwiftTests(object):
self.store.large_object_chunk_size = orig_temp_size
self.store.large_object_size = orig_max_size
self.assertEqual("swift1", metadata['backend'])
self.assertEqual("swift1", metadata['store'])
self.assertEqual(expected_location, loc)
self.assertEqual(expected_swift_size, size)
self.assertEqual(expected_checksum, checksum)
@ -948,7 +948,7 @@ class SwiftTests(object):
self.store.large_object_size = orig_max_size
MAX_SWIFT_OBJECT_SIZE = orig_max_swift_object_size
self.assertEqual("swift1", metadata['backend'])
self.assertEqual("swift1", metadata['store'])
self.assertEqual(expected_location, loc)
self.assertEqual(expected_swift_size, size)
self.assertEqual(expected_checksum, checksum)