Change in vdsm[master]: Add lvm.lvPhysDev() to get the physical LV device.

ewarszaw at redhat.com ewarszaw at redhat.com
Wed Oct 24 08:18:13 UTC 2012


Eduardo has posted comments on this change.

Change subject: Add lvm.lvPhysDev() to get the physical LV device.
......................................................................


Patch Set 1: (1 inline comment)

....................................................
File vdsm/storage/lvm.py
Line 1159: def lvPhysDev(vgName, lvName):
Line 1160:     """Return the LV physical device.
Line 1161: 
Line 1162:     returns: dm-X
Line 1163:     The LV should be active.
f the LV is inactive there is no dm device. Calling this function will raise OSError.
Line 1164:     """
Line 1165: 
Line 1166:     lvp = lvPath(vgName, lvName)
Line 1167:     return os.path.basename(os.readlink(lvp))


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I6d3286589ea02025713d10d85a9892024f586b73
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Adam Litke <agl at us.ibm.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Mark Wu <wudxw at linux.vnet.ibm.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan at redhat.com>


More information about the vdsm-patches mailing list