Dan Kenigsberg has submitted this change and it was merged.

Change subject: migration: usage of single reactor in vdsm
......................................................................


migration: usage of single reactor in vdsm

In most of the cases for each standalone client there is reactor but
we want to share one in vdsm. There are already too many threads in vdsm
so it is not desired to add more.

With this change by default client owns a reactor and it will manage its
life cycle. For internal clients we let vdsm to handle its life cycle
because we want to share it.


Backport-To: 4.0
Change-Id: I06e21151e3e9f9e7da9e178bb0199c07f269ae8d
Signed-off-by: Piotr Kliczewski <[email protected]>
Bug-Url: https://bugzilla.redhat.com/1349461
Reviewed-on: https://gerrit.ovirt.org/59720
Continuous-Integration: Jenkins CI
Tested-by: Milan Zamazal <[email protected]>
Reviewed-by: Francesco Romani <[email protected]>
Reviewed-by: Nir Soffer <[email protected]>
Reviewed-by: Michal Skrivanek <[email protected]>
Reviewed-by: Dan Kenigsberg <[email protected]>
---
M lib/yajsonrpc/stompreactor.py
1 file changed, 13 insertions(+), 4 deletions(-)

Approvals:
  Nir Soffer: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Looks good to me, approved
  Francesco Romani: Looks good to me, but someone else must approve
  Michal Skrivanek: Looks good to me, but someone else must approve
  Milan Zamazal: Verified



-- 
To view, visit https://gerrit.ovirt.org/59720
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I06e21151e3e9f9e7da9e178bb0199c07f269ae8d
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Piotr Kliczewski <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin PeÅ™ina <[email protected]>
Gerrit-Reviewer: Michal Skrivanek <[email protected]>
Gerrit-Reviewer: Milan Zamazal <[email protected]>
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
Gerrit-Reviewer: Piotr Kliczewski <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: gerrit-hooks <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/admin/lists/[email protected]

Reply via email to