summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorArata Notsu <notsu@virtualtech.jp>2012-10-18 22:40:01 +0900
committerArata Notsu <notsu@virtualtech.jp>2012-10-18 22:40:01 +0900
commit2cb47fbabf09ced76178e8453d508c66f86a7a3a (patch)
tree3c1e9fa05db610481d209e6b2f2be730bc2e04cf
parent5d677e204c99fbcf86e0c721ad2fd327a03c35c2 (diff)
downloadnova-2cb47fbabf09ced76178e8453d508c66f86a7a3a.tar.gz
nova-2cb47fbabf09ced76178e8453d508c66f86a7a3a.tar.xz
nova-2cb47fbabf09ced76178e8453d508c66f86a7a3a.zip
Remove ComputeDriver.update_host_status()
Nobody calls the method. Change-Id: Ib585f769788753c64e19fe49a1f186880e1c6f49
-rw-r--r--nova/tests/test_virt_drivers.py5
-rw-r--r--nova/virt/baremetal/driver.py5
-rw-r--r--nova/virt/driver.py4
-rw-r--r--nova/virt/fake.py4
-rw-r--r--nova/virt/hyperv/driver.py4
-rw-r--r--nova/virt/libvirt/driver.py8
-rw-r--r--nova/virt/xenapi/driver.py5
7 files changed, 0 insertions, 35 deletions
diff --git a/nova/tests/test_virt_drivers.py b/nova/tests/test_virt_drivers.py
index 05fe140d1..d5a2203f3 100644
--- a/nova/tests/test_virt_drivers.py
+++ b/nova/tests/test_virt_drivers.py
@@ -459,11 +459,6 @@ class _VirtDriverTestCase(_FakeDriverBackendTestCase):
self.assertIn('host_memory_free', host_status)
@catch_notimplementederror
- def test_update_host_status(self):
- host_status = self.connection.update_host_status()
- self._check_host_status_fields(host_status)
-
- @catch_notimplementederror
def test_get_host_stats(self):
host_status = self.connection.get_host_stats()
self._check_host_status_fields(host_status)
diff --git a/nova/virt/baremetal/driver.py b/nova/virt/baremetal/driver.py
index e0941c37c..544e38c17 100644
--- a/nova/virt/baremetal/driver.py
+++ b/nova/virt/baremetal/driver.py
@@ -692,11 +692,6 @@ class BareMetalDriver(driver.ComputeDriver):
"""See comments of same method in firewall_driver."""
pass
- def update_host_status(self):
- """Update the status info of the host, and return those values
- to the calling program."""
- return self.HostState.update_status()
-
def get_host_stats(self, refresh=False):
"""Return the current state of the host. If 'refresh' is
True, run the update first."""
diff --git a/nova/virt/driver.py b/nova/virt/driver.py
index d741524b0..59fac115f 100644
--- a/nova/virt/driver.py
+++ b/nova/virt/driver.py
@@ -586,10 +586,6 @@ class ComputeDriver(object):
"""Unplug VIFs from networks."""
raise NotImplementedError()
- def update_host_status(self):
- """Refresh host stats"""
- raise NotImplementedError()
-
def get_host_stats(self, refresh=False):
"""Return currently known host stats"""
raise NotImplementedError()
diff --git a/nova/virt/fake.py b/nova/virt/fake.py
index 959ab174c..6134019b2 100644
--- a/nova/virt/fake.py
+++ b/nova/virt/fake.py
@@ -273,10 +273,6 @@ class FakeDriver(driver.ComputeDriver):
""" Removes the named VM, as if it crashed. For testing"""
self.instances.pop(instance_name)
- def update_host_status(self):
- """Return fake Host Status of ram, disk, network."""
- return self.host_status
-
def get_host_stats(self, refresh=False):
"""Return fake Host Status of ram, disk, network."""
return self.host_status
diff --git a/nova/virt/hyperv/driver.py b/nova/virt/hyperv/driver.py
index 1026ced95..f02ad3511 100644
--- a/nova/virt/hyperv/driver.py
+++ b/nova/virt/hyperv/driver.py
@@ -123,10 +123,6 @@ class HyperVDriver(driver.ComputeDriver):
def get_available_resource(self):
return self._vmops.get_available_resource()
- def update_host_status(self):
- """See xenapi_conn.py implementation."""
- pass
-
def get_host_stats(self, refresh=False):
"""See xenapi_conn.py implementation."""
return {}
diff --git a/nova/virt/libvirt/driver.py b/nova/virt/libvirt/driver.py
index 763a98e80..2d3b1d954 100644
--- a/nova/virt/libvirt/driver.py
+++ b/nova/virt/libvirt/driver.py
@@ -2732,14 +2732,6 @@ class LibvirtDriver(driver.ComputeDriver):
self.firewall_driver.unfilter_instance(instance_ref,
network_info=network_info)
- def update_host_status(self):
- """Retrieve status info from libvirt.
-
- Query libvirt to get the state of the compute node, such
- as memory and disk usage.
- """
- return self.host_state.update_status()
-
def get_host_stats(self, refresh=False):
"""Return the current state of the host.
diff --git a/nova/virt/xenapi/driver.py b/nova/virt/xenapi/driver.py
index 04b75598c..2db7c1b17 100644
--- a/nova/virt/xenapi/driver.py
+++ b/nova/virt/xenapi/driver.py
@@ -554,11 +554,6 @@ class XenAPIDriver(driver.ComputeDriver):
def refresh_provider_fw_rules(self):
return self._vmops.refresh_provider_fw_rules()
- def update_host_status(self):
- """Update the status info of the host, and return those values
- to the calling program."""
- return self.host_state.update_status()
-
def get_host_stats(self, refresh=False):
"""Return the current state of the host. If 'refresh' is
True, run the update first."""