[Bugs] [Bug 1318427] New: gfid-reset of a directory in distributed replicate volume doesn't set gfid on 2nd till last subvolumes

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=1318427

            Bug ID: 1318427
           Summary: gfid-reset of a directory in distributed replicate
                    volume doesn't set gfid on 2nd till last subvolumes
           Product: Red Hat Gluster Storage
           Version: 3.1
         Component: gluster-afr
          Severity: high
          Assignee: pkarampu at redhat.com
          Reporter: dblack at redhat.com
        QA Contact: storage-qa-internal at redhat.com
                CC: bugs at gluster.org, pkarampu at redhat.com
        Depends On: 1312816
            Blocks: 1313693



+++ This bug was initially created as a clone of Bug #1312816 +++

Description of problem:
    Afr does dict_ref of the xattr_req that comes to it and deletes "gfid-req"
key.
    Dht uses same dict to send lookup to other subvolumes. So in case of
    directories and more than 1 dht subvolumes, second subvolume till the last
    subvolume won't get a lookup request with "gfid-req". So gfid reset never
    happens on the directories in distributed replicate subvolume for 2nd till
last
    subvolumes.


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


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

--- Additional comment from Vijay Bellur on 2016-02-29 05:26:17 EST ---

REVIEW: http://review.gluster.org/13545 (cluster/afr: Don't delete gfid-req
from lookup request) posted (#1) for review on master by Pranith Kumar
Karampuri (pkarampu at redhat.com)

--- Additional comment from Vijay Bellur on 2016-03-01 03:58:38 EST ---

REVIEW: http://review.gluster.org/13545 (cluster/afr: Don't delete gfid-req
from lookup request) posted (#2) for review on master by Pranith Kumar
Karampuri (pkarampu at redhat.com)

--- Additional comment from Vijay Bellur on 2016-03-01 21:36:25 EST ---

REVIEW: http://review.gluster.org/13545 (cluster/afr: Don't delete gfid-req
from lookup request) posted (#3) for review on master by Pranith Kumar
Karampuri (pkarampu at redhat.com)

--- Additional comment from Vijay Bellur on 2016-03-02 03:55:01 EST ---

COMMIT: http://review.gluster.org/13545 committed in master by Pranith Kumar
Karampuri (pkarampu at redhat.com) 
------
commit 9b022c3a3f2f774904b5b458ae065425b46cc15d
Author: Pranith Kumar K <pkarampu at redhat.com>
Date:   Sat Feb 27 23:08:06 2016 +0530

    cluster/afr: Don't delete gfid-req from lookup request

    Problem:
    Afr does dict_ref of the xattr_req that comes to it and deletes "gfid-req"
key.
    Dht uses same dict to send lookup to other subvolumes. So in case of
    directories and more than 1 dht subvolumes, second subvolume till the last
    subvolume won't get a lookup request with "gfid-req". So gfid reset never
    happens on the directories in distributed replicate subvolume for 2nd till
last
    subvolumes.

    Fix:
    Make a copy of lookup xattr request.

    Also fixed replies_wipe possibly resetting gfid to NULL gfid

    BUG: 1312816
    Change-Id: Ic16260e5a4664837d069c1dc05b9e96ca05bda88
    Signed-off-by: Pranith Kumar K <pkarampu at redhat.com>
    Reviewed-on: http://review.gluster.org/13545
    Smoke: Gluster Build System <jenkins at build.gluster.com>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: Krutika Dhananjay <kdhananj at redhat.com>

--- Additional comment from Vijay Bellur on 2016-03-16 12:28:37 EDT ---

REVIEW: http://review.gluster.org/13754 (cluster/afr: Enhance the test to be
more robust) posted (#1) for review on master by Pranith Kumar Karampuri
(pkarampu at redhat.com)

--- Additional comment from Vijay Bellur on 2016-03-16 12:40:59 EDT ---

REVIEW: http://review.gluster.org/13754 (cluster/afr: Enhance the test to be
more robust) posted (#2) for review on master by Pranith Kumar Karampuri
(pkarampu at redhat.com)


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1312816
[Bug 1312816] gfid-reset of a directory in distributed replicate volume
doesn't set gfid on 2nd till last subvolumes
https://bugzilla.redhat.com/show_bug.cgi?id=1313693
[Bug 1313693] gfid-reset of a directory in distributed replicate volume
doesn't set gfid on 2nd till last subvolumes
-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=CP3l7DatVK&a=cc_unsubscribe