diff options
| author | Arata Notsu <notsu@virtualtech.jp> | 2012-10-18 22:40:01 +0900 |
|---|---|---|
| committer | Arata Notsu <notsu@virtualtech.jp> | 2012-10-18 22:40:01 +0900 |
| commit | 2cb47fbabf09ced76178e8453d508c66f86a7a3a (patch) | |
| tree | 3c1e9fa05db610481d209e6b2f2be730bc2e04cf | |
| parent | 5d677e204c99fbcf86e0c721ad2fd327a03c35c2 (diff) | |
| download | nova-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.py | 5 | ||||
| -rw-r--r-- | nova/virt/baremetal/driver.py | 5 | ||||
| -rw-r--r-- | nova/virt/driver.py | 4 | ||||
| -rw-r--r-- | nova/virt/fake.py | 4 | ||||
| -rw-r--r-- | nova/virt/hyperv/driver.py | 4 | ||||
| -rw-r--r-- | nova/virt/libvirt/driver.py | 8 | ||||
| -rw-r--r-- | nova/virt/xenapi/driver.py | 5 |
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.""" |
