From a04a5fa0cc0f87193502da2e68efe5102aef55b3 Mon Sep 17 00:00:00 2001 From: Julien Danjou Date: Wed, 14 Dec 2011 14:41:03 +0100 Subject: Make various methods static A lot of stuff in nova.virt contain methods that are actually static. Marking them this way makes the code clearer. Change-Id: I287058d3b2315562f2980999239fcd3240d2b1ec Signed-off-by: Julien Danjou --- nova/virt/fake.py | 3 ++- nova/virt/libvirt/connection.py | 33 ++++++++++++++++++++++----------- nova/virt/xenapi_conn.py | 3 ++- 3 files changed, 26 insertions(+), 13 deletions(-) (limited to 'nova') diff --git a/nova/virt/fake.py b/nova/virt/fake.py index f6d93bc53..c1e32be90 100644 --- a/nova/virt/fake.py +++ b/nova/virt/fake.py @@ -115,7 +115,8 @@ class FakeConnection(driver.ComputeDriver): def reboot(self, instance, network_info, reboot_type): pass - def get_host_ip_addr(self): + @staticmethod + def get_host_ip_addr(): return '192.168.0.1' def resize(self, instance, flavor): diff --git a/nova/virt/libvirt/connection.py b/nova/virt/libvirt/connection.py index 566db2d77..db5212623 100644 --- a/nova/virt/libvirt/connection.py +++ b/nova/virt/libvirt/connection.py @@ -252,7 +252,8 @@ class LibvirtConnection(driver.ComputeDriver): uri = FLAGS.libvirt_uri or 'qemu:///system' return uri - def _connect(self, uri, read_only): + @staticmethod + def _connect(uri, read_only): auth = [[libvirt.VIR_CRED_AUTHNAME, libvirt.VIR_CRED_NOECHOPROMPT], 'root', None] @@ -266,7 +267,8 @@ class LibvirtConnection(driver.ComputeDriver): return [self._conn.lookupByID(x).name() for x in self._conn.listDomainsID()] - def _map_to_instance_info(self, domain): + @staticmethod + def _map_to_instance_info(domain): """Gets info from a virsh domain object into an InstanceInfo""" # domain.info() returns a list of: @@ -416,7 +418,8 @@ class LibvirtConnection(driver.ComputeDriver): mount_device) virt_dom.attachDevice(xml) - def _get_disk_xml(self, xml, device): + @staticmethod + def _get_disk_xml(xml, device): """Returns the xml for the disk mounted at device""" try: doc = ElementTree.fromstring(xml) @@ -754,7 +757,8 @@ class LibvirtConnection(driver.ComputeDriver): libvirt_utils.run_ajaxterm(ajaxterm_cmd, token, port) return {'token': token, 'host': host, 'port': port} - def get_host_ip_addr(self): + @staticmethod + def get_host_ip_addr(): return FLAGS.my_ip @exception.wrap_exception() @@ -809,14 +813,16 @@ class LibvirtConnection(driver.ComputeDriver): else: libvirt_utils.copy_image(base, target) - def _fetch_image(self, context, target, image_id, user_id, project_id, + @staticmethod + def _fetch_image(context, target, image_id, user_id, project_id, size=None): """Grab image and optionally attempt to resize it""" images.fetch_to_raw(context, image_id, target, user_id, project_id) if size: disk.extend(target, size) - def _create_local(self, target, local_size, unit='G', fs_format=None): + @staticmethod + def _create_local(target, local_size, unit='G', fs_format=None): """Create a blank image of specified size""" if not fs_format: @@ -831,7 +837,8 @@ class LibvirtConnection(driver.ComputeDriver): self._create_local(target, local_size) disk.mkfs(os_type, fs_label, target) - def _create_swap(self, target, swap_mb): + @staticmethod + def _create_swap(target, swap_mb): """Create a swap file of specified size""" libvirt_utils.create_image('raw', target, '%dM' % swap_mb) libvirt_utils.mkfs(target, 'swap') @@ -1052,7 +1059,8 @@ class LibvirtConnection(driver.ComputeDriver): if FLAGS.libvirt_type == 'uml': libvirt_utils.chown(basepath('disk'), 'root') - def _volume_in_mapping(self, mount_device, block_device_info): + @staticmethod + def _volume_in_mapping(mount_device, block_device_info): block_device_list = [block_device.strip_dev(vol['mount_device']) for vol in driver.block_device_info_get_mapping( @@ -1305,7 +1313,8 @@ class LibvirtConnection(driver.ComputeDriver): return interfaces - def get_vcpu_total(self): + @staticmethod + def get_vcpu_total(): """Get vcpu number of physical computer. :returns: the number of cpu core. @@ -1321,7 +1330,8 @@ class LibvirtConnection(driver.ComputeDriver): "This error can be safely ignored for now.")) return 0 - def get_memory_mb_total(self): + @staticmethod + def get_memory_mb_total(): """Get the total memory size(MB) of physical computer. :returns: the total amount of memory(MB). @@ -1336,7 +1346,8 @@ class LibvirtConnection(driver.ComputeDriver): # transforming kb to mb. return int(meminfo[idx + 1]) / 1024 - def get_local_gb_total(self): + @staticmethod + def get_local_gb_total(): """Get the total hdd size(GB) of physical computer. :returns: diff --git a/nova/virt/xenapi_conn.py b/nova/virt/xenapi_conn.py index 65cad20df..61b830054 100644 --- a/nova/virt/xenapi_conn.py +++ b/nova/virt/xenapi_conn.py @@ -339,7 +339,8 @@ class XenAPIConnection(driver.ComputeDriver): """Return link to instance's ajax console""" return self._vmops.get_ajax_console(instance) - def get_host_ip_addr(self): + @staticmethod + def get_host_ip_addr(): xs_url = urlparse.urlparse(FLAGS.xenapi_connection_url) return xs_url.netloc -- cgit