Sahina Bose has submitted this change and it was merged. Change subject: gluster: fix alignment issue for brick creation on JBOD ......................................................................
gluster: fix alignment issue for brick creation on JBOD Chunk size and alignment value should be 256KB for JBOD use case. So assigining these values incase of JBOD. Note: blivet doesn't support alignment values in PV,VG, Thinpool creation. So we were using blivet API only when i we don't have any alignment and LVM commands directly when we have alignment values. Now, with this patch we will use lvm commands always as we have alignment values in all the cases and Blivet API usage can be removed completely in PV, Thin pool, VG creation. Bug-Url: https://bugzilla.redhat.com/1270792 Signed-off-by: Ramesh Nachimuthu <[email protected]> Change-Id: I58cc322cb5140de2d2006d59b4c1dceaba2e5968 Signed-off-by: Timothy Asir Jeyasingh <[email protected]> Reviewed-on: https://gerrit.ovirt.org/47959 Continuous-Integration: Jenkins CI Reviewed-by: Sahina Bose <[email protected]> Reviewed-by: Nir Soffer <[email protected]> Reviewed-on: https://gerrit.ovirt.org/50806 Continuous-Integration: Sahina Bose <[email protected]> --- M vdsm/gluster/storagedev.py 1 file changed, 72 insertions(+), 98 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/50806 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I58cc322cb5140de2d2006d59b4c1dceaba2e5968 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.5-gluster Gerrit-Owner: Ramesh N <[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
