Bug 1138387
Summary: | DHT :- data loss - file is missing on renaming same file from multiple client at same time | ||
---|---|---|---|
Product: | [Community] GlusterFS | Reporter: | Shyamsundar <srangana> |
Component: | distribute | Assignee: | Shyamsundar <srangana> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | |
Severity: | high | Docs Contact: | |
Priority: | unspecified | ||
Version: | 3.6.0 | CC: | bugs, gluster-bugs, jdarcy, nsathyan, racpatel, ssamanta |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | x86_64 | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | glusterfs-3.6.0beta1 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | 1117851 | Environment: | |
Last Closed: | 2014-11-11 08:38:04 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: | 1117135, 1117851, 1139988, 1146889 | ||
Bug Blocks: | 1117822, 1129527 |
Description
Shyamsundar
2014-09-04 17:15:48 UTC
REVIEW: http://review.gluster.org/8605 (dht: fix rename race) posted (#1) for review on release-3.6 by Shyamsundar Ranganathan (srangana) REVIEW: http://review.gluster.org/8604 (storage/posix: removing deleting entries in case of creation failures) posted (#2) for review on release-3.6 by Shyamsundar Ranganathan (srangana) REVIEW: http://review.gluster.org/8605 (dht: fix rename race) posted (#2) for review on release-3.6 by Shyamsundar Ranganathan (srangana) REVIEW: http://review.gluster.org/8604 (storage/posix: removing deleting entries in case of creation failures) posted (#3) for review on release-3.6 by Shyamsundar Ranganathan (srangana) REVIEW: http://review.gluster.org/8605 (dht: fix rename race) posted (#3) for review on release-3.6 by Shyamsundar Ranganathan (srangana) REVIEW: http://review.gluster.org/8604 (storage/posix: removing deleting entries in case of creation failures) posted (#4) for review on release-3.6 by Shyamsundar Ranganathan (srangana) REVIEW: http://review.gluster.org/8605 (dht: fix rename race) posted (#4) for review on release-3.6 by Shyamsundar Ranganathan (srangana) COMMIT: http://review.gluster.org/8604 committed in release-3.6 by Vijay Bellur (vbellur) ------ commit 71c1079fb8e73cd27aa3418b1be21c2507dc24c9 Author: Raghavendra G <rgowdapp> Date: Thu Sep 4 14:03:04 2014 -0400 storage/posix: removing deleting entries in case of creation failures The code is not atomic enough to not to delete a dentry created by a prallel dentry creation operation. Change-Id: I9bd6d2aa9e7a1c0688c0a937b02a4b4f56d7aa3d BUG: 1138387 Signed-off-by: Raghavendra G <rgowdapp> Reviewed-on-master: http://review.gluster.org/8327 Reviewed-by: Pranith Kumar Karampuri <pkarampu> Tested-by: Gluster Build System <jenkins.com> Reviewed-by: Vijay Bellur <vbellur> COMMIT: http://review.gluster.org/8605 committed in release-3.6 by Vijay Bellur (vbellur) ------ commit d34029cceb290bda27357ca38238a714a0cba286 Author: Nithya Balachandran <nbalacha> Date: Thu Sep 4 14:05:04 2014 -0400 dht: fix rename race Additional check to check if we created the linkto file before deleting it in the rename cleanup function Change-Id: I919cd7cb24f948ba4917eb9cf50d5169bb730a67 BUG: 1138387 Signed-off-by: Nithya Balachandran <nbalacha> Reviewed-on-master: http://review.gluster.org/8338 Tested-by: Gluster Build System <jenkins.com> Reviewed-by: Raghavendra G <rgowdapp> Reviewed-by: Vijay Bellur <vbellur> Reviewed-on: http://review.gluster.org/8605 Reviewed-by: Jeff Darcy <jdarcy> REVIEW: http://review.gluster.org/8691 (storage/posix: removing deleting entries in case of creation failures) posted (#1) for review on release-3.6 by Vijay Bellur (vbellur) REVIEW: http://review.gluster.org/8693 (storage/posix: removing deleting entries in case of creation failures) posted (#1) for review on release-3.6 by Vijay Bellur (vbellur) COMMIT: http://review.gluster.org/8693 committed in release-3.6 by Vijay Bellur (vbellur) ------ commit 2460b9175b60981096203a4331e434302da14d4b Author: Raghavendra G <rgowdapp> Date: Thu Sep 4 14:03:04 2014 -0400 storage/posix: removing deleting entries in case of creation failures The code is not atomic enough to not to delete a dentry created by a prallel dentry creation operation. Change-Id: I9bd6d2aa9e7a1c0688c0a937b02a4b4f56d7aa2e BUG: 1138387 Signed-off-by: Raghavendra G <rgowdapp> Reviewed-on-master: http://review.gluster.org/8327 Reviewed-by: Pranith Kumar Karampuri <pkarampu> Tested-by: Gluster Build System <jenkins.com> Reviewed-by: Vijay Bellur <vbellur> Reviewed-on: http://review.gluster.org/8693 Tested-by: Vijay Bellur <vbellur> A beta release for GlusterFS 3.6.0 has been released. Please verify if the release solves this bug report for you. In case the glusterfs-3.6.0beta1 release does not have a resolution for this issue, leave a comment in this bug and move the status to ASSIGNED. If this release fixes the problem for you, leave a note and change the status to VERIFIED. Packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update (possibly an "updates-testing" repository) infrastructure for your distribution. [1] http://supercolony.gluster.org/pipermail/gluster-users/2014-September/018836.html [2] http://supercolony.gluster.org/pipermail/gluster-users/ 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.6.1, please reopen this bug report. glusterfs-3.6.1 has been announced [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://supercolony.gluster.org/pipermail/gluster-users/2014-November/019410.html [2] http://supercolony.gluster.org/mailman/listinfo/gluster-users |