Bug 1385762 - Don't create a directory if one with the same gfid exists
Summary: Don't create a directory if one with the same gfid exists
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: GlusterFS
Classification: Community
Component: posix
Version: mainline
Hardware: Unspecified
OS: Unspecified
high
medium
Target Milestone: ---
Assignee: Sheetal Pamecha
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-10-17 16:11 UTC by Nithya Balachandran
Modified: 2020-03-12 14:31 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2020-03-12 14:31:33 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Gluster.org Gerrit 23245 0 None Abandoned storage/posix: Skip mkdir if gfid exists 2020-05-24 19:31:09 UTC

Description Nithya Balachandran 2016-10-17 16:11:26 UTC
Description of problem:

In case a directory rename fails on a brick, a DHT lookup tries to create a directory with the new name as part of the heal.This means that there will be two directories with different names but the same gfid on the brick. This causes a lot of issues wrt file access as some files will not be visible on the mount point.


If the code is changed so that the directory is not created if a dir with the same gfid already exists, there will be different dirs with the same gfid across bricks but file listing still works and it is easier to recover from the scenario.


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Worker Ant 2016-10-17 16:17:20 UTC
REVIEW: http://review.gluster.org/15660 (storage/posix: Skip mkdir if gfid exists) posted (#1) for review on master by N Balachandran (nbalacha)

Comment 2 Worker Ant 2016-10-17 16:20:29 UTC
REVIEW: http://review.gluster.org/15660 (storage/posix: Skip mkdir if gfid exists) posted (#2) for review on master by N Balachandran (nbalacha)

Comment 3 Worker Ant 2016-10-18 11:13:49 UTC
REVIEW: http://review.gluster.org/15660 (storage/posix: Skip mkdir if gfid exists) posted (#3) for review on master by N Balachandran (nbalacha)

Comment 4 Worker Ant 2019-08-16 08:33:52 UTC
REVIEW: https://review.gluster.org/23245 (storage/posix: Skip mkdir if gfid exists) posted (#1) for review on master by Sheetal Pamecha

Comment 5 Worker Ant 2020-03-12 14:31:33 UTC
This bug is moved to https://github.com/gluster/glusterfs/issues/1059, and will be tracked there from now on. Visit GitHub issues URL for further details


Note You need to log in before you can comment on or make changes to this bug.