Discussion:
Change in vdsm[ovirt-4.1]: stomp: introduce restore_subscriptions in client
Code Review
2017-08-29 12:56:17 UTC
Permalink
From Yaniv Bronhaim <***@redhat.com>:

Yaniv Bronhaim has posted comments on this change.

Change subject: stomp: introduce restore_subscriptions in client
......................................................................


Patch Set 4: Code-Review+2

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I9815faf5314c3b932067576ef044cf3bee0869e9
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.1
Gerrit-Owner: Irit Goihman <***@redhat.com>
Gerrit-Reviewer: Irit Goihman <***@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Piotr Kliczewski <***@gmail.com>
Gerrit-Reviewer: Yaniv Bronhaim <***@redhat.com>
Gerrit-Reviewer: gerrit-hooks <***@ovirt.org>
Gerrit-HasComments: No
_______________________________________________
vdsm-patches mailing list -- vdsm-***@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches
Code Review
2017-08-29 12:56:25 UTC
Permalink
From Yaniv Bronhaim <***@redhat.com>:

Yaniv Bronhaim has submitted this change and it was merged. ( https://gerrit.ovirt.org/78919 )

Change subject: stomp: introduce restore_subscriptions in client
......................................................................


stomp: introduce restore_subscriptions in client

restore_subscriptions is required for persisting subscriptions
after client reconnects to server.

Change-Id: I9815faf5314c3b932067576ef044cf3bee0869e9
Signed-off-by: Irit Goihman <***@redhat.com>
Bug-Url: https://bugzilla.redhat.com/1376843
---
M lib/yajsonrpc/stomp.py
M tests/stompAsyncClientTests.py
2 files changed, 19 insertions(+), 0 deletions(-)

Approvals:
Piotr Kliczewski: Looks good to me, but someone else must approve
Yaniv Bronhaim: Looks good to me, approved
Jenkins CI: Passed CI tests
Irit Goihman: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I9815faf5314c3b932067576ef044cf3bee0869e9
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.1
Gerrit-Owner: Irit Goihman <***@redhat.com>
Gerrit-Reviewer: Irit Goihman <***@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Piotr Kliczewski <***@gmail.com>
Gerrit-Reviewer: Yaniv Bronhaim <***@redhat.com>
Gerrit-Reviewer: gerrit-hooks <***@ovirt.org>
_______________________________________________
vdsm-patches mailing list -- vdsm-***@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-***@lists.fedorahosted
Loading...