Ramesh N has uploaded a new change for review.
Change subject: gluster: remove clientIf frmo GlusterApi
......................................................................
gluster: remove clientIf frmo GlusterApi
Currently clientIf is initialized in GlusterApi but it not used
any where. So removing clientIf from GlusterApi
Change-Id: I2f795a9d0aea8ea4478666f2a59b48cd3e0b7c98
Signed-off-by: Ramesh Nachimuthu <rnach...@redhat.com>
---
M vdsm/clientIF.py
M vdsm/gluster/api.py
M vdsm/gluster/apiwrapper.py
3 files changed, 3 insertions(+), 7 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/29/59929/1
diff --git a/vdsm/clientIF.py b/vdsm/clientIF.py
index 1147e36..c8ae50b 100644
--- a/vdsm/clientIF.py
+++ b/vdsm/clientIF.py
@@ -94,7 +94,7 @@
self._subscriptions = defaultdict(list)
self._scheduler = scheduler
if _glusterEnabled:
- self.gluster = gapi.GlusterApi(self, log)
+ self.gluster = gapi.GlusterApi()
else:
self.gluster = None
try:
diff --git a/vdsm/gluster/api.py b/vdsm/gluster/api.py
index b6f11c3..716448a 100644
--- a/vdsm/gluster/api.py
+++ b/vdsm/gluster/api.py
@@ -287,9 +287,7 @@
"""
- def __init__(self, cif, log):
- self.cif = cif
- self.log = log
+ def __init__(self):
self.svdsmProxy = svdsm.getProxy()
@exportAsVerb
diff --git a/vdsm/gluster/apiwrapper.py b/vdsm/gluster/apiwrapper.py
index 08d2140..73c29a0 100644
--- a/vdsm/gluster/apiwrapper.py
+++ b/vdsm/gluster/apiwrapper.py
@@ -17,7 +17,6 @@
#
# Refer to the README and COPYING files for full details of the license
#
-from clientIF import clientIF
from gluster.api import GlusterApi, META_VOLUME
@@ -25,8 +24,7 @@
ctorArgs = []
def __init__(self):
- self._cif = clientIF.getInstance()
- self._gluster = GlusterApi(self._cif, self._cif.log)
+ self._gluster = GlusterApi()
class GlusterHook(GlusterApiBase):
--
To view, visit https://gerrit.ovirt.org/59929
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I2f795a9d0aea8ea4478666f2a59b48cd3e0b7c98
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ramesh N <rnach...@redhat.com>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org
