Yaniv Bronhaim 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 <rnach...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/50802 Reviewed-by: Sahina Bose <sab...@redhat.com> Reviewed-by: Nir Soffer <nsof...@redhat.com> Continuous-Integration: Nir Soffer <nsof...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/50854 Reviewed-by: Yaniv Bronhaim <ybron...@redhat.com> Continuous-Integration: Yaniv Bronhaim <ybron...@redhat.com> --- M vdsm/gluster/storagedev.py 1 file changed, 3 insertions(+), 5 deletions(-) Approvals: Yaniv Bronhaim: Looks good to me, approved; Passed CI tests Sahina Bose: Looks good to me, but someone else must approve Ramesh N: Verified -- To view, visit https://gerrit.ovirt.org/50854 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3a1609544db36b16b5b8fc0a52d5ef04e770c18b Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Ramesh N <rnach...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Ramesh N <rnach...@redhat.com> Gerrit-Reviewer: Sahina Bose <sab...@redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
