Dan Kenigsberg has submitted this change and it was merged. Change subject: vdsm: lastclient info in jsonrpc ......................................................................
vdsm: lastclient info in jsonrpc In xml binding there is a hack for getCapabilites which before sending response adds listclient and lastClientIface. We can not modify API.py to provide this information in a common place because it is meaningless for mom thread , or any other in-process API consumer. We provide similar hack for jsonrpc at the level where we have enough information to send back required information. Bug-Url: https://bugzilla.redhat.com/1099500 Change-Id: I402e15cb05f89a98dab14491d9da5985335e095e Signed-off-by: pkliczewski <[email protected]> Reviewed-on: http://gerrit.ovirt.org/28817 Reviewed-by: Saggi Mizrahi <[email protected]> --- M lib/yajsonrpc/__init__.py M lib/yajsonrpc/stompReactor.py M tests/bridgeTests.py M tests/jsonRpcTests.py M vdsm/rpc/BindingJsonRpc.py M vdsm/rpc/Bridge.py 6 files changed, 46 insertions(+), 11 deletions(-) Approvals: Piotr Kliczewski: Verified Saggi Mizrahi: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28817 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I402e15cb05f89a98dab14491d9da5985335e095e Gerrit-PatchSet: 22 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]> Gerrit-Reviewer: Barak Azulay <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Oved Ourfali <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
