Bug 765572 - (GLUSTER-3840) Detect and repair root gfid when a volume is created from a former volume's parent dir
Detect and repair root gfid when a volume is created from a former volume's p...
Product: GlusterFS
Classification: Community
Component: posix (Show other bugs)
x86_64 Linux
medium Severity medium
: ---
: ---
Assigned To: Divya
Vijaykumar Koppad
Depends On:
  Show dependency treegraph
Reported: 2011-12-02 08:12 EST by Joe Julian
Modified: 2014-08-24 20:49 EDT (History)
3 users (show)

See Also:
Fixed In Version: glusterfs-3.5.0
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2014-04-17 07:37:55 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: DP
Verified Versions: 3.3.0qa45
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

  None (edit)
Description Joe Julian 2011-12-02 08:12:15 EST
The following was reported via IRC where the user had created a volume using /hekafs-export/www as the brick directory. Later, he removed that volume and used /hekafs-export as the brick directory. This led to an endless series of this error message:

Dec  2 01:32:44 hakafs01 GlusterFS[17549]: [2011-12-02 01:32:44.661161] C [inode.c:232:__is_dentry_cyclic] 0-posix-acl-autoload/inode: detected cyclic loop formation during inode linkage. inode (1/00000000-0000-0000-0000-000000000001) linking under itself as www

getfattr -m . -d -e hex /hekafs-export /hekafs-export/www
getfattr: Entferne führenden '/' von absoluten Pfadnamen
# file: hekafs-export

# file: hekafs-export/www

The user did not understand the error message, nor was he able to deduce from the error what needed to be done to correct the problem. It seems like this should be able to be self-corrected.
Comment 1 Anand Avati 2011-12-26 00:41:17 EST
CHANGE: http://review.gluster.com/781 (extras: clean up a brick's gfid xattr) merged in master by Vijay Bellur (vijay@gluster.com)
Comment 2 Anand Avati 2012-01-05 12:18:45 EST
CHANGE: http://review.gluster.com/2514 (extras: add check for brick path existence) merged in master by Vijay Bellur (vijay@gluster.com)
Comment 3 Vijaykumar Koppad 2012-06-07 09:56:29 EDT
This patch doesn't seem to do any behavioural changes. Its just a script clear_xattr.sh in the glusterFS source  which can be used to remove xattr from the bricks previously belong to some other volume. I think this should be documented properly. Moving it to DP .
Comment 4 Niels de Vos 2014-04-17 07:37:55 EDT
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.5.0, please reopen this bug report.

glusterfs-3.5.0 has been announced on the Gluster Developers mailinglist [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://thread.gmane.org/gmane.comp.file-systems.gluster.devel/6137
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user

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