Bug 1161518

Summary: DHT + rebalance :- skipped file count is always 'zero' even though rebalance has skipped many files .
Product: [Community] GlusterFS Reporter: Nithya Balachandran <nbalacha>
Component: distributeAssignee: Nithya Balachandran <nbalacha>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: medium Docs Contact:
Priority: medium    
Version: mainlineCC: bugs, gluster-bugs, mzywusko, nbalacha, nsathyan, psriniva, sharne, ssamanta, storage-qa-internal, surs, vagarwal
Target Milestone: ---Keywords: ZStream
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: glusterfs-3.7.0 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: 1140517 Environment:
Last Closed: 2015-05-15 17:09:00 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: 1140517, 1168552    
Bug Blocks: 1087818    

Comment 2 Anand Avati 2014-11-07 10:04:41 UTC
REVIEW: http://review.gluster.org/9070 (Cluster/DHT : Rebalance skipped file count fix) posted (#1) for review on master by N Balachandran (nbalacha)

Comment 3 Anand Avati 2014-11-11 06:25:45 UTC
REVIEW: http://review.gluster.org/9070 (Cluster/DHT : Rebalance skipped file count fix) posted (#2) for review on master by N Balachandran (nbalacha)

Comment 4 Anand Avati 2014-11-11 16:37:43 UTC
COMMIT: http://review.gluster.org/9070 committed in master by Vijay Bellur (vbellur) 
------
commit a24a354737577d466ed1c29be1c4fa6bf1a31e11
Author: Nithya Balachandran <nbalacha>
Date:   Fri Nov 7 15:29:45 2014 +0530

    Cluster/DHT : Rebalance skipped file count fix
    
    The return value in dht_migrate_file is used to indicate the status
    of the file migration. This value was being masked by the lock operation
    causing the skipped and failure file counts to be incorrectly calculated.
    
    Change-Id: Ice3d2f5d57766e18aa52659f22a76867d188dc65
    BUG: 1161518
    Signed-off-by: Nithya Balachandran <nbalacha>
    Reviewed-on: http://review.gluster.org/9070
    Reviewed-by: susant palai <spalai>
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Shyamsundar Ranganathan <srangana>
    Reviewed-by: Vijay Bellur <vbellur>

Comment 5 Niels de Vos 2015-05-15 17:09:00 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.7.0, please open a new bug report.

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

[1] http://thread.gmane.org/gmane.comp.file-systems.gluster.devel/10939
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user