Merge "Fix hacking N302 import only modules"

9 files changed, 98 insertions, 93 deletions

diff --git a/nova/db/sqlalchemy/session.py b/nova/db/sqlalchemy/session.py
index cfabc7085..eb5d8016f 100644
--- a/nova/db/sqlalchemy/session.py
+++ b/nova/db/sqlalchemy/session.py
@@ -236,9 +236,7 @@ import sqlalchemy.orm
 from sqlalchemy.pool import NullPool, StaticPool
 from sqlalchemy.sql.expression import literal_column
 
-from nova.exception import DBDuplicateEntry
-from nova.exception import DBError
-from nova.exception import InvalidUnicodeParameter
+import nova.exception
 from nova.openstack.common import cfg
 import nova.openstack.common.log as logging
 from nova.openstack.common import timeutils
@@ -362,7 +360,7 @@ def raise_if_duplicate_entry_error(integrity_error, engine_name):
         columns = columns.strip().split(", ")
     else:
         columns = get_columns_from_uniq_cons_or_name(columns)
-    raise DBDuplicateEntry(columns, integrity_error)
+    raise nova.exception.DBDuplicateEntry(columns, integrity_error)
 
 
 def wrap_db_error(f):
@@ -370,7 +368,7 @@ def wrap_db_error(f):
         try:
             return f(*args, **kwargs)
         except UnicodeEncodeError:
-            raise InvalidUnicodeParameter()
+            raise nova.exception.InvalidUnicodeParameter()
         # note(boris-42): We should catch unique constraint violation and
         # wrap it by our own DBDuplicateEntry exception. Unique constraint
         # violation is wrapped by IntegrityError.
@@ -381,10 +379,10 @@ def wrap_db_error(f):
             # means we should get names of columns, which values violate
             # unique constraint, from error message.
             raise_if_duplicate_entry_error(e, get_engine().name)
-            raise DBError(e)
+            raise nova.exception.DBError(e)
         except Exception, e:
             LOG.exception(_('DB exception wrapped.'))
-            raise DBError(e)
+            raise nova.exception.DBError(e)
     _wrap.func_name = f.func_name
     return _wrap
 
diff --git a/nova/servicegroup/api.py b/nova/servicegroup/api.py
index 358b7dcbc..0fb30cdf5 100644
--- a/nova/servicegroup/api.py
+++ b/nova/servicegroup/api.py
@@ -23,7 +23,7 @@ from nova.openstack.common import lockutils
 from nova.openstack.common import log as logging
 from nova import utils
 
-from random import choice
+import random
 
 
 LOG = logging.getLogger(__name__)
@@ -144,4 +144,4 @@ class ServiceGroupDriver(object):
         length = len(members)
         if length == 0:
             return None
-        return choice(members)
+        return random.choice(members)
diff --git a/nova/tests/api/openstack/compute/contrib/test_availability_zone.py b/nova/tests/api/openstack/compute/contrib/test_availability_zone.py
index 8abe7f388..fb9a36ba9 100644
--- a/nova/tests/api/openstack/compute/contrib/test_availability_zone.py
+++ b/nova/tests/api/openstack/compute/contrib/test_availability_zone.py
@@ -13,7 +13,7 @@
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
-from datetime import datetime
+import datetime
 from lxml import etree
 import webob
 
