[Bugs] [Bug 1592666] New: lookup not assigning gfid if file is not present in all bricks of replica

[bugzilla at redhat.com]

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

            Bug ID: 1592666
           Summary: lookup not assigning gfid if file is not present in
                    all bricks of replica
           Product: Red Hat Gluster Storage
           Version: 3.4
         Component: replicate
          Keywords: Triaged
          Assignee: ravishankar at redhat.com
          Reporter: ravishankar at redhat.com
        QA Contact: nchilaka at redhat.com
                CC: bugs at gluster.org, nchilaka at redhat.com,
                    rhs-bugs at redhat.com, sankarshan at redhat.com,
                    storage-qa-internal at redhat.com
        Depends On: 1591193



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

Description of problem:


    commit 20fa80057eb430fd72b4fa31b9b65598b8ec1265 introduced a regression
    wherein if a file is present in only 1 brick of replica *and* doesn't
    have a gfid associated with it, it doesn't get healed upon the next
    lookup from the client. 


Found this while automating a glusto-test case which adds files directly from
the backend and expects lookup to assign gfid and complete the heal.

Steps to reproduce:
- Create a 1x3 vol and add different files to different bricks of the replica
directly on the backend.
- Try a lookup on the files individually from the client. It will fail with
ESTALE.

Comments:
While adding files directly to the bricks is not a supported usecase,  we could
hit this in the client FOP path too if the bricks go down at the right time
etc.

--- Additional comment from Worker Ant on 2018-06-14 05:18:42 EDT ---

REVIEW: https://review.gluster.org/20271 (afr: heal gfids when file is not
present on all bricks) posted (#1) for review on master by Ravishankar N

--- Additional comment from Ravishankar N on 2018-06-14 05:26:22 EDT ---

Correction:

s/ESTALE/ENODATA in the bug description

--- Additional comment from Worker Ant on 2018-06-19 02:05:48 EDT ---

COMMIT: https://review.gluster.org/20271 committed in master by "Pranith Kumar
Karampuri" <pkarampu at redhat.com> with a commit message- afr: heal gfids when
file is not present on all bricks

commit 20fa80057eb430fd72b4fa31b9b65598b8ec1265 introduced a regression
wherein if a file is present in only 1 brick of replica *and* doesn't
have a gfid associated with it, it doesn't get healed upon the next
lookup from the client. Fix it.

Change-Id: I7d1111dcb45b1b8b8340a7d02558f05df70aa599
fixes: bz#1591193
Signed-off-by: Ravishankar N <ravishankar at redhat.com>


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1591193
[Bug 1591193] lookup not assigning gfid if file is not present in all
bricks of replica
-- 
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=HjiSDQ8DB6&a=cc_unsubscribe