Bug 769494

Summary:	marker should not index modifications of special clients
Product:	[Community] GlusterFS	Reporter:	Csaba Henk <csaba>
Component:	geo-replication	Assignee:	Junaid <junaid>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	mainline	CC:	gluster-bugs, vagarwal, vkoppad
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	glusterfs-3.4.0	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:
Clones:	815041 (view as bug list)		Environment:
Last Closed:	2013-07-24 17:31:30 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Bug Depends On:
Bug Blocks:	811632, 815041, 817967

Description Csaba Henk 2011-12-21 02:52:26 UTC

Description of problem:

Gluster clients using -1 as pid are considered internal to the gluster ecosystem and therefore marker translator should not update xtimes for the changes made by thnm.

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

Seen on master & release-3.2

How reproducible:

Solid.

Steps to Reproduce:
1. rm -rf /backing/dir
2. gluster volume create my-vol my-ip:/backing/dir && gluster volume start my-vol
3. gluster volume set my-vol indexing on
4. glusterfs --client-pid=-1 --volfile-id=my-vol -s localhost /mnt/gluster0/
5. touch /mnt/gluster0/x
6. getfattr -m /backing/dir | grep xtime | wc -l
  
Actual results:

1

Expected results:

0

Additional info:

Comment 1 Anand Avati 2012-01-04 22:24:25 UTC

CHANGE: http://review.gluster.com/2559 (features/marker: xtime should not be updated for modifications done by special client.) merged in master by Vijay Bellur (vijay)

Comment 2 Anand Avati 2012-01-24 11:34:59 UTC

CHANGE: http://review.gluster.com/2564 (features/marker: xtime should not be updated for modifications done by special client.) merged in release-3.2 by Vijay Bellur (vijay)

Comment 3 Vijaykumar Koppad 2012-02-10 10:20:30 UTC

Verified on gluster-3.2.6qa2. It works as expected.

Comment 4 Anand Avati 2012-03-20 14:47:27 UTC

CHANGE: http://review.gluster.com/2971 (features/marker: Replacing -1 with GF_CLIENT_PID_GSYNCD as part of code cleanup.) merged in master by Vijay Bellur (vijay)

Comment 5 Vijay Bellur 2013-02-07 21:36:39 UTC

CHANGE: http://review.gluster.org/2969 (features/marker: Replacing -1 with GF_CLIENT_PID_GSYNCD as part of code cleanup.) merged in release-3.2 by Anand Avati (avati)