Change in vdsm[master]: Override getLocalPathBase in GlusterFSConnection

deepakcs at linux.vnet.ibm.com deepakcs at linux.vnet.ibm.com
Sun Feb 17 04:03:40 UTC 2013


Deepak C Shetty has uploaded a new change for review.

Change subject: Override getLocalPathBase in GlusterFSConnection
......................................................................

Override getLocalPathBase in GlusterFSConnection

Override getLocalPathBase in GlusterFSConnection as a better
way to specify mount point for GlusterSD

Change-Id: I8d7d0980b0a863f8d47d61936b2d8fdab71a6a0c
Signed-off-by: Deepak C Shetty <deepakcs at linux.vnet.ibm.com>
---
M vdsm/storage/storageServer.py
1 file changed, 2 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/27/12127/1

diff --git a/vdsm/storage/storageServer.py b/vdsm/storage/storageServer.py
index 94b56a9..d7fa8af 100644
--- a/vdsm/storage/storageServer.py
+++ b/vdsm/storage/storageServer.py
@@ -251,11 +251,8 @@
 
 class GlusterFSConnection(MountConnection):
 
-    # TODO: Can this be made more cleaner, by avoiding this override ?
-    def _getLocalPath(self):
-        return os.path.join(
-            self.localPathBase, "glusterSD",
-            self._remotePath.replace("_", "__").replace("/", "_"))
+    def getLocalPathBase(cls):
+        return os.path.join(MountConnection.getLocalPathBase(), "glusterSD")
 
 
 class NFSConnection(object):


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8d7d0980b0a863f8d47d61936b2d8fdab71a6a0c
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Deepak C Shetty <deepakcs at linux.vnet.ibm.com>


More information about the vdsm-patches mailing list