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

Erez Sh erez at redhat.com
Mon Jul 11 18:32:13 UTC 2011


New patch submitted by Erez Sh (erez at redhat.com)

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

commit b4b7c5efc10d4c0702523c69d4ac54ffba2d87b1
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 0516c3b..89330c4 100644
--- a/vdsm/libvirtvm.py
+++ b/vdsm/libvirtvm.py
@@ -214,7 +214,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