Bug 1366815
| Summary: | spurious heal info as pending heal entries never end on an EC volume while IOs are going on | |||
|---|---|---|---|---|
| Product: | [Community] GlusterFS | Reporter: | Pranith Kumar K <pkarampu> | |
| Component: | disperse | Assignee: | Ashish Pandey <aspandey> | |
| Status: | CLOSED CURRENTRELEASE | QA Contact: | ||
| Severity: | urgent | Docs Contact: | ||
| Priority: | unspecified | |||
| Version: | mainline | 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.10.0 | Doc Type: | If docs needed, set a value | |
| Doc Text: | Story Points: | --- | ||
| Clone Of: | 1347257 | |||
| : | 1383913 (view as bug list) | Environment: | ||
| Last Closed: | 2017-03-06 17:22:10 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 | |||
| Bug Blocks: | 1383913 | |||
|
Comment 1
Worker Ant
2016-09-22 08:10:42 UTC
REVIEW: http://review.gluster.org/15543 (cluster/ec: Implement heal info with lock) posted (#4) for review on master by Ashish Pandey (aspandey) REVIEW: http://review.gluster.org/15543 (cluster/ec: Implement heal info with lock) posted (#5) for review on master by Ashish Pandey (aspandey) REVIEW: http://review.gluster.org/15543 (cluster/ec: Implement heal info with lock) posted (#6) for review on master by Ashish Pandey (aspandey) REVIEW: http://review.gluster.org/15543 (cluster/ec: Implement heal info with lock) posted (#7) for review on master by Ashish Pandey (aspandey) REVIEW: http://review.gluster.org/15543 (cluster/ec: Implement heal info with lock) posted (#8) for review on master by Ashish Pandey (aspandey) REVIEW: http://review.gluster.org/15543 (cluster/ec: Implement heal info with lock) posted (#9) for review on master by Ashish Pandey (aspandey) REVIEW: http://review.gluster.org/15543 (cluster/ec: Implement heal info with lock) posted (#10) for review on master by Ashish Pandey (aspandey) REVIEW: http://review.gluster.org/15543 (cluster/ec: Implement heal info with lock) posted (#11) for review on master by Ashish Pandey (aspandey) REVIEW: http://review.gluster.org/15543 (cluster/ec: Implement heal info with lock) posted (#12) for review on master by Ashish Pandey (aspandey) REVIEW: http://review.gluster.org/15543 (cluster/ec: Implement heal info with lock) posted (#13) for review on master by Ashish Pandey (aspandey) REVIEW: http://review.gluster.org/15543 (cluster/ec: Implement heal info with lock) posted (#14) for review on master by Ashish Pandey (aspandey) COMMIT: http://review.gluster.org/15543 committed in master by Xavier Hernandez (xhernandez) ------ commit 0fed7e7f0aad9973900c89434f736797d9ace2bd 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. Change-Id: I6361e2813ece369be12d02e74816df4eddb81cfa BUG: 1366815 Signed-off-by: Ashish Pandey <aspandey> 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> 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.10.0, please open a new bug report. glusterfs-3.10.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://lists.gluster.org/pipermail/gluster-users/2017-February/030119.html [2] https://www.gluster.org/pipermail/gluster-users/ |