Bug 1025955 - 'stime' xattr is not the minimum of all distribute subvolumes
Summary: 'stime' xattr is not the minimum of all distribute subvolumes
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: distribute
Version: mainline
Hardware: All
OS: All
medium
low
Target Milestone: ---
Assignee: bugs@gluster.org
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 1025956
TreeView+ depends on / blocked
 
Reported: 2013-11-02 13:19 UTC by Venky Shankar
Modified: 2018-06-18 13:38 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 1025956 (view as bug list)
Environment:
Last Closed: 2017-09-08 07:08:28 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

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.


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