Bug 1251346

Summary: statfs giving incorrect values for AFR arbiter volumes
Product: [Community] GlusterFS Reporter: Ravishankar N <ravishankar>
Component: replicateAssignee: Ravishankar N <ravishankar>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: mainlineCC: bugs, gluster-bugs
Target Milestone: ---Keywords: Reopened
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: glusterfs-3.8rc2 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 1251380 (view as bug list) Environment:
Last Closed: 2016-06-16 13:29:42 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:    
Bug Blocks: 1251380    

Description Ravishankar N 2015-08-07 04:38:19 UTC
Description of problem:

AFR serves statfs from the brick having the least free space available. Since the size to be allocated to the arbiter brick in a 3 way replica is supposed to be considerably lesser than the other 2 bricks, statfs will always be served from this brick which is incorrect.

Comment 1 Ravishankar N 2015-08-07 04:39:43 UTC
The problem was reported by Frederik Brandt on gluster-users: http://www.gluster.org/pipermail/gluster-users/2015-August/023026.html

Comment 2 Anand Avati 2015-08-07 05:01:58 UTC
REVIEW: http://review.gluster.org/11857 (afr: Do not wind statfs to arbiter brick) posted (#1) for review on master by Ravishankar N (ravishankar)

Comment 3 Anand Avati 2015-08-07 07:03:13 UTC
REVIEW: http://review.gluster.org/11857 (afr: Do not wind statfs to arbiter brick) posted (#2) for review on master by Ravishankar N (ravishankar)

Comment 4 Anand Avati 2015-08-07 11:49:59 UTC
COMMIT: http://review.gluster.org/11857 committed in master by Niels de Vos (ndevos) 
------
commit 2a3f0fb035cf677d058a920d2fc1931a18e565ec
Author: Ravishankar N <ravishankar>
Date:   Fri Aug 7 10:10:54 2015 +0530

    afr: Do not wind statfs to arbiter brick
    
    Problem: AFR serves statfs from the brick having the least free space
    available. Since the size to be allocated to the arbiter brick in a 3
    way replica is supposed to be considerably lesser than the other 2
    bricks, statfs will be served from this brick which is incorrect.
    
    Fix: Don't serve statfs from the arbiter brick.
    
    Change-Id: I5af098b9c50626f52cf3d7dbb060bf754c797f05
    BUG: 1251346
    Reported-by: Fredrik Brandt <fredrikb>
    Signed-off-by: Ravishankar N <ravishankar>
    Reviewed-on: http://review.gluster.org/11857
    Reviewed-by: Pranith Kumar Karampuri <pkarampu>
    Tested-by: Gluster Build System <jenkins.com>
    Tested-by: NetBSD Build System <jenkins.org>

Comment 5 Anand Avati 2015-08-07 12:06:21 UTC
REVIEW: http://review.gluster.org/11864 (tests: add basic/afr/arbiter-statfs.t) posted (#1) for review on master by Ravishankar N (ravishankar)

Comment 6 Ravishankar N 2015-08-07 12:09:25 UTC
ToDo: Move BZ state to MODIFIED after patch in comment #5 gets merged.

Comment 7 Anand Avati 2015-08-07 12:36:26 UTC
REVIEW: http://review.gluster.org/11864 (tests: add basic/afr/arbiter-statfs.t) posted (#2) for review on master by Ravishankar N (ravishankar)

Comment 8 Anand Avati 2015-08-11 00:23:45 UTC
REVIEW: http://review.gluster.org/11864 (tests: add basic/afr/arbiter-statfs.t) posted (#4) for review on master by Ravishankar N (ravishankar)

Comment 9 Anand Avati 2015-08-11 13:35:27 UTC
COMMIT: http://review.gluster.org/11864 committed in master by Raghavendra Talur (rtalur) 
------
commit f550efa2879c02dec4c8f6876a94fcfd95b5bf91
Author: Ravishankar N <ravishankar>
Date:   Fri Aug 7 17:32:29 2015 +0530

    tests: add basic/afr/arbiter-statfs.t
    
    This is a test-case for BZ 1251346
    
    Change-Id: Icbde8d17c823a3f2c98056c14a75f0ef5227b848
    BUG: 1251346
    Signed-off-by: Ravishankar N <ravishankar>
    Reviewed-on: http://review.gluster.org/11864
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Raghavendra Talur <rtalur>
    Tested-by: NetBSD Build System <jenkins.org>

Comment 10 Vijay Bellur 2015-09-02 21:06:07 UTC
REVIEW: http://review.gluster.org/12094 (tests: Parsing df output isn't reliable, use stat -f) posted (#1) for review on master by Raghavendra Talur (rtalur)

Comment 11 Vijay Bellur 2015-09-03 06:38:41 UTC
REVIEW: http://review.gluster.org/12096 (tests: check free space only on gluster mount) posted (#1) for review on master by Ravishankar N (ravishankar)

Comment 12 Vijay Bellur 2015-09-03 07:05:39 UTC
REVIEW: http://review.gluster.org/12096 (tests: check free space only on gluster mount) posted (#2) for review on master by Ravishankar N (ravishankar)

Comment 13 Vijay Bellur 2015-09-04 04:38:32 UTC
COMMIT: http://review.gluster.org/12096 committed in master by Raghavendra Talur (rtalur) 
------
commit 378b625698fe01ed2c3a6f4f065a699b2d573947
Author: Ravishankar N <ravishankar>
Date:   Thu Sep 3 12:05:12 2015 +0530

    tests: check free space only on gluster mount
    
    Change-Id: I06de4f555e66fac2594676572c8f8a4ee08f8131
    BUG: 1251346
    Signed-off-by: Ravishankar N <ravishankar>
    Reviewed-on: http://review.gluster.org/12096
    Reviewed-by: Raghavendra Talur <rtalur>
    Tested-by: NetBSD Build System <jenkins.org>
    Tested-by: Gluster Build System <jenkins.com>

Comment 14 Nagaprasad Sathyanarayana 2015-10-25 15:21:35 UTC
Fix for this BZ is already present in a GlusterFS release. You can find clone of this BZ, fixed in a GlusterFS release and closed. Hence closing this mainline BZ as well.

Comment 15 Niels de Vos 2016-06-16 13:29:42 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.8.0, please open a new bug report.

glusterfs-3.8.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://blog.gluster.org/2016/06/glusterfs-3-8-released/
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user