Bug 1139230 - AFR: invalid deletion of entry from indices/xattrop directory
Summary: AFR: invalid deletion of entry from indices/xattrop directory
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: replicate
Version: mainline
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Anuradha
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 1144744
TreeView+ depends on / blocked
 
Reported: 2014-09-08 12:50 UTC by Anuradha
Modified: 2016-09-20 02:00 UTC (History)
2 users (show)

Fixed In Version: glusterfs-3.7.0
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 1144744 (view as bug list)
Environment:
Last Closed: 2015-05-14 17:27:32 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Anuradha 2014-09-08 12:50:44 UTC
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:

Comment 1 Anand Avati 2014-09-08 13:13:14 UTC
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)

Comment 2 Anand Avati 2014-09-09 06:01:59 UTC
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)

Comment 3 Anand Avati 2014-09-12 09:11:23 UTC
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)

Comment 4 Anand Avati 2014-09-15 10:30:18 UTC
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)

Comment 5 Anand Avati 2014-09-16 10:08:07 UTC
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)

Comment 6 Anand Avati 2014-09-17 02:49:40 UTC
COMMIT: http://review.gluster.org/8652 committed in master by Pranith Kumar Karampuri (pkarampu) 
------
commit 6f5bb1bfc520a91d70a02a9c13104c798e2dba14
Author: Anuradha <atalur>
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>
    Reviewed-on: http://review.gluster.org/8652
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Pranith Kumar Karampuri <pkarampu>
    Tested-by: Pranith Kumar Karampuri <pkarampu>

Comment 7 Niels de Vos 2015-05-14 17:27:32 UTC
This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.7.0, please open a new bug report.

glusterfs-3.7.0 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution.

[1] http://thread.gmane.org/gmane.comp.file-systems.gluster.devel/10939
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user

Comment 8 Niels de Vos 2015-05-14 17:35:35 UTC
This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.7.0, please open a new bug report.

glusterfs-3.7.0 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution.

[1] http://thread.gmane.org/gmane.comp.file-systems.gluster.devel/10939
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user

Comment 9 Niels de Vos 2015-05-14 17:37:57 UTC
This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.7.0, please open a new bug report.

glusterfs-3.7.0 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution.

[1] http://thread.gmane.org/gmane.comp.file-systems.gluster.devel/10939
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user

Comment 10 Niels de Vos 2015-05-14 17:43:41 UTC
This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.7.0, please open a new bug report.

glusterfs-3.7.0 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution.

[1] http://thread.gmane.org/gmane.comp.file-systems.gluster.devel/10939
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user


Note You need to log in before you can comment on or make changes to this bug.