Bug 1383913 - spurious heal info as pending heal entries never end on an EC volume while IOs are going on
Summary: spurious heal info as pending heal entries never end on an EC volume while IO...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: disperse
Version: 3.9
Hardware: Unspecified
OS: Unspecified
unspecified
urgent
Target Milestone: ---
Assignee: Ashish Pandey
QA Contact:
URL:
Whiteboard:
Depends On: 1347257 1366815
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-10-12 07:11 UTC by Ashish Pandey
Modified: 2016-12-06 05:58 UTC (History)
10 users (show)

Fixed In Version: glusterfs-3.9.0
Clone Of: 1366815
Environment:
Last Closed: 2016-12-06 05:58:59 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:

Attachments (Terms of Use)

Comment 1 Worker Ant 2016-10-12 07:27:04 UTC 
REVIEW: http://review.gluster.org/15627 (cluster/ec: Implement heal info with lock) posted (#1) for review on release-3.9 by Ashish Pandey (aspandey)
Comment 2 Worker Ant 2016-10-13 14:35:50 UTC 
COMMIT: http://review.gluster.org/15627 committed in release-3.9 by Pranith Kumar Karampuri (pkarampu) 
------
commit 2f287ccec79ac8c3b0d808da4db0d661c99fa856
Author: Ashish Pandey <aspandey>
Date:   Tue Sep 20 12:32:28 2016 +0530

    cluster/ec: Implement heal info with lock
    
    Problem: Currently heal info command prints all
    the files/directories if the index for the
    file/directory is present in .glusterfs/indices folder.
    After implementing patch http://review.gluster.org/#/c/13733/
    indices of the file which is going through update fop
    will also be present in .glusterfs/indices even
    if the fop is successful on all the brick. At this time
    if heal info command is being used, it will also display this
    file which is actually healthy and does not require any heal.
    
    Solution: Take lock on a file corresponding to the indices
    and inspect xattrs to decide if the file needs heal or not.
    
    >Reviewed-on: http://review.gluster.org/15543
    >NetBSD-regression: NetBSD Build System <jenkins.org>
    >Reviewed-by: Pranith Kumar Karampuri <pkarampu>
    >Reviewed-by: Xavier Hernandez <xhernandez>
    >CentOS-regression: Gluster Build System <jenkins.org>
    >Smoke: Gluster Build System <jenkins.org>
    >Signed-off-by: Ashish Pandey <aspandey>
    
    Change-Id: I6361e2813ece369be12d02e74816df4eddb81cfa
    BUG: 1383913
    Signed-off-by: Ashish Pandey <aspandey>
    Reviewed-on: http://review.gluster.org/15627
    Smoke: Gluster Build System <jenkins.org>
    CentOS-regression: Gluster Build System <jenkins.org>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    Reviewed-by: Xavier Hernandez <xhernandez>
Comment 3 Aravinda VK 2016-10-27 05:22:18 UTC 
glusterfs-3.9.0rc2 is released[1] and packages are available for different distributions[2] to test.

[1] http://www.gluster.org/pipermail/maintainers/2016-October/001601.html
[2] http://www.gluster.org/pipermail/maintainers/2016-October/001605.html and http://www.gluster.org/pipermail/maintainers/2016-October/001606.html
Comment 4 Aravinda VK 2016-12-06 05:58:59 UTC 
Gluster 3.9 GA is released http://blog.gluster.org/2016/11/announcing-gluster-3-9/
Note You need to log in before you can comment on or make changes to this bug.