[NEW PATCH] BZ#720425 - No longer using the return value from wait, because there is None. (via gerrit-bot)

Dan Kenigsberg danken at redhat.com
Sun Jul 17 18:26:03 UTC 2011


New patch submitted by Dan Kenigsberg (danken at redhat.com)

You can review this change at: http://gerrit.usersys.redhat.com/684

commit ae07f25888a36eea66e8da5c44b7195e707caa46
Author: Erez Sh <erez at redhat.com>
Date:   Mon Jul 11 18:30:48 2011 +0300

    BZ#720425 - No longer using the return value from wait, because there is None.
    
    Change-Id: I1f6f22fb4e047dc24335eeef7664c33e1076eb02

diff --git a/vdsm/libvirtvm.py b/vdsm/libvirtvm.py
index 79da4b2..21961c2 100644
--- a/vdsm/libvirtvm.py
+++ b/vdsm/libvirtvm.py
@@ -222,7 +222,8 @@ class MigrationMonitorThread(threading.Thread):
     def run(self):
         self._vm.log.debug('starting migration monitor thread')
 
-        while not self._stop.wait(self._MIGRATION_MONITOR_INTERVAL):
+        while not self._stop.isSet():
+            self._stop.wait(self._MIGRATION_MONITOR_INTERVAL)
             jobType, timeElapsed, _,     \
             dataTotal, dataProcessed, _, \
             memTotal, memProcessed, _,   \




More information about the vdsm-patches mailing list