Nir Soffer has submitted this change and it was merged. Change subject: gluster: ensure LV size is multiple of VG's PE value ......................................................................
gluster: ensure LV size is multiple of VG's PE value Alignment value for JBOD is changed to 256 KiB from 0 in https://gerrit.ovirt.org/#/c/47959 and alignment value is used as PE value in VG creation. It resulted in following error for some JBOD disks. "DeviceError: ('Adding new lv is too large to fit in free space', u'vg-brick2-pool-brick2'" Size of LV should be multiple of PE value. But this calculation was done only for RAID volume not for JBOD and that was the casue for above error. With this patch, lv size will be adjusted to the multiples of PE always regardless of whether its RAID or JBOD. Change-Id: I3a1609544db36b16b5b8fc0a52d5ef04e770c18b Bug-Url: https://bugzilla.redhat.com/1270792 Signed-off-by: Ramesh Nachimuthu <[email protected]> Reviewed-on: https://gerrit.ovirt.org/50802 Reviewed-by: Sahina Bose <[email protected]> Reviewed-by: Nir Soffer <[email protected]> Continuous-Integration: Nir Soffer <[email protected]> --- M vdsm/gluster/storagedev.py 1 file changed, 3 insertions(+), 5 deletions(-) Approvals: Nir Soffer: Looks good to me, approved; Passed CI tests Sahina Bose: Looks good to me, but someone else must approve Ramesh N: Verified Objections: Jenkins CI: Failed CI tests -- To view, visit https://gerrit.ovirt.org/50802 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3a1609544db36b16b5b8fc0a52d5ef04e770c18b Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Ramesh N <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Manoj Pillai <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Ramesh N <[email protected]> Gerrit-Reviewer: Sahina Bose <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
