Change in vdsm[master]: lvm: Make the isActiveLV() method public.

lyarwood at redhat.com lyarwood at redhat.com
Mon Nov 18 16:40:25 UTC 2013


Lee Yarwood has uploaded a new change for review.

Change subject: lvm: Make the isActiveLV() method public.
......................................................................

lvm: Make the isActiveLV() method public.

This change makes the private isActiveLV() method public for
use in the blockSD layer.

Change-Id: I670fe3969b88819891c64ea3ac487b070618aaec
Signed-off-by: Lee Yarwood <lyarwood at redhat.com>
---
M vdsm/storage/lvm.py
1 file changed, 1 insertion(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/86/21386/1

diff --git a/vdsm/storage/lvm.py b/vdsm/storage/lvm.py
index 112f1c0..76184b8 100644
--- a/vdsm/storage/lvm.py
+++ b/vdsm/storage/lvm.py
@@ -1221,10 +1221,8 @@
     return os.path.basename(os.readlink(lvp))
 
 
-def _isLVActive(vgName, lvName):
+def isLVActive(vgName, lvName):
     """Active volumes have a mp link.
-
-    This function should not be used out of this module.
     """
     return os.path.exists(lvPath(vgName, lvName))
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I670fe3969b88819891c64ea3ac487b070618aaec
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Lee Yarwood <lyarwood at redhat.com>


More information about the vdsm-patches mailing list