Bug 1025955

Summary:	'stime' xattr is not the minimum of all distribute subvolumes
Product:	[Community] GlusterFS	Reporter:	Venky Shankar <vshankar>
Component:	distribute	Assignee:	bugs <bugs>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	low	Docs Contact:
Priority:	medium
Version:	mainline	CC:	avishwan, bugs, nbalacha, spalai, vshankar
Target Milestone:	---	Keywords:	Triaged
Target Release:	---
Hardware:	All
OS:	All
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:
Clones:	1025956 (view as bug list)		Environment:
Last Closed:	2017-09-08 07:08:28 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:	1025956

Description Venky Shankar 2013-11-02 13:19:41 UTC

Description of problem:
'stime' xattr is not fetched as the minimum of all distribute subvolumes. This is required by geo-replication for correctly identify the slave lag.


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

How reproducible:
always

Comment 1 Anand Avati 2013-11-02 14:41:10 UTC

REVIEW: http://review.gluster.org/6214 (cluster/dht: fix 'stime' getxattr getting overwritten) posted (#1) for review on master by Venky Shankar (vshankar)

Comment 3 Nithya Balachandran 2016-08-10 05:40:11 UTC

This looks like it might have been fixed by http://review.gluster.org/#/c/6405.

Can you please check and update the BZ?

Comment 4 Susant Kumar Palai 2017-09-08 07:08:28 UTC

The intended patch in comment 1 is fixed as part of https://review.gluster.org/#/c/6405.

From the commit messag:

glusterd/geo-rep: more glusterd and cli fixes for geo-rep.

    -> handle option validation cases in reset case.
    -> Creating valid conf path when glusterd restarts.
    -> Reading the gsyncd worker thread status and displaying it.
    -> Displaying status-detail per worker.
    -> Fetch checkpoint info in geo-rep status.
    -> use-tarssh value validation added.

misc: misc geo-rep fixes based on cluster, logrotate etc..
    -> cluster/dht: fix 'stime' getxattr getting overwritten.
    -> cluster/afr: return max of 'stime' values in subvol.
    -> geo-rep-logrotate: Sending SIGHUP to geo-rep auxiliary.
    -> cluster/dht: fix convoluted logic while aggregating.
    -> cluster/*: fix 'stime' min/max fetch logic. <--- fix for stime logic


Marking this as closed since the patch is already merged.