Change in vdsm[master]: multipath.getMPDevNamesIter() return /dev/mapper.

abaron at redhat.com abaron at redhat.com
Wed Aug 28 21:31:24 UTC 2013


Ayal Baron has posted comments on this change.

Change subject: multipath.getMPDevNamesIter() return /dev/mapper.
......................................................................


Patch Set 1:

(1 comment)

....................................................
File vdsm/storage/multipath.py
Line 380: 
Line 381: 
Line 382: def getMPDevNamesIter():
Line 383:     for _, name in getMPDevsIter():
Line 384:         yield os.path.join(devicemapper.DMPATH_PREFIX, name)
this is begging for a dmpath method: devicemapper.path(name)
Line 385: 
Line 386: 
Line 387: def getMPDevsIter():
Line 388:     """


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I1ec54147205992ac130684d01e73cd7aceccad48
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Elad Ben Aharon <eladba1990 at gmail.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list