Change in vdsm[master]: numa: make use of DisconnectedVirDomain

fromani at redhat.com fromani at redhat.com
Wed Aug 12 13:43:26 UTC 2015


Francesco Romani has uploaded a new change for review.

Change subject: numa: make use of DisconnectedVirDomain
......................................................................

numa: make use of DisconnectedVirDomain

avoid nasty AttributeError check.

Change-Id: I1dd095e23bc792b82def10db643762eac5e18a0e
Signed-off-by: Francesco Romani <fromani at redhat.com>
---
M vdsm/numaUtils.py
1 file changed, 3 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/62/44762/1

diff --git a/vdsm/numaUtils.py b/vdsm/numaUtils.py
index f6bd0da..3b462c0 100644
--- a/vdsm/numaUtils.py
+++ b/vdsm/numaUtils.py
@@ -125,10 +125,10 @@
 
 
 def _get_vcpu_positioning(vm):
-    try:
+    if vm._dom.connected:
+        # TODO: remove this evilness
         return vm._dom.vcpus()[0]
-    except AttributeError:
-        # _dom may be reset to none asynchronously
+    else:
         return None
 
 


-- 
To view, visit https://gerrit.ovirt.org/44762
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1dd095e23bc792b82def10db643762eac5e18a0e
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>


More information about the vdsm-patches mailing list