Bug 1003851

Summary: DHT - after adding brick/s, layout for root directory is getting changed (without running rebalance) so files created on mount point after adding bricks are distributed to new brick/sub-volume also
Product: [Community] GlusterFS Reporter: shishir gowda <sgowda>
Component: distributeAssignee: Nagaprasad Sathyanarayana <nsathyan>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: high Docs Contact:
Priority: medium    
Version: mainlineCC: gluster-bugs, nsathyan, racpatel, rhs-bugs, smohan, vbellur
Target Milestone: ---   
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: glusterfs-3.4.3 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: 950314 Environment:
Last Closed: 2014-04-17 13:14:11 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 921408, 950314    
Bug Blocks:    

Comment 1 Anand Avati 2013-09-03 11:28:23 UTC
REVIEW: http://review.gluster.org/5762 (cluster/dht: Fix anomaly check) posted (#2) for review on master by Shishir Gowda (sgowda)

Comment 2 Anand Avati 2013-09-10 06:14:27 UTC
REVIEW: http://review.gluster.org/5762 (cluster/dht: Fix anomaly check) posted (#3) for review on master by Shishir Gowda (sgowda)

Comment 3 Anand Avati 2013-09-23 18:41:18 UTC
COMMIT: http://review.gluster.org/5762 committed in master by Anand Avati (avati) 
------
commit 711484d759cedbd77e449d306f7e31d596e6ae0d
Author: shishir gowda <sgowda>
Date:   Tue Sep 3 09:13:21 2013 +0000

    cluster/dht: Fix anomaly check
    
    We were wrongly detecting holes/overlaps for already accounted
    errors. Additionally, sort should also handle zero'ed out layout
    
    Change-Id: Ic3d13e1d735b914f9acc01fe919bc90656baea48
    BUG: 1003851
    Signed-off-by: shishir gowda <sgowda>
    Reviewed-on: http://review.gluster.org/5762
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Amar Tumballi <amarts>
    Reviewed-by: Anand Avati <avati>

Comment 6 Anand Avati 2013-12-10 09:20:22 UTC
REVIEW: http://review.gluster.org/6469 (cluster/dht: Fix anomaly check) posted (#1) for review on release-3.4 by Shishir Gowda (gowda.shishir)

Comment 7 Anand Avati 2013-12-14 14:25:29 UTC
COMMIT: http://review.gluster.org/6469 committed in release-3.4 by Vijay Bellur (vbellur) 
------
commit 790c2813ef4898b28ebae8545c8d891ac4114bf1
Author: shishir gowda <sgowda>
Date:   Tue Sep 3 09:13:21 2013 +0000

    cluster/dht: Fix anomaly check
    
    We were wrongly detecting holes/overlaps for already accounted
    errors. Additionally, sort should also handle zero'ed out layout
    
    Change-Id: Ic3d13e1d735b914f9acc01fe919bc90656baea48
    BUG: 1003851
    Signed-off-by: shishir gowda <sgowda>
    Reviewed-on: http://review.gluster.org/5762
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Amar Tumballi <amarts>
    Reviewed-by: Anand Avati <avati>
    Reviewed-on: http://review.gluster.org/6469
    Reviewed-by: Vijay Bellur <vbellur>

Comment 8 Niels de Vos 2014-04-17 13:14:11 UTC
This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.4.3, please reopen this bug report.

glusterfs-3.4.3 has been announced on the Gluster Developers mailinglist [1], packages for several distributions should already be or become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution.

The fix for this bug likely to be included in all future GlusterFS releases i.e. release > 3.4.3. In the same line the recent release i.e. glusterfs-3.5.0 [3] likely to have the fix. You can verify this by reading the comments in this bug report and checking for comments mentioning "committed in release-3.5".

[1] http://thread.gmane.org/gmane.comp.file-systems.gluster.devel/5978
[2] http://news.gmane.org/gmane.comp.file-systems.gluster.user
[3] http://thread.gmane.org/gmane.comp.file-systems.gluster.devel/6137