@@ -40,29 +40,29 @@ def fake_service_get_all(context, disabled=None):
 
     if disabled:
         return [__fake_service("nova-compute", "zone-2",
-                               datetime(2012, 11, 14, 9, 53, 25, 0),
-                               datetime(2012, 12, 26, 14, 45, 25, 0),
+                               datetime.datetime(2012, 11, 14, 9, 53, 25, 0),
+                               datetime.datetime(2012, 12, 26, 14, 45, 25, 0),
                                "fake_host-1", True),
                 __fake_service("nova-scheduler", "internal",
-                               datetime(2012, 11, 14, 9, 57, 3, 0),
-                               datetime(2012, 12, 26, 14, 45, 25, 0),
+                               datetime.datetime(2012, 11, 14, 9, 57, 3, 0),
+                               datetime.datetime(2012, 12, 26, 14, 45, 25, 0),
                                "fake_host-1", True),
                 __fake_service("nova-network", "internal",
-                               datetime(2012, 11, 16, 7, 25, 46, 0),
-                               datetime(2012, 12, 26, 14, 45, 24, 0),
+                               datetime.datetime(2012, 11, 16, 7, 25, 46, 0),
+                               datetime.datetime(2012, 12, 26, 14, 45, 24, 0),
                                "fake_host-2", True)]
     else:
         return [__fake_service("nova-compute", "zone-1",
-                               datetime(2012, 11, 14, 9, 53, 25, 0),
-                               datetime(2012, 12, 26, 14, 45, 25, 0),
+                               datetime.datetime(2012, 11, 14, 9, 53, 25, 0),
+                               datetime.datetime(2012, 12, 26, 14, 45, 25, 0),
                                "fake_host-1", False),
                 __fake_service("nova-sched", "internal",
-                               datetime(2012, 11, 14, 9, 57, 03, 0),
-                               datetime(2012, 12, 26, 14, 45, 25, 0),
+                               datetime.datetime(2012, 11, 14, 9, 57, 03, 0),
+                               datetime.datetime(2012, 12, 26, 14, 45, 25, 0),
                                "fake_host-1", False),
                 __fake_service("nova-network", "internal",
-                               datetime(2012, 11, 16, 7, 25, 46, 0),
-                               datetime(2012, 12, 26, 14, 45, 24, 0),
+                               datetime.datetime(2012, 11, 16, 7, 25, 46, 0),
+                               datetime.datetime(2012, 12, 26, 14, 45, 24, 0),
                                "fake_host-2", False)]
 
 
@@ -218,18 +218,21 @@ class AvailabilityZoneSerializerTest(test.TestCase):
               'hosts': {'fake_host-1': {
                             'nova-compute': {'active': True, 'available': True,
                                 'updated_at':
-                                    datetime(2012, 12, 26, 14, 45, 25)}}}},
+                                    datetime.datetime(
+                                        2012, 12, 26, 14, 45, 25)}}}},
              {'zoneName': 'internal',
               'zoneState': {'available': True},
               'hosts': {'fake_host-1': {
                             'nova-sched': {'active': True, 'available': True,
                                 'updated_at':
-                                    datetime(2012, 12, 26, 14, 45, 25)}},
+                                    datetime.datetime(
+                                        2012, 12, 26, 14, 45, 25)}},
                         'fake_host-2': {
                             'nova-network': {'active': True,
                                              'available': False,
                                              'updated_at':
-                                    datetime(2012, 12, 26, 14, 45, 24)}}}},
+                                    datetime.datetime(
+                                        2012, 12, 26, 14, 45, 24)}}}},
              {'zoneName': 'zone-2',
               'zoneState': {'available': False},
               'hosts': None}]
diff --git a/nova/tests/api/openstack/compute/contrib/test_services.py b/nova/tests/api/openstack/compute/contrib/test_services.py
index 3a6e5db7c..aba1b92c1 100644
--- a/nova/tests/api/openstack/compute/contrib/test_services.py
+++ b/nova/tests/api/openstack/compute/contrib/test_services.py
@@ -14,7 +14,8 @@
 #    under the License.
 
 
-from datetime import datetime
+import datetime
+
 from nova.api.openstack.compute.contrib import services
 from nova import context
 from nova import db
@@ -24,35 +25,36 @@ from nova import test
 from nova.tests.api.openstack import fakes
 
 
