Bug 1383913

Summary:	spurious heal info as pending heal entries never end on an EC volume while IOs are going on
Product:	[Community] GlusterFS	Reporter:	Ashish Pandey <aspandey>
Component:	disperse	Assignee:	Ashish Pandey <aspandey>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	urgent	Docs Contact:
Priority:	unspecified
Version:	3.9	CC:	amukherj, aspandey, bugs, mzywusko, nchilaka, pkarampu, ravishankar, rcyriac, rhinduja, rhs-bugs
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	glusterfs-3.9.0	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:	1366815	Environment:
Last Closed:	2016-12-06 05:58:59 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:	1347257, 1366815
Bug Blocks:

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/