Bug 1464078 - with AFR now making both nodes to return UUID for a file will result in georep consuming more resources
Summary: with AFR now making both nodes to return UUID for a file will result in geore...
Keywords:
Status: CLOSED EOL
Alias: None
Product: GlusterFS
Classification: Community
Component: replicate
Version: 3.10
Hardware: Unspecified
OS: Unspecified
unspecified
urgent
Target Milestone: ---
Assignee: Karthik U S
QA Contact:
URL:
Whiteboard: 3.3.0-devel-freeze-exception
Depends On: 1462790 1487647
Blocks: 1315781 glusterfs-3.10.4 1462693
TreeView+ depends on / blocked
 
Reported: 2017-06-22 11:50 UTC by Karthik U S
Modified: 2018-06-20 18:25 UTC (History)
11 users (show)

Fixed In Version: glusterfs-3.10.4
Clone Of: 1462693
Environment:
Last Closed: 2018-06-20 18:25:12 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Comment 1 Karthik U S 2017-06-22 11:51:06 UTC
Description of problem
======================
Previously when a read is requested, AFR was returning only one uuid out of the two replica bricks, but this meant that when a rebalance was triggered only one brick participating and hence load balancing was missing.
This problem was fixed with BZ#1315781 - AFR returns the node uuid of the same node for every file in the replica

However this fix didn't consider other scenarios and consumers of this facility.
for eg, georep too was relying on this service from AFR, and now with this service, it means that georep will be served by both replica bricks and hence syncing of data can happen twice for the same data. Now instead of one brick being passive and other being active, both turn active and hence lead to unnecessary resource utilization(eg CPU)

Comment 2 Worker Ant 2017-06-22 11:54:43 UTC
REVIEW: https://review.gluster.org/17603 (cluster/afr: Returning single and list of node uuids from AFR) posted (#1) for review on release-3.10 by Karthik U S (ksubrahm)

Comment 3 Worker Ant 2017-07-01 15:55:35 UTC
COMMIT: https://review.gluster.org/17603 committed in release-3.10 by Raghavendra Talur (rtalur) 
------
commit bc4d14d3af603c10f61f7d5bf45284ab29f6e494
Author: karthik-us <ksubrahm>
Date:   Mon Jun 19 19:54:36 2017 +0530

    cluster/afr: Returning single and list of node uuids from AFR
    
    Problem:
    The change in afr to return list of node uuids was causing problems
    with geo-rep.
    
    Fix:
    This patch will allow to get the single node uuid as it was doing
    before with the key "GF_XATTR_NODE_UUID_KEY", and will also allow
    to get the list of node uuids by using a new key
    "GF_XATTR_LIST_NODE_UUIDS_KEY". This will solve the problem with
    geo-rep and any other feature which were depending on this.
    
    > Change-Id: I09885dac6dfca127be94b708470c8c2941356f9a
    > BUG: 1462790
    > Signed-off-by: karthik-us <ksubrahm>
    > Reviewed-on: https://review.gluster.org/17576
    > Smoke: Gluster Build System <jenkins.org>
    > Reviewed-by: Ravishankar N <ravishankar>
    > Reviewed-by: Kotresh HR <khiremat>
    > NetBSD-regression: NetBSD Build System <jenkins.org>
    > CentOS-regression: Gluster Build System <jenkins.org>
    > Reviewed-by: Jeff Darcy <jeff.us>
    
    (cherry picked from commit 475ec9928ef96b63a0bfa859a9ae68709275033c)
    
    Change-Id: I5e741a48a426ee9a3cc69612051e0e9bcf33b500
    BUG: 1464078
    Signed-off-by: karthik-us <ksubrahm>
    Reviewed-on: https://review.gluster.org/17603
    Reviewed-by: Pranith Kumar Karampuri <pkarampu>
    Smoke: Gluster Build System <jenkins.org>
    CentOS-regression: Gluster Build System <jenkins.org>

Comment 4 Worker Ant 2017-07-03 06:51:51 UTC
REVIEW: https://review.gluster.org/17667 (cluster/ec: Node uuid xattr support update for EC) posted (#1) for review on release-3.10 by Sunil Kumar Acharya (sheggodu)

Comment 5 Worker Ant 2017-07-03 07:04:07 UTC
REVIEW: https://review.gluster.org/17667 (cluster/ec: Node uuid xattr support update for EC) posted (#2) for review on release-3.10 by Sunil Kumar Acharya (sheggodu)

Comment 6 Worker Ant 2017-09-01 10:08:16 UTC
REVIEW: https://review.gluster.org/17667 (cluster/ec: Node uuid xattr support update for EC) posted (#3) for review on release-3.10 by Sunil Kumar Acharya (sheggodu)

Comment 7 Ashish Pandey 2017-09-01 11:44:00 UTC

Patch for EC was also sent using this bug id. That has not been merged . So moving it to post and as soon as patch https://review.gluster.org/17667  will be merged, will move it to closed.

Comment 8 Shyamsundar 2018-06-20 18:25:12 UTC
This bug reported is against a version of Gluster that is no longer maintained (or has been EOL'd). See https://www.gluster.org/release-schedule/ for the versions currently maintained.

As a result this bug is being closed.

If the bug persists on a maintained version of gluster or against the mainline gluster repository, request that it be reopened and the Version field be marked appropriately.


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