-fake_services_list = [{'binary': 'nova-scheduler',
-                       'host': 'host1',
-                       'id': 1,
-                       'disabled': True,
-                       'topic': 'scheduler',
-                       'updated_at': datetime(2012, 10, 29, 13, 42, 2),
-                       'created_at': datetime(2012, 9, 18, 2, 46, 27)},
-                      {'binary': 'nova-compute',
-                       'host': 'host1',
-                       'id': 2,
-                       'disabled': True,
-                       'topic': 'compute',
-                       'updated_at': datetime(2012, 10, 29, 13, 42, 5),
-                       'created_at': datetime(2012, 9, 18, 2, 46, 27)},
-                      {'binary': 'nova-scheduler',
-                       'host': 'host2',
-                       'id': 3,
-                       'disabled': False,
-                       'topic': 'scheduler',
-                       'updated_at': datetime(2012, 9, 19, 6, 55, 34),
-                       'created_at': datetime(2012, 9, 18, 2, 46, 28)},
-                      {'binary': 'nova-compute',
-                       'host': 'host2',
-                       'id': 4,
-                       'disabled': True,
-                       'topic': 'compute',
-                       'updated_at': datetime(2012, 9, 18, 8, 3, 38),
-                       'created_at': datetime(2012, 9, 18, 2, 46, 28)},
-                      ]
+fake_services_list = [
+        {'binary': 'nova-scheduler',
+         'host': 'host1',
+         'id': 1,
+         'disabled': True,
+         'topic': 'scheduler',
+         'updated_at': datetime.datetime(2012, 10, 29, 13, 42, 2),
+         'created_at': datetime.datetime(2012, 9, 18, 2, 46, 27)},
+        {'binary': 'nova-compute',
+         'host': 'host1',
+         'id': 2,
+         'disabled': True,
+         'topic': 'compute',
+         'updated_at': datetime.datetime(2012, 10, 29, 13, 42, 5),
+         'created_at': datetime.datetime(2012, 9, 18, 2, 46, 27)},
+        {'binary': 'nova-scheduler',
+         'host': 'host2',
+         'id': 3,
+         'disabled': False,
+         'topic': 'scheduler',
+         'updated_at': datetime.datetime(2012, 9, 19, 6, 55, 34),
+         'created_at': datetime.datetime(2012, 9, 18, 2, 46, 28)},
+        {'binary': 'nova-compute',
+         'host': 'host2',
+         'id': 4,
+         'disabled': True,
+         'topic': 'compute',
+         'updated_at': datetime.datetime(2012, 9, 18, 8, 3, 38),
+         'created_at': datetime.datetime(2012, 9, 18, 2, 46, 28)},
+        ]
 
 
 class FakeRequest(object):
@@ -103,7 +105,7 @@ def fake_service_update(context, service_id, values):
 
 
 def fake_utcnow():
-    return datetime(2012, 10, 29, 13, 42, 11)
+    return datetime.datetime(2012, 10, 29, 13, 42, 11)
 
 
 class ServicesTest(test.TestCase):
@@ -130,19 +132,19 @@ class ServicesTest(test.TestCase):
         response = {'services': [{'binary': 'nova-scheduler',
                     'host': 'host1', 'zone': 'internal',
                     'status': 'disabled', 'state': 'up',
-                    'updated_at': datetime(2012, 10, 29, 13, 42, 2)},
+                    'updated_at': datetime.datetime(2012, 10, 29, 13, 42, 2)},
                     {'binary': 'nova-compute',
                      'host': 'host1', 'zone': 'nova',
                      'status': 'disabled', 'state': 'up',
-                     'updated_at': datetime(2012, 10, 29, 13, 42, 5)},
+                     'updated_at': datetime.datetime(2012, 10, 29, 13, 42, 5)},
                     {'binary': 'nova-scheduler', 'host': 'host2',
                      'zone': 'internal',
                      'status': 'enabled', 'state': 'down',
-                     'updated_at': datetime(2012, 9, 19, 6, 55, 34)},
+                     'updated_at': datetime.datetime(2012, 9, 19, 6, 55, 34)},
                     {'binary': 'nova-compute', 'host': 'host2',
                      'zone': 'nova',
                      'status': 'disabled', 'state': 'down',
-                     'updated_at': datetime(2012, 9, 18, 8, 3, 38)}]}
+                     'updated_at': datetime.datetime(2012, 9, 18, 8, 3, 38)}]}
         self.assertEqual(res_dict, response)
 
     def test_services_list_with_host(self):
@@ -152,11 +154,11 @@ class ServicesTest(test.TestCase):
         response = {'services': [{'binary': 'nova-scheduler', 'host': 'host1',
                     'zone': 'internal',
                     'status': 'disabled', 'state': 'up',
-                    'updated_at': datetime(2012, 10, 29, 13, 42, 2)},
+                    'updated_at': datetime.datetime(2012, 10, 29, 13, 42, 2)},
                    {'binary': 'nova-compute', 'host': 'host1',
                     'zone': 'nova',
                     'status': 'disabled', 'state': 'up',
-                    'updated_at': datetime(2012, 10, 29, 13, 42, 5)}]}
+                    'updated_at': datetime.datetime(2012, 10, 29, 13, 42, 5)}]}
         self.assertEqual(res_dict, response)
 
     def test_services_list_with_service(self):
