Yaniv Bronhaim has submitted this change and it was merged.

Change subject: gluster: fix xfs filesystem is created with wrong inode size
......................................................................


gluster: fix xfs filesystem is created with wrong inode size

Currently brick create function creates an xfs filesystem with
inode size of 256 for brick which uses raid0 or JBOD devices.

This patch uses the striping parameters "-d su=stripe_unit_size,
sw=stripe_width_in_number_of_disks" during xfs file system
create for RAID6. For RAID 10, JBOD and other devices it omits
the "-d su=<>,sw=<>" option and set only the inode size to 512
and the directory block size to 8192.

Change-Id: If3f350d2f79172705a93a9d69fe1b7b78867861e
Bug-Url: https://bugzilla.redhat.com/1293537
Signed-off-by: Timothy Asir Jeyasingh <tjeya...@redhat.com>
Reviewed-on: https://gerrit.ovirt.org/43847
Reviewed-by: Bala.FA <barum...@redhat.com>
Reviewed-by: Manoj Pillai <mpil...@redhat.com>
Continuous-Integration: Jenkins CI
Reviewed-by: Dan Kenigsberg <dan...@redhat.com>
Reviewed-on: https://gerrit.ovirt.org/50849
Tested-by: Ramesh N <rnach...@redhat.com>
Reviewed-by: Sahina Bose <sab...@redhat.com>
Reviewed-by: Yaniv Bronhaim <ybron...@redhat.com>
---
M vdsm/gluster/storagedev.py
1 file changed, 15 insertions(+), 5 deletions(-)

Approvals:
  Yaniv Bronhaim: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Sahina Bose: Looks good to me, but someone else must approve
  Ramesh N: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: If3f350d2f79172705a93a9d69fe1b7b78867861e
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.6
Gerrit-Owner: Ramesh N <rnach...@redhat.com>
Gerrit-Reviewer: Bala.FA <barum...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Manoj Pillai <mpil...@redhat.com>
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: Ramesh N <rnach...@redhat.com>
Gerrit-Reviewer: Sahina Bose <sab...@redhat.com>
Gerrit-Reviewer: Timothy Asir <tjeya...@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

Reply via email to