Change in vdsm[master]: clientIF: Move wrapApiMethod into bindingXMLRPC class

Adam Litke has uploaded a new change for review.

Change subject: clientIF: Move wrapApiMethod into bindingXMLRPC class
......................................................................

clientIF: Move wrapApiMethod into bindingXMLRPC class

wrapApiMethod is now only used by bindingXMLRPC so make it a class member
instead of a global function.

Change-Id: Icfd8da9a0e072da192f4c3f86c05d0fa1c305f48
Signed-off-by: Adam Litke <agl(a)us.ibm.com&gt;
---
M vdsm/clientIF.py
1 file changed, 33 insertions(+), 33 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/21/621/1
--
To view, visit http://gerrit.ovirt.org/621
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Icfd8da9a0e072da192f4c3f86c05d0fa1c305f48
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Adam Litke <agl(a)us.ibm.com&gt;

Adam Litke has abandoned this change.

Change subject: clientIF: Move wrapApiMethod into bindingXMLRPC class
......................................................................


Patch Set 1: Abandoned

Oops.  This seems to be an old patch that I forgot to abandon.

--
To view, visit http://gerrit.ovirt.org/621
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: abandon
Gerrit-Change-Id: Icfd8da9a0e072da192f4c3f86c05d0fa1c305f48
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Adam Litke <agl(a)us.ibm.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;

[Dan Kenigsberg]

Dan Kenigsberg has posted comments on this change.

Change subject: clientIF: Move wrapApiMethod into bindingXMLRPC class
......................................................................


Patch Set 1: I would prefer that you didn't submit this

why whould the api wrapper be applicable to xmlrpc only? I would assume that you would
need to log api calls, and catch unhandled exceptions, in any binding.

--
To view, visit http://gerrit.ovirt.org/621
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Icfd8da9a0e072da192f4c3f86c05d0fa1c305f48
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Adam Litke <agl(a)us.ibm.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;