Change in vdsm[ovirt-3.5-gluster]: gluster: ensure LV size is multiple of VG's PE value

sabose at redhat.com sabose at redhat.com
Tue Dec 22 05:46:12 UTC 2015


Sahina Bose 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 <rnachimu at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/50802
Reviewed-by: Sahina Bose <sabose at redhat.com>
Reviewed-by: Nir Soffer <nsoffer at redhat.com>
Continuous-Integration: Nir Soffer <nsoffer at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/50809
Continuous-Integration: Sahina Bose <sabose at redhat.com>
---
M vdsm/gluster/storagedev.py
1 file changed, 3 insertions(+), 5 deletions(-)

Approvals:
  Nir Soffer: Looks good to me, but someone else must approve
  Sahina Bose: Looks good to me, approved; Passed CI tests
  Ramesh N: Verified



-- 
To view, visit https://gerrit.ovirt.org/50809
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I3a1609544db36b16b5b8fc0a52d5ef04e770c18b
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5-gluster
Gerrit-Owner: Ramesh N <rnachimu at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Ramesh N <rnachimu at redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the vdsm-patches mailing list