Dan Kenigsberg has submitted this change and it was merged. Change subject: Refactor communication infra ......................................................................
Refactor communication infra This is one phase in many, there is still a lot to do but it's a good step in the right direction. As I said, this is far from complete but it's enough code for a commit IMO. - Properly layer code to allow multiple transports - Isolate json-rpc logic to the actual json-rpc layer - Someone more conferment but still incomplete json-rpc 2.0 implementation - Make sure tcp handling uses one thread - Write some more tests Change-Id: I7a65364824cc8433263ccf6ed44f1f97141eff2c Signed-off-by: Saggi Mizrahi <[email protected]> --- M configure.ac M tests/Makefile.am M tests/apiTests.py A tests/jsonRpcTests.py M tests/testrunner.py M vdsm.spec.in M vdsm/clientIF.py M vdsm/utils.py M vdsm_api/BindingJsonRpc.py M vdsm_api/Bridge.py M vdsm_api/Makefile.am A vdsm_api/jsonrpc/Makefile.am A vdsm_api/jsonrpc/__init__.py A vdsm_api/jsonrpc/tcpReactor.py 14 files changed, 786 insertions(+), 106 deletions(-) Approvals: Dan Kenigsberg: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9986 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7a65364824cc8433263ccf6ed44f1f97141eff2c Gerrit-PatchSet: 18 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Barak Azulay <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: Vinzenz Feenstra <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
