Change in vdsm[master]: gluster: fix pyflakes error in mount point verification

tjeyasin at redhat.com tjeyasin at redhat.com
Mon May 18 07:36:32 UTC 2015


Timothy Asir has uploaded a new change for review.

Change subject: gluster: fix pyflakes error in mount point verification
......................................................................

gluster: fix pyflakes error in mount point verification

Change-Id: Id4ad096e0c8cb470311d9f219e7469b16fd697de
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1209475
Signed-off-by: Timothy Asir Jeyasingh <tjeyasin at redhat.com>
---
M vdsm/gluster/exception.py
M vdsm/gluster/storagedev.py
2 files changed, 2 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/46/41046/1

diff --git a/vdsm/gluster/exception.py b/vdsm/gluster/exception.py
index 2fa1e88..238c561 100644
--- a/vdsm/gluster/exception.py
+++ b/vdsm/gluster/exception.py
@@ -470,10 +470,7 @@
 class GlusterHostStorageMountPointInUseException(GlusterHostException):
     code = 4517
 
-    def __init__(self, mountPoint, rc=0, out=(), err=()):
-        self.rc = rc
-        self.out = out
-        self.err = err
+    def __init__(self, mountPoint):
         self.message = "Mount point %s is in use" % mountPoint
 
 
diff --git a/vdsm/gluster/storagedev.py b/vdsm/gluster/storagedev.py
index 7754051..db2ffec 100644
--- a/vdsm/gluster/storagedev.py
+++ b/vdsm/gluster/storagedev.py
@@ -217,8 +217,7 @@
             return blivetEnv.devicetree.getDeviceByName(poolLv.name)
 
     if os.path.ismount(mountPoint):
-        raise ge.GlusterHostStorageMountPointInUseException(
-            mountPoint, rc, out, err)
+        raise ge.GlusterHostStorageMountPointInUseException(mountPoint)
 
     vgName = "vg-" + brickName
     poolName = "pool-" + brickName


-- 
To view, visit https://gerrit.ovirt.org/41046
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id4ad096e0c8cb470311d9f219e7469b16fd697de
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Timothy Asir <tjeyasin at redhat.com>


More information about the vdsm-patches mailing list