Bug 1399470 - Wrong value in Last Synced column during Hybrid Crawl
Summary: Wrong value in Last Synced column during Hybrid Crawl
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: geo-replication
Version: 3.9
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Aravinda VK
QA Contact:
URL:
Whiteboard:
Depends On: 1396081
Blocks: 1397286
TreeView+ depends on / blocked
 
Reported: 2016-11-29 06:28 UTC by Aravinda VK
Modified: 2017-03-08 10:20 UTC (History)
1 user (show)

Fixed In Version: glusterfs-3.9.1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1396081
Environment:
Last Closed: 2017-03-08 10:20:37 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Aravinda VK 2016-11-29 06:28:59 UTC
+++ This bug was initially created as a clone of Bug #1396081 +++

Description of problem:
During Hybrid Crawl, Last synced status shows wrong value.

--- Additional comment from Aravinda VK on 2016-11-17 07:15:11 EST ---

Last synced status picks from internal markings of stime xattr while updating for directories. Last synced value to be shown only when it is updated in Brick root(not sub directories)

Note: Last Synced Value will not change untill Hybrid crawl completes.

--- Additional comment from Worker Ant on 2016-11-17 07:32:13 EST ---

REVIEW: http://review.gluster.org/15869 (geo-rep: Fix Last synced status column issue during Hybrid Crawl) posted (#1) for review on master by Aravinda VK (avishwan)

--- Additional comment from Worker Ant on 2016-11-29 01:24:49 EST ---

COMMIT: http://review.gluster.org/15869 committed in master by Aravinda VK (avishwan) 
------
commit 1876454d2e7950f25d1e5bb8e2c07ab27d521498
Author: Aravinda VK <avishwan>
Date:   Thu Nov 17 17:47:29 2016 +0530

    geo-rep: Fix Last synced status column issue during Hybrid Crawl
    
    During Hybrid crawl, Geo-rep maintains stime xattr in subdirectories along
    with the Brick root. This is done to skip directories if Geo-rep crashes
    before Hybrid crawl completes.
    
    Update Last synced status only when stime xattr updated in brick root.
    Status output will mislead if it shows sub directory stime as
    last synced time.
    
    BUG: 1396081
    Change-Id: I5b73aee7ae4a1c1e2d1001d1f55559b9f9efd6e6
    Signed-off-by: Aravinda VK <avishwan>
    Reviewed-on: http://review.gluster.org/15869
    NetBSD-regression: NetBSD Build System <jenkins.org>
    CentOS-regression: Gluster Build System <jenkins.org>
    Smoke: Gluster Build System <jenkins.org>
    Reviewed-by: Kotresh HR <khiremat>

Comment 1 Worker Ant 2016-11-29 06:52:47 UTC
REVIEW: http://review.gluster.org/15962 (geo-rep: Fix Last synced status column issue during Hybrid Crawl) posted (#1) for review on release-3.9 by Aravinda VK (avishwan)

Comment 2 Worker Ant 2016-12-02 08:21:57 UTC
COMMIT: http://review.gluster.org/15962 committed in release-3.9 by Aravinda VK (avishwan) 
------
commit f1ef6b0c1e5d6f0f7eb67e85b3837c0cf33642c9
Author: Aravinda VK <avishwan>
Date:   Thu Nov 17 17:47:29 2016 +0530

    geo-rep: Fix Last synced status column issue during Hybrid Crawl
    
    During Hybrid crawl, Geo-rep maintains stime xattr in subdirectories along
    with the Brick root. This is done to skip directories if Geo-rep crashes
    before Hybrid crawl completes.
    
    Update Last synced status only when stime xattr updated in brick root.
    Status output will mislead if it shows sub directory stime as
    last synced time.
    
    > Reviewed-on: http://review.gluster.org/15869
    > NetBSD-regression: NetBSD Build System <jenkins.org>
    > CentOS-regression: Gluster Build System <jenkins.org>
    > Smoke: Gluster Build System <jenkins.org>
    > Reviewed-by: Kotresh HR <khiremat>
    
    BUG: 1399470
    Change-Id: I5b73aee7ae4a1c1e2d1001d1f55559b9f9efd6e6
    Signed-off-by: Aravinda VK <avishwan>
    (cherry picked from commit 1876454d2e7950f25d1e5bb8e2c07ab27d521498)
    Reviewed-on: http://review.gluster.org/15962
    Smoke: Gluster Build System <jenkins.org>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    CentOS-regression: Gluster Build System <jenkins.org>
    Reviewed-by: Saravanakumar Arumugam <sarumuga>

Comment 3 Kaushal 2017-03-08 10:20:37 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.9.1, please open a new bug report.

glusterfs-3.9.1 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://lists.gluster.org/pipermail/gluster-users/2017-January/029725.html
[2] https://www.gluster.org/pipermail/gluster-users/


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