Bug 1212110
Summary: | bricks process crash | ||||||
---|---|---|---|---|---|---|---|
Product: | [Community] GlusterFS | Reporter: | Saurabh <saujain> | ||||
Component: | posix | Assignee: | Pranith Kumar K <pkarampu> | ||||
Status: | CLOSED CURRENTRELEASE | QA Contact: | |||||
Severity: | high | Docs Contact: | |||||
Priority: | unspecified | ||||||
Version: | mainline | CC: | akhakhar, bugs, kdhananj, mzywusko, pkarampu, rtalur, sasundar | ||||
Target Milestone: | --- | Keywords: | Triaged | ||||
Target Release: | --- | ||||||
Hardware: | x86_64 | ||||||
OS: | Linux | ||||||
Whiteboard: | |||||||
Fixed In Version: | glusterfs-3.8rc2 | Doc Type: | Bug Fix | ||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2016-06-16 12:52:08 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: | |||||||
Attachments: |
|
Description
Saurabh
2015-04-15 14:53:11 UTC
Created attachment 1014818 [details]
coredump of brick process
The most recent change that went into master in posix xlator was http://review.gluster.org/#/c/10180/ (sent by me!), which was merged on April 13th. It's probably what caused the crash. Will investigate. Thanks for the bug report. Seeing the same crashes on the servers while running BVT. OK, it has nothing to do with http://review.gluster.org/#/c/10180/. RCA'd it. Will send out a patch soon. The bug is hit with quota due to a race between xattrop and unlink. After quota winds an xattrop fop on a path, if by the time it reaches posix, another client has unlinked it, the brick crashes due to the absence of the gfid link && the absence of a proper NULL-check to handle this case. The fix involves handling the case of missing gfid link on the backend, in posix xlator appropriately and returning failure to the xlator above. Sent http://review.gluster.org/10999 to identify the code path where this can happen a lot earlier. This is not a fix but a step in identifying how the malformed link got created in the first place. So not moving the bug to POST http://review.gluster.org/11028 is sent to prevent crash when this issue is hit. Now it will fail with ESTALE instead of crashing. This is still not the complete fix. So not moving bug to POST. REVIEW: http://review.gluster.org/10999 (storage/posix: Prevent malformed internal link creations) posted (#2) for review on master by Pranith Kumar Karampuri (pkarampu) REVIEW: http://review.gluster.org/10999 (storage/posix: Prevent malformed internal link creations) posted (#3) for review on master by Vijay Bellur (vbellur) *** Bug 1222942 has been marked as a duplicate of this bug. *** COMMIT: http://review.gluster.org/11028 committed in master by Pranith Kumar Karampuri (pkarampu) ------ commit 476d4070dbdb9d73b36bd04f3eb3d6eda84abe73 Author: Pranith Kumar K <pkarampu> Date: Mon Jun 1 13:34:33 2015 +0530 storage/posix: Handle MAKE_INODE_HANDLE failures Change-Id: Ia176ccd4cac82c66ba50e3896fbe72c2da860c20 BUG: 1212110 Signed-off-by: Pranith Kumar K <pkarampu> Reviewed-on: http://review.gluster.org/11028 Reviewed-by: Krutika Dhananjay <kdhananj> Tested-by: NetBSD Build System <jenkins.org> REVIEW: http://review.gluster.org/10999 (storage/posix: Prevent malformed internal link creations) posted (#4) for review on master by Pranith Kumar Karampuri (pkarampu) REVIEW: http://review.gluster.org/10999 (storage/posix: Prevent malformed internal link creations) posted (#5) for review on master by Pranith Kumar Karampuri (pkarampu) 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 |