Change in vdsm[master]: BZ#829110 Remove a "Domain not found" error from stats threa...

ghammer at redhat.com ghammer at redhat.com
Tue Sep 4 14:15:33 UTC 2012


Gal Hammer has uploaded a new change for review.

Change subject: BZ#829110 Remove a "Domain not found" error from stats thread.
......................................................................

BZ#829110 Remove a "Domain not found" error from stats thread.

Removed redundant code which changed VM status if error occured on
the stats thead.

Change-Id: I1f648b9e3a70dca66cbf056c1f143bc6f0455e4a
Signed-off-by: Gal Hammer <ghammer at redhat.com>
---
M vdsm/libvirtvm.py
1 file changed, 0 insertions(+), 11 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/46/7746/1

diff --git a/vdsm/libvirtvm.py b/vdsm/libvirtvm.py
index 6323d0c..2f6e544 100644
--- a/vdsm/libvirtvm.py
+++ b/vdsm/libvirtvm.py
@@ -291,17 +291,6 @@
         if ex.get_error_code() != libvirt.VIR_ERR_NO_DOMAIN:
             return False
 
-        # If a VM is down, hibernating, migrating, destroyed or in the
-        # process of being shutdown we were expecting it to disappear
-        if ((self._vm.lastStatus in ('Down',
-                                     'Saving State', 'Migration Source'))
-                or self._vm.destroyed
-                or self._vm._guestEvent == 'Powering down'):
-            return True
-
-        self._log.debug("VM not found, moving to Down", exc_info=True)
-        self._vm.setDownStatus(ERROR, str(ex))
-
         return True
 
 


--
To view, visit http://gerrit.ovirt.org/7746
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1f648b9e3a70dca66cbf056c1f143bc6f0455e4a
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Gal Hammer <ghammer at redhat.com>


More information about the vdsm-patches mailing list