From 2d4a2c68829f794643a3472d17ae810711e07771 Mon Sep 17 00:00:00 2001 From: Stephen Finucane Date: Fri, 5 Jul 2019 12:22:35 +0100 Subject: [PATCH] claims: Remove useless caching Nothing was ever setting the '_numa_topology_loaded' attribute which means this caching has never worked. Just drop the idea entirely. Change-Id: I67c1d96f5b7c7c2c171869fbd54392d118b735af Signed-off-by: Stephen Finucane --- nova/compute/claims.py | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/nova/compute/claims.py b/nova/compute/claims.py index 356d071c71..52c27b6de7 100644 --- a/nova/compute/claims.py +++ b/nova/compute/claims.py @@ -63,7 +63,6 @@ class Claim(NopClaim): # Stash a copy of the instance at the current point of time self.instance = instance.obj_clone() self.nodename = nodename - self._numa_topology_loaded = False self.tracker = tracker self._pci_requests = pci_requests self.context = context @@ -74,11 +73,7 @@ class Claim(NopClaim): @property def numa_topology(self): - if self._numa_topology_loaded: - return self._numa_topology - else: - self._numa_topology = self.instance.numa_topology - return self._numa_topology + return self.instance.numa_topology def abort(self): """Compute operation requiring claimed resources has failed or