Bug 764196 (GLUSTER-2464)

Summary: [7b07d444a77526f27f860210930bf1d4c7fbea9b]: rm -rf gives Invalid argumenrt error
Product: [Community] GlusterFS Reporter: Raghavendra Bhat <rabhat>
Component: distributeAssignee: shishir gowda <sgowda>
Status: CLOSED NOTABUG QA Contact:
Severity: medium Docs Contact:
Priority: urgent    
Version: mainlineCC: aavati, anush, berend, buddie, fharshav, gluster-bugs, jdarcy, joe, lakshmipathi, nsathyan, pierre.francois, rahulcs, renee, sac, sgowda, vijay
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: Type: ---
Regression: --- Mount Type: fuse
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Raghavendra Bhat 2011-02-25 11:53:01 UTC 
Steps:

1) Cloned a glusterfs git repository onto the glusterfs mountpoint 

2) did ./autogen.sh, configure and make

Now doing rm -rf of the repository gives the following error. 

Its not always reproducible. But most of the times it happens.




root@bigbang:/mnt/dir/glusterfs.git# cd ..
root@bigbang:/mnt/dir# ls
fstest_10292a823178f683f79fe700ef073f2e  glusterfs.git
root@bigbang:/mnt/dir# rm -rf *
rm: cannot remove `glusterfs.git/.git/config': Invalid argument




This is the entry in the log file corresponding to the error:



[2011-02-25 17:12:11.600672] D [stat-prefetch.c:3587:sp_release] vol-stat-prefetch: cache hits: 0, cache miss: 1
[2011-02-25 17:12:11.603777] D [stat-prefetch.c:3587:sp_release] vol-stat-prefetch: cache hits: 1, cache miss: 0
[2011-02-25 17:12:11.611700] D [client3_1-fops.c:1937:client3_1_lookup_cbk] vol-client-2: gfid changed for /glusterfs.git/.git/config
[2011-02-25 17:12:11.611753] W [fuse-bridge.c:907:fuse_unlink_cbk] glusterfs-fuse: 580322: UNLINK() /glusterfs.git/.git/config => -1 (Invalid
 argument)
[2011-02-25 17:12:11.611807] D [dht-common.c:2319:dht_open] vol-dht: no cached subvolume for fd=0x7fbf8231c270
[2011-02-25 17:12:11.611828] W [fuse-bridge.c:584:fuse_fd_cbk] glusterfs-fuse: 580323: OPEN() /glusterfs.git/.git/config => -1 (Invalid argum
ent)
[2011-02-25 17:12:11.613157] D [stat-prefetch.c:3587:sp_release] vol-stat-prefetch: cache hits: 5, cache miss: 0
[2011-02-25 17:12:11.614779] D [stat-prefetch.c:3587:sp_release] vol-stat-prefetch: cache hits: 0, cache miss: 1



The log file is attached.
Comment 1 Pranith Kumar K 2011-04-08 07:19:20 UTC 
*** Bug 2682 has been marked as a duplicate of this bug. ***
Comment 2 shishir gowda 2011-05-23 03:00:05 UTC 
In our test setup, disabling stat-pretech xlator seems to be mitigating this issue. Consider this as a work around till a fix is out.
Comment 3 Joe Julian 2011-05-31 05:49:08 UTC 
*** Bug 2962 has been marked as a duplicate of this bug. ***
Comment 4 Anand Avati 2011-05-31 13:10:41 UTC 
PATCH: http://patches.gluster.com/patch/7267 in master (dht-rename: Unlink older link files before creating new one)
Comment 5 Anand Avati 2011-05-31 13:11:42 UTC 
PATCH: http://patches.gluster.com/patch/7323 in release-3.1 (DHT-rename: Unlink older link files before creating new one.)
Comment 6 Anand Avati 2011-05-31 13:12:59 UTC 
PATCH: http://patches.gluster.com/patch/7265 in release-3.2 (dht-rename: Unlink older link files before creating new one)
Comment 7 shishir gowda 2011-06-01 02:17:34 UTC 
*** Bug 2895 has been marked as a duplicate of this bug. ***
Comment 8 Lakshmipathi G 2011-06-03 05:48:18 UTC 
tested with 3.2.1qa2
Comment 9 Anand Avati 2011-07-21 19:35:16 UTC 
Issue still exists. Previous fix is incorrect.
Comment 10 Joe Julian 2011-07-21 20:21:15 UTC 
The patch you've had me test:
http://shell.gluster.com/~avati/0001-cluster-dht-dht_rename-handle-GFID-situations.patch

passes all the failure tests I was able to repro with vanilla v3.1.5 on my test VMs.
Comment 11 Joe Julian 2011-07-21 20:56:02 UTC 
*** Bug 3188 has been marked as a duplicate of this bug. ***
Comment 12 Harshavardhana 2011-07-22 03:50:04 UTC 
*** Bug 3216 has been marked as a duplicate of this bug. ***
Comment 13 Anand Avati 2011-07-23 13:55:27 UTC 
CHANGE: http://review.gluster.com/78 (Change-Id: Ia024fd136c936c5a73e2cdacdb738e5d2912d619) merged in release-3.1 by Anand Avati (avati)
Comment 14 shishir gowda 2011-07-25 04:32:35 UTC 
Incorrectly closed as patches for 3.2 and master still need to be committed.
Comment 15 shishir gowda 2011-07-25 04:33:30 UTC 
*** Bug 3220 has been marked as a duplicate of this bug. ***
Comment 16 Anand Avati 2011-07-25 13:22:32 UTC 
CHANGE: http://review.gluster.com/90 (In the pre-GFID era, the linkfile of the destination file could be reused) merged in master by Anand Avati (avati)
Comment 17 Anand Avati 2011-07-25 13:22:44 UTC 
CHANGE: http://review.gluster.com/88 (In the pre-GFID era, the linkfile of the destination file could be reused) merged in release-3.1 by Anand Avati (avati)
Comment 18 Anand Avati 2011-07-25 13:22:54 UTC 
CHANGE: http://review.gluster.com/89 (In the pre-GFID era, the linkfile of the destination file could be reused) merged in release-3.2 by Anand Avati (avati)
Comment 19 shishir gowda 2011-08-01 03:45:33 UTC 
*** Bug 3222 has been marked as a duplicate of this bug. ***
Comment 20 Renee 2011-09-14 07:08:35 UTC 
After upgrading to 3.2.3 Cypress still sees this issue.  I need a response from Engineering on this today please.
Comment 21 shishir gowda 2011-09-20 00:54:49 UTC 
The issue seems to exist because few of the clients being used were still running gluster version 3.2.2. On upgradation to 3.2.3, the client has not seen the tests failing. Downgrading the severity to normal, till the client confirms that upgrading to 3.2.3 solves the problem
Comment 22 shishir gowda 2011-09-22 02:34:07 UTC 
Closing the bug as the client has not run into the error after upgrading all glusterfs clients to 3.2.3 version