Change in vdsm[ovirt-4.1]: stomp: handle timeout in server side
From Yaniv Bronhaim : Yaniv Bronhaim has submitted this change and it was merged. ( https://gerrit.ovirt.org/78917 ) Change subject: stomp: handle timeout in server side .. stomp: handle timeout in server side When the server stops receiving incoming heartbeats, it should cleanup subscriptions and close the fd. Signed-off-by: Irit Goihman Change-Id: Ic83ccb947771e79cb08d9d9fa3b1da64d074fda5 Bug-Url: https://bugzilla.redhat.com/1376843 --- M lib/yajsonrpc/stompreactor.py M tests/stompAdapterTests.py 2 files changed, 57 insertions(+), 4 deletions(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Yaniv Bronhaim: Looks good to me, approved; Passed CI tests Irit Goihman: Verified -- To view, visit https://gerrit.ovirt.org/78917 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic83ccb947771e79cb08d9d9fa3b1da64d074fda5 Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: ovirt-4.1 Gerrit-Owner: Irit Goihman Gerrit-Reviewer: Francesco Romani Gerrit-Reviewer: Irit Goihman Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer Gerrit-Reviewer: Piotr Kliczewski Gerrit-Reviewer: Yaniv Bronhaim Gerrit-Reviewer: gerrit-hooks ___ vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org
Change in vdsm[ovirt-4.1]: stomp: handle timeout in server side
From Yaniv Bronhaim : Yaniv Bronhaim has posted comments on this change. Change subject: stomp: handle timeout in server side .. Patch Set 4: Continuous-Integration+1 -- To view, visit https://gerrit.ovirt.org/78917 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic83ccb947771e79cb08d9d9fa3b1da64d074fda5 Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: ovirt-4.1 Gerrit-Owner: Irit Goihman Gerrit-Reviewer: Francesco Romani Gerrit-Reviewer: Irit Goihman Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer Gerrit-Reviewer: Piotr Kliczewski Gerrit-Reviewer: Yaniv Bronhaim Gerrit-Reviewer: gerrit-hooks Gerrit-HasComments: No ___ vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org
Change in vdsm[ovirt-4.1]: stomp: handle timeout in server side
From Yaniv Bronhaim : Yaniv Bronhaim has posted comments on this change. Change subject: stomp: handle timeout in server side .. Patch Set 3: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/78917 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic83ccb947771e79cb08d9d9fa3b1da64d074fda5 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: ovirt-4.1 Gerrit-Owner: Irit Goihman Gerrit-Reviewer: Francesco Romani Gerrit-Reviewer: Irit Goihman Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer Gerrit-Reviewer: Piotr Kliczewski Gerrit-Reviewer: Yaniv Bronhaim Gerrit-Reviewer: gerrit-hooks Gerrit-HasComments: No ___ vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org