Change in vdsm[master]: Remove the parameter of clientIF instance from API

danken at redhat.com danken at redhat.com
Thu Jun 21 10:26:53 UTC 2012


Dan Kenigsberg has submitted this change and it was merged.

Change subject: Remove the parameter of clientIF instance from API
......................................................................


Remove the parameter of clientIF instance from API

Currently, the API clients(like the xmlrpc binding and rest binding)
need hold the vdsm internal clientIF instance to call API. To make the
code more modular, we create a single API instance intialized with
clientIF instance. And then the API clients can interact with vdsm
without having the vdsm internal clientIF instance.

Change-Id: I9dd1c555d066c20bd13da6db569fa709772c74ef
Signed-off-by: Mark Wu <wudxw at linux.vnet.ibm.com>
---
M vdsm/API.py
M vdsm/BindingXMLRPC.py
M vdsm/clientIF.py
M vdsm/vdsm
4 files changed, 170 insertions(+), 151 deletions(-)

Approvals:
  Mark Wu: Verified
  Dan Kenigsberg: Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I9dd1c555d066c20bd13da6db569fa709772c74ef
Gerrit-PatchSet: 9
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Mark Wu <wudxw at linux.vnet.ibm.com>
Gerrit-Reviewer: Adam Litke <agl at us.ibm.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Mark Wu <wudxw at linux.vnet.ibm.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Xu He Jie <xuhj at linux.vnet.ibm.com>


More information about the vdsm-patches mailing list