@@ -166,11 +168,11 @@ class ServicesTest(test.TestCase):
         response = {'services': [{'binary': 'nova-compute', 'host': 'host1',
                     'zone': 'nova',
                     'status': 'disabled', 'state': 'up',
-                    'updated_at': datetime(2012, 10, 29, 13, 42, 5)},
+                    'updated_at': datetime.datetime(2012, 10, 29, 13, 42, 5)},
                     {'binary': 'nova-compute', 'host': 'host2',
                      'zone': 'nova',
                      'status': 'disabled', 'state': 'down',
-                     'updated_at': datetime(2012, 9, 18, 8, 3, 38)}]}
+                     'updated_at': datetime.datetime(2012, 9, 18, 8, 3, 38)}]}
         self.assertEqual(res_dict, response)
 
     def test_services_list_with_host_service(self):
@@ -180,7 +182,7 @@ class ServicesTest(test.TestCase):
         response = {'services': [{'binary': 'nova-compute', 'host': 'host1',
                     'zone': 'nova',
                     'status': 'disabled', 'state': 'up',
-                    'updated_at': datetime(2012, 10, 29, 13, 42, 5)}]}
+                    'updated_at': datetime.datetime(2012, 10, 29, 13, 42, 5)}]}
         self.assertEqual(res_dict, response)
 
     def test_services_enable(self):
diff --git a/nova/tests/conf_fixture.py b/nova/tests/conf_fixture.py
index 9155a3f68..2f4d0ebb1 100644
--- a/nova/tests/conf_fixture.py
+++ b/nova/tests/conf_fixture.py
@@ -22,7 +22,7 @@ from nova import config
 from nova import ipv6
 from nova.openstack.common import cfg
 from nova import paths
-from nova.tests.utils import cleanup_dns_managers
+from nova.tests import utils
 
 CONF = cfg.CONF
 CONF.import_opt('use_ipv6', 'nova.netconf')
@@ -70,5 +70,5 @@ class ConfFixture(fixtures.Fixture):
         self.conf.set_default('vlan_interface', 'eth0')
         config.parse_args([], default_config_files=[])
         self.addCleanup(self.conf.reset)
-        self.addCleanup(cleanup_dns_managers)
+        self.addCleanup(utils.cleanup_dns_managers)
         self.addCleanup(ipv6.api.reset_backend)
diff --git a/nova/tests/test_api.py b/nova/tests/test_api.py
index fb2e76e45..11c16d6dd 100644
--- a/nova/tests/test_api.py
+++ b/nova/tests/test_api.py
@@ -26,9 +26,9 @@ from boto.ec2 import regioninfo
 from boto import exception as boto_exc
 # newer versions of boto use their own wrapper on top of httplib.HTTPResponse
 try:
-    from boto.connection import HTTPResponse
+    import boto.connection as httplib
 except ImportError:
-    from httplib import HTTPResponse
+    import httplib
 import fixtures
 import webob
 
@@ -79,7 +79,7 @@ class FakeHttplibConnection(object):
         # guess that's a function the web server usually provides.
         resp = "HTTP/1.0 %s" % resp
         self.sock = FakeHttplibSocket(resp)
-        self.http_response = HTTPResponse(self.sock)
+        self.http_response = httplib.HTTPResponse(self.sock)
         # NOTE(vish): boto is accessing private variables for some reason
         self._HTTPConnection__response = self.http_response
         self.http_response.begin()
diff --git a/nova/tests/test_metadata.py b/nova/tests/test_metadata.py
index f15d71633..c2f0b5a11 100644
--- a/nova/tests/test_metadata.py
+++ b/nova/tests/test_metadata.py
@@ -19,7 +19,7 @@
 """Tests for metadata service."""
 
 import base64
-from copy import copy
+import copy
 import json
 import re
 
@@ -120,14 +120,14 @@ class MetadataTestCase(test.TestCase):
                                                           spectacular=True)
 
     def test_user_data(self):
-        inst = copy(self.instance)
+        inst = copy.copy(self.instance)
         inst['user_data'] = base64.b64encode("happy")
         md = fake_InstanceMetadata(self.stubs, inst)
         self.assertEqual(
             md.get_ec2_metadata(version='2009-04-04')['user-data'], "happy")
 
     def test_no_user_data(self):
