Bug 951800

Summary: AFR fops fail to propagate xdata
Product: [Community] GlusterFS Reporter: Prashanth Pai <nullpai>
Component: replicateAssignee: Pranith Kumar K <pkarampu>
Status: CLOSED DUPLICATE QA Contact:
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: mainlineCC: gluster-bugs, jdarcy, yinyin2010
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-07-13 07:08:09 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 Prashanth Pai 2013-04-13 08:49:35 UTC
AFR writev fop(and possibly many others) ignore xdata.

How to reproduce :
If xdata is set by a translator (in my case, the compression translator) located below AFR, AFR loses xdata.

What needs to be done in afr_writev (and others) : 

if(xdata)
   local->xdata_req = dict_ref(xdata);

and then pass local->xdata_req wherever STACK_WIND is called.

Comment 1 Anand Avati 2013-04-18 05:19:06 UTC
REVIEW: http://review.gluster.org/4845 (afr: make writev not ignore xdata) posted (#1) for review on master by Prashanth Pai (nullpai)

Comment 2 Anand Avati 2013-04-18 06:41:49 UTC
REVIEW: http://review.gluster.org/4845 (afr: make writev not ignore xdata) posted (#2) for review on master by Prashanth Pai (nullpai)

Comment 3 Jeff Darcy 2013-04-19 14:05:00 UTC
This seems to be a problem with all fops, though the submitted patch only fixes writev (which is OK and appreciated).

Comment 4 Anand Avati 2013-08-12 07:31:16 UTC
REVIEW: http://review.gluster.org/4845 (afr: make writev not ignore xdata) posted (#3) for review on master by Prashanth Pai (ppai)

Comment 5 Pranith Kumar K 2014-07-13 07:08:09 UTC
This issue is addressed by refactor by Avati in http://review.gluster.org/6010. So closing this one.

*** This bug has been marked as a duplicate of bug 1021686 ***