From Dan Kenigsberg <dan...@redhat.com>: Dan Kenigsberg has submitted this change and it was merged.
Change subject: yajsonrpc: remove bridge from Notification ...................................................................... yajsonrpc: remove bridge from Notification As part of adding notification support in JsonRpcClient, Notification class should stop getting bridge as a parameter to its constructor. Bridge is related to the server and not to the client, and since it only provides the event schema, it can be replaced with the event_schema parameter. Change-Id: I0b2982e6284ee161f06434b11d9d8e5fd54f4f50 Signed-off-by: Irit Goihman <igoih...@redhat.com> --- M lib/yajsonrpc/__init__.py M tests/integration/jsonRpcHelper.py M vdsm/clientIF.py 3 files changed, 7 insertions(+), 6 deletions(-) Approvals: Piotr Kliczewski: Looks good to me, approved Jenkins CI: Passed CI tests Irit Goihman: Verified -- To view, visit https://gerrit.ovirt.org/70739 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0b2982e6284ee161f06434b11d9d8e5fd54f4f50 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Irit Goihman <igoih...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Irit Goihman <igoih...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczew...@gmail.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> _______________________________________________ vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org
