[Bugs] [Bug 1144744] New: AFR: invalid deletion of entry from indices/xattrop directory

[bugzilla at redhat.com]

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

            Bug ID: 1144744
           Summary: AFR: invalid deletion of entry from indices/xattrop
                    directory
           Product: GlusterFS
           Version: 3.6.0
         Component: replicate
          Assignee: gluster-bugs at redhat.com
          Reporter: atalur at redhat.com
                CC: bugs at gluster.org, gluster-bugs at redhat.com
        Depends On: 1139230



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

Description of problem:
During an xattrop fop if the xattr is not populated, index entry corresponding
to the file on which the xattrop is being performed gets deleted from
indices/xattrop directory.

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 Anand Avati on 2014-09-08 09:13:14 EDT ---

REVIEW: http://review.gluster.org/8652 (index: Do not add/delete index file if
xattr is empty) posted (#1) for review on master by Anuradha Talur
(atalur at redhat.com)

--- Additional comment from Anand Avati on 2014-09-09 02:01:59 EDT ---

REVIEW: http://review.gluster.org/8652 (index: Do not add/delete index file if
xattr is empty) posted (#2) for review on master by Anuradha Talur
(atalur at redhat.com)

--- Additional comment from Anand Avati on 2014-09-12 05:11:23 EDT ---

REVIEW: http://review.gluster.org/8652 (cluster/afr: Set all the xattrs needed
by index xlator) posted (#3) for review on master by Anuradha Talur
(atalur at redhat.com)

--- Additional comment from Anand Avati on 2014-09-15 06:30:18 EDT ---

REVIEW: http://review.gluster.org/8652 (cluster/afr: Set all the xattrs needed
by index xlator) posted (#4) for review on master by Anuradha Talur
(atalur at redhat.com)

--- Additional comment from Anand Avati on 2014-09-16 06:08:07 EDT ---

REVIEW: http://review.gluster.org/8652 (cluster/afr: Set all the xattrs needed
by index xlator) posted (#5) for review on master by Anuradha Talur
(atalur at redhat.com)

--- Additional comment from Anand Avati on 2014-09-16 22:49:40 EDT ---

COMMIT: http://review.gluster.org/8652 committed in master by Pranith Kumar
Karampuri (pkarampu at redhat.com) 
------
commit 6f5bb1bfc520a91d70a02a9c13104c798e2dba14
Author: Anuradha <atalur at redhat.com>
Date:   Mon Sep 8 18:32:04 2014 +0530

    cluster/afr: Set all the xattrs needed by index xlator

    Index xlator removes the index file from indices
    xattrop directory in case the value for keys sent
    are zero.

    If all the required keys are not set by afr
    then index file might be removed in an invalid
    way.

    With this change all the keys required by index
    xlator are set by afr such that invalid removal of
    files does not occur.

    Change-Id: Idbed0764a95157fd5cab8d6685057a43788fc7df
    BUG: 1139230
    Signed-off-by: Anuradha <atalur at redhat.com>
    Reviewed-on: http://review.gluster.org/8652
    Tested-by: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: Pranith Kumar Karampuri <pkarampu at redhat.com>
    Tested-by: Pranith Kumar Karampuri <pkarampu at redhat.com>


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1139230
[Bug 1139230] AFR: invalid deletion of entry from indices/xattrop directory
-- 
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=1NaWY6l4Ld&a=cc_unsubscribe