-        inst = copy(self.instance)
+        inst = copy.copy(self.instance)
         del inst['user_data']
         md = fake_InstanceMetadata(self.stubs, inst)
         obj = object()
@@ -136,7 +136,7 @@ class MetadataTestCase(test.TestCase):
             obj)
 
     def test_security_groups(self):
-        inst = copy(self.instance)
+        inst = copy.copy(self.instance)
         sgroups = [{'name': 'default'}, {'name': 'other'}]
         expected = ['default', 'other']
 
@@ -145,7 +145,7 @@ class MetadataTestCase(test.TestCase):
         self.assertEqual(data['meta-data']['security-groups'], expected)
 
     def test_local_hostname_fqdn(self):
-        md = fake_InstanceMetadata(self.stubs, copy(self.instance))
+        md = fake_InstanceMetadata(self.stubs, copy.copy(self.instance))
         data = md.get_ec2_metadata(version='2009-04-04')
         self.assertEqual(data['meta-data']['local-hostname'],
             "%s.%s" % (self.instance['hostname'], CONF.dhcp_domain))
@@ -195,7 +195,7 @@ class MetadataTestCase(test.TestCase):
                          expected)
 
     def test_pubkey(self):
-        md = fake_InstanceMetadata(self.stubs, copy(self.instance))
+        md = fake_InstanceMetadata(self.stubs, copy.copy(self.instance))
         pubkey_ent = md.lookup("/2009-04-04/meta-data/public-keys")
 
         self.assertEqual(base.ec2_md_print(pubkey_ent),
@@ -204,7 +204,7 @@ class MetadataTestCase(test.TestCase):
             self.instance['key_data'])
 
     def test_image_type_ramdisk(self):
-        inst = copy(self.instance)
+        inst = copy.copy(self.instance)
         inst['ramdisk_id'] = 'ari-853667c0'
         md = fake_InstanceMetadata(self.stubs, inst)
         data = md.lookup("/latest/meta-data/ramdisk-id")
@@ -213,7 +213,7 @@ class MetadataTestCase(test.TestCase):
         self.assertTrue(re.match('ari-[0-9a-f]{8}', data))
 
     def test_image_type_kernel(self):
-        inst = copy(self.instance)
+        inst = copy.copy(self.instance)
         inst['kernel_id'] = 'aki-c2e26ff2'
         md = fake_InstanceMetadata(self.stubs, inst)
         data = md.lookup("/2009-04-04/meta-data/kernel-id")
@@ -229,7 +229,7 @@ class MetadataTestCase(test.TestCase):
             md.lookup, "/2009-04-04/meta-data/kernel-id")
 
     def test_check_version(self):
-        inst = copy(self.instance)
+        inst = copy.copy(self.instance)
         md = fake_InstanceMetadata(self.stubs, inst)
 
         self.assertTrue(md._check_version('1.0', '2009-04-04'))
@@ -250,7 +250,7 @@ class OpenStackMetadataTestCase(test.TestCase):
 
     def test_top_level_listing(self):
         # request for /openstack/<version>/ should show metadata.json
-        inst = copy(self.instance)
+        inst = copy.copy(self.instance)
         mdinst = fake_InstanceMetadata(self.stubs, inst)
 
         listing = mdinst.lookup("/openstack/")
@@ -267,14 +267,14 @@ class OpenStackMetadataTestCase(test.TestCase):
 
     def test_version_content_listing(self):
         # request for /openstack/<version>/ should show metadata.json
-        inst = copy(self.instance)
+        inst = copy.copy(self.instance)
         mdinst = fake_InstanceMetadata(self.stubs, inst)
 
         listing = mdinst.lookup("/openstack/2012-08-10")
         self.assertTrue("meta_data.json" in listing)
 
     def test_metadata_json(self):
-        inst = copy(self.instance)
+        inst = copy.copy(self.instance)
         content = [
             ('/etc/my.conf', "content of my.conf"),
             ('/root/hello', "content of /root/hello"),
@@ -309,7 +309,7 @@ class OpenStackMetadataTestCase(test.TestCase):
 
     def test_extra_md(self):
         # make sure extra_md makes it through to metadata
-        inst = copy(self.instance)
+        inst = copy.copy(self.instance)
         extra = {'foo': 'bar', 'mylist': [1, 2, 3],
                  'mydict': {"one": 1, "two": 2}}
         mdinst = fake_InstanceMetadata(self.stubs, inst, extra_md=extra)
@@ -322,14 +322,14 @@ class OpenStackMetadataTestCase(test.TestCase):
 
     def test_password(self):
         # make sure extra_md makes it through to metadata
-        inst = copy(self.instance)
+        inst = copy.copy(self.instance)
         mdinst = fake_InstanceMetadata(self.stubs, inst)
 
         result = mdinst.lookup("/openstack/latest/password")
         self.assertEqual(result, password.handle_password)
 
     def test_userdata(self):
-        inst = copy(self.instance)
+        inst = copy.copy(self.instance)
         mdinst = fake_InstanceMetadata(self.stubs, inst)
 
         userdata_found = mdinst.lookup("/openstack/2012-08-10/user_data")
@@ -348,7 +348,7 @@ class OpenStackMetadataTestCase(test.TestCase):
             mdinst.lookup, "/openstack/2012-08-10/user_data")
 
     def test_random_seed(self):
-        inst = copy(self.instance)
+        inst = copy.copy(self.instance)
         mdinst = fake_InstanceMetadata(self.stubs, inst)
 
         # verify that 2013-04-04 has the 'random' field
@@ -364,7 +364,7 @@ class OpenStackMetadataTestCase(test.TestCase):
 
     def test_no_dashes_in_metadata(self):
         # top level entries in meta_data should not contain '-' in their name
-        inst = copy(self.instance)
+        inst = copy.copy(self.instance)
         mdinst = fake_InstanceMetadata(self.stubs, inst)
         mdjson = json.loads(mdinst.lookup("/openstack/latest/meta_data.json"))
 
@@ -522,7 +522,7 @@ class MetadataPasswordTestCase(test.TestCase):
         super(MetadataPasswordTestCase, self).setUp()
         fake_network.stub_out_nw_api_get_instance_nw_info(self.stubs,
                                                           spectacular=True)
-        self.instance = copy(INSTANCES[0])
+        self.instance = copy.copy(INSTANCES[0])
         self.mdinst = fake_InstanceMetadata(self.stubs, self.instance,
             address=None, sgroups=None)
 
diff --git a/nova/virt/hyperv/vif.py b/nova/virt/hyperv/vif.py
index a898d3ac2..e01006eaa 100644
--- a/nova/virt/hyperv/vif.py
+++ b/nova/virt/hyperv/vif.py
@@ -15,6 +15,7 @@
 #    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 abc
 import sys
 import uuid
 
@@ -22,11 +23,12 @@ import uuid
 if sys.platform == 'win32':
     import wmi
 
-from abc import abstractmethod
+
 from nova.openstack.common import cfg
 from nova.openstack.common import log as logging
 from nova.virt.hyperv import vmutils
 
+
 hyperv_opts = [
     cfg.StrOpt('vswitch_name',
                default=None,
@@ -42,11 +44,11 @@ LOG = logging.getLogger(__name__)
 
 
 class HyperVBaseVIFDriver(object):
-    @abstractmethod
+    @abc.abstractmethod
     def plug(self, instance, vif):
         pass
 
-    @abstractmethod
+    @abc.abstractmethod
     def unplug(self, instance, vif):
         pass
 
diff --git a/nova/volume/cinder.py b/nova/volume/cinder.py
index 3e1ccc66b..daca69854 100644
--- a/nova/volume/cinder.py
+++ b/nova/volume/cinder.py
@@ -20,7 +20,7 @@
 Handles all requests relating to volumes + cinder.
 """
 
-from copy import deepcopy
+import copy
 import sys
 
 from cinderclient import exceptions as cinder_exception
@@ -139,7 +139,7 @@ def _untranslate_volume_summary_view(context, vol):
         d['volume_metadata'].append(item)
 
     if hasattr(vol, 'volume_image_metadata'):
-        d['volume_image_metadata'] = deepcopy(vol.volume_image_metadata)
+        d['volume_image_metadata'] = copy.deepcopy(vol.volume_image_metadata)
 
     return d