Change in vdsm[master]: Handle spaces in device names using lvm filter

ewarszaw at redhat.com ewarszaw at redhat.com
Mon Nov 19 09:40:07 UTC 2012


Eduardo has posted comments on this change.

Change subject: Handle spaces in device names using lvm filter
......................................................................


Patch Set 3: Looks good to me, but someone else must approve

Since this patch is working and fulfils the requirements of the BZ we can put it in.
Anyway I think that a further patch should take care of unifying all the device name instances. The internal representation of the name in vdsm should include spaces and not escape characters. This should be done decoding the input of the multipath names (GUIDs).

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I1efb6dfd30c0999f14c413fff778213b247a178f
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yeela Kaplan <ykaplan at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>
Gerrit-Reviewer: Mark Wu <wudxw at linux.vnet.ibm.com>
Gerrit-Reviewer: Shu Ming <shuming at linux.vnet.ibm.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan at redhat.com>


More information about the vdsm-patches mailing list