From 96ee3afb598652392c0467defc439a30c843f12a Mon Sep 17 00:00:00 2001 From: Devananda van der Veen Date: Fri, 28 Mar 2014 11:15:55 -0700 Subject: [PATCH] Clean up IronicNodeStates.update_from_compute_node This method was needlessly wordy. This patch does not change any functionality, merely makes the method easier to read. Change-Id: I7d66ea5e2146877f70a2e9060d3d4244f9493b9d --- ironic/nova/scheduler/ironic_host_manager.py | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/ironic/nova/scheduler/ironic_host_manager.py b/ironic/nova/scheduler/ironic_host_manager.py index 1dab66993d..fde0c55596 100644 --- a/ironic/nova/scheduler/ironic_host_manager.py +++ b/ironic/nova/scheduler/ironic_host_manager.py @@ -36,15 +36,12 @@ class IronicNodeState(host_manager.HostState): def update_from_compute_node(self, compute): """Update information about a host from its compute_node info.""" - all_ram_mb = compute['memory_mb'] + self.free_ram_mb = compute['free_ram_mb'] + self.total_usable_ram_mb = compute['memory_mb'] - free_disk_mb = compute['free_disk_gb'] * 1024 - free_ram_mb = compute['free_ram_mb'] - - self.free_ram_mb = free_ram_mb - self.total_usable_ram_mb = all_ram_mb - self.free_disk_mb = free_disk_mb + self.free_disk_mb = compute['free_disk_gb'] * 1024 self.total_usable_disk_gb = compute['local_gb'] + self.vcpus_total = compute['vcpus'] self.vcpus_used = compute['vcpus_used']