Change in vdsm[master]: supervdsmServer: Run sourceRouteThread in a thread

nsoffer at redhat.com nsoffer at redhat.com
Tue Dec 24 16:08:27 UTC 2013


Nir Soffer has uploaded a new change for review.

Change subject: supervdsmServer: Run sourceRouteThread in a thread
......................................................................

supervdsmServer: Run sourceRouteThread in a thread

The current code was running the sourceRouteThread main function in the
main thread instead of starting it in a thread. This probably leads to
bogus message in the supervdsm log. Now the code is doing what is try
to do.

Change-Id: I5eb78b57b9ae6ada9948815020ac3d5ac005d20a
Signed-off-by: Nir Soffer <nsoffer at redhat.com>
---
M vdsm/supervdsmServer
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/18/22718/1

diff --git a/vdsm/supervdsmServer b/vdsm/supervdsmServer
index 59df478..b3b3bff 100755
--- a/vdsm/supervdsmServer
+++ b/vdsm/supervdsmServer
@@ -407,7 +407,7 @@
             log.debug("Started serving super vdsm object")
 
             _sourceRouteThread = threading.Thread(
-                target=sourceRouteThread.subscribeToInotifyLoop(),
+                target=sourceRouteThread.subscribeToInotifyLoop,
                 name='sourceRoute')
             _sourceRouteThread.daemon = True
             _sourceRouteThread.start()


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5eb78b57b9ae6ada9948815020ac3d5ac005d20a
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>


More information about the vdsm-patches mailing list