Change in vdsm[master]: sourcerouting: make thread daemonic

asegurap at redhat.com asegurap at redhat.com
Thu Dec 12 14:53:56 UTC 2013


Antoni Segura Puimedon has uploaded a new change for review.

Change subject: sourcerouting: make thread daemonic
......................................................................

sourcerouting: make thread daemonic

The Python interpreter waits for non daemonic threads. The
sourceRouteThread shouldn't keep the process waiting.

Change-Id: I45effe0d787c36564a1ae3eb45844c22ac2a2c36
Signed-off-by: Antoni S. Puimedon <asegurap at redhat.com>
---
M vdsm/supervdsmServer
1 file changed, 5 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/45/22345/1

diff --git a/vdsm/supervdsmServer b/vdsm/supervdsmServer
index f7b23ae..3f2cf85 100755
--- a/vdsm/supervdsmServer
+++ b/vdsm/supervdsmServer
@@ -407,8 +407,11 @@
 
             log.debug("Started serving super vdsm object")
 
-            threading.Thread(target=sourceRouteThread.subscribeToInotifyLoop(),
-                             name='sourceRoute').start()
+            _sourceRouteThread = threading.Thread(
+                target=sourceRouteThread.subscribeToInotifyLoop(),
+                name='sourceRoute')
+            _sourceRouteThread.daemon = True
+            _sourceRouteThread.start()
 
             # Python bug of thread.join() will block signal
             # http://bugs.python.org/issue1167930


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I45effe0d787c36564a1ae3eb45844c22ac2a2c36
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Antoni Segura Puimedon <asegurap at redhat.com>


More information about the vdsm-patches mailing list