Dan Kenigsberg has submitted this change and it was merged. Change subject: API: Separate clientIF and the xmlrpc server ......................................................................
API: Separate clientIF and the xmlrpc server Currently the xmlrpc server functionality is entwined with the clientIF class. In order to proceed with the API bridge plan (http://ovirt.org/wiki/Vdsm_API) I would like to separate the xmlrpc server into its own class. This brings clientIF one step closer to becoming a generic API Bridge that is able to host additional bindings such as REST and QMF. Despite the size of this patch, this was a pretty simple refactoring. I made the following choices as I split this out: - The xmlrpc server binding class has start() and prepareForShutdown() APIs - clientIF will still own the storage dispatcher but API wrapper mechanisms have moved into the xmlrpc class. Change-Id: I5026e11f3d39343c957b59606e90c23acbb598ba Signed-off-by: Adam Litke <[email protected]> --- M vdsm/clientIF.py M vdsm/vm.py 2 files changed, 167 insertions(+), 112 deletions(-) Approvals: Saggi Mizrahi: Verified; Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/620 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5026e11f3d39343c957b59606e90c23acbb598ba Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Igor Lvovsky <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://fedorahosted.org/mailman/listinfo/vdsm-patches
