Bug 1139230

Summary:	AFR: invalid deletion of entry from indices/xattrop directory
Product:	[Community] GlusterFS	Reporter:	Anuradha <atalur>
Component:	replicate	Assignee:	Anuradha <atalur>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	mainline	CC:	gluster-bugs, smohan
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	glusterfs-3.7.0	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:
Clones:	1144744 (view as bug list)		Environment:
Last Closed:	2015-05-14 17:27:32 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:
Bug Depends On:
Bug Blocks:	1144744

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