Change in vdsm[master]: stomp: allow single stomp reactor

danken at redhat.com danken at redhat.com
Thu Jun 4 13:50:44 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: stomp: allow single stomp reactor
......................................................................


stomp: allow single stomp reactor

We could use several instances of stompReactor to create stomp client or
listener. Thanks to client and server side subscription implementation
we can use single instance.

Signed-off-by: pkliczewski <piotr.kliczewski at gmail.com>
Change-Id: Ifdd7afef991d51c293d5624d936f8e689ccb8de4
Reviewed-on: https://gerrit.ovirt.org/41070
Reviewed-by: Yaniv Bronhaim <ybronhei at redhat.com>
Reviewed-by: Yeela Kaplan <ykaplan at redhat.com>
Continuous-Integration: Jenkins CI
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M lib/yajsonrpc/stompreactor.py
M vdsm/rpc/bindingjsonrpc.py
2 files changed, 10 insertions(+), 12 deletions(-)

Approvals:
  Piotr Kliczewski: Verified
  Yeela Kaplan: Looks good to me, but someone else must approve
  Yaniv Bronhaim: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ifdd7afef991d51c293d5624d936f8e689ccb8de4
Gerrit-PatchSet: 11
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Dima Kuznetsov <dkuznets at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list