Change in vdsm[master]: fix LV_ATTR_BITS for use with lvm 2.02.95(2)-RHEL6

ewarszaw at redhat.com ewarszaw at redhat.com
Sun Mar 18 13:50:41 UTC 2012


Eduardo has posted comments on this change.

Change subject: fix LV_ATTR_BITS for use with lvm 2.02.95(2)-RHEL6
......................................................................


Patch Set 1:

For sure we will not check lvm versions in vdsm.
Only the minimal lvm version requisite should be changed in the .spec.
We want the code to be "fragile" if the lvm output changes and   we should not try to parse partially unknown outputs.
This is the main reason for reject parsing outputs or errors.
Choping the output is not sufficient.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I634f84f9f816f4583adbe3c53c88d882a9b339c4
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Peter V. Saveliev <peet 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: Igor Lvovsky <ilvovsky at redhat.com>
Gerrit-Reviewer: Peter V. Saveliev <peet at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>


More information about the vdsm-patches mailing list