Bug 1615795

Summary:	Trash xlator do not send the gfid in the dict while recursively creating dirs
Product:	[Community] GlusterFS	Reporter:	Karthik U S <ksubrahm>
Component:	trash-xlator	Assignee:	bugs <bugs>
Status:	CLOSED WONTFIX	QA Contact:
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	mainline	CC:	anoopcs, bugs, jthottan, pkarampu
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2018-08-24 04:37:27 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:

Description Karthik U S 2018-08-14 09:27:06 UTC

Description of problem:
When trash xlator tries to create directories recursively inside .trashcan as part of the trash_truncate_mkdir_cbk() function, it does not send the gfid to be set for the newly created directory in the dictionary. When posix_gfid_set() function checks for the gfid in the xattr_req dictionary it will not see one being sent there and will return from there. So when we try to send the appropriate error if there is no gfid in the dict, the trash_truncate operation will fail.

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


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:
trash xlator should send the gfid xattr value in the dictionary while creating directories recursively as part of trash_truncate_mkdir_cbk().


Additional info:

Comment 1 Pranith Kumar K 2018-08-17 05:25:37 UTC

Anoop,
    How long do you think it will take to fix this issue? Based on that we can take the decision about whether to merge the patch Karthik sent or wait it out.

Let us know.

Pranith

Comment 2 Anoop C S 2018-08-17 06:16:52 UTC

Jiffin had a comment on change 20630:

https://review.gluster.org/c/glusterfs/+/20630#message-80e53c67e1347a855228aa68616dc0bfddccf0b2