Change in vdsm[ovirt-4.1]: stomp: handle timeout in server side

2017-08-23 Thread Code Review
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

2017-08-23 Thread Code Review
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

2017-08-23 Thread Code Review
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