Bug 769494 - marker should not index modifications of special clients
Summary: marker should not index modifications of special clients
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: geo-replication
Version: mainline
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ---
Assignee: Junaid
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 811632 815041 817967
TreeView+ depends on / blocked
 
Reported: 2011-12-21 02:52 UTC by Csaba Henk
Modified: 2013-08-06 22:38 UTC (History)
3 users (show)

Fixed In Version: glusterfs-3.4.0
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 815041 (view as bug list)
Environment:
Last Closed: 2013-07-24 17:31:30 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

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)


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