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

danken at redhat.com danken at redhat.com
Fri Jan 3 00:09:08 UTC 2014


Dan Kenigsberg has submitted this change and it was merged.

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 it was
trying to do.

Change-Id: I5eb78b57b9ae6ada9948815020ac3d5ac005d20a
Signed-off-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/22718
Reviewed-by: Antoni Segura Puimedon <asegurap at redhat.com>
Reviewed-by: Ayal Baron <abaron at redhat.com>
Reviewed-by: Assaf Muller <amuller at redhat.com>
Reviewed-by: Yaniv Bronhaim <ybronhei at redhat.com>
Tested-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/supervdsmServer
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Ayal Baron: Looks good to me, approved
  Yaniv Bronhaim: Looks good to me, but someone else must approve
  Antoni Segura Puimedon: Looks good to me, but someone else must approve
  Assaf Muller: Looks good to me, but someone else must approve
  Dan Kenigsberg: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I5eb78b57b9ae6ada9948815020ac3d5ac005d20a
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Assaf Muller <amuller at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list