Change in vdsm[master]: Improve the lvm.USER_DEV_LIST parsing

danken at redhat.com danken at redhat.com
Fri Feb 17 21:58:54 UTC 2012


Dan Kenigsberg has submitted this change and it was merged.

Change subject: Improve the lvm.USER_DEV_LIST parsing
......................................................................


Improve the lvm.USER_DEV_LIST parsing

In this patch:
* Remove the ConfigParser.Error exception handling (impossible)
* Filter out empty strings

Change-Id: I8020b990f1b74b081a46de5e215dc27745466186
---
M vdsm/storage/lvm.py
1 file changed, 2 insertions(+), 5 deletions(-)

Approvals:
  Ayal Baron: Looks good to me, but someone else must approve
  Dan Kenigsberg: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I8020b990f1b74b081a46de5e215dc27745466186
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce 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: Saggi Mizrahi <smizrahi at redhat.com>


More information about the vdsm-patches mailing list