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)
REVIEW: https://review.gluster.org/17602 (cluster/afr: Returning single and list of node uuids from AFR) posted (#1) for review on release-3.11 by Karthik U S (ksubrahm)
REVIEW: https://review.gluster.org/17615 (cluster/ec: Node uuid xattr support update for EC) posted (#1) for review on release-3.11 by Pranith Kumar Karampuri (pkarampu)
COMMIT: https://review.gluster.org/17602 committed in release-3.11 by Shyamsundar Ranganathan (srangana) ------ commit 63be8940a73cc92b54e8cee20df5df19bdb9daee 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: I6f6e8320a1eb5909ef601e23f8a8d3499807e319 BUG: 1463250 Signed-off-by: karthik-us <ksubrahm> Reviewed-on: https://review.gluster.org/17602 Reviewed-by: Pranith Kumar Karampuri <pkarampu> Smoke: Gluster Build System <jenkins.org> CentOS-regression: Gluster Build System <jenkins.org>
REVIEW: https://review.gluster.org/17615 (cluster/ec: Node uuid xattr support update for EC) posted (#2) for review on release-3.11 by Pranith Kumar Karampuri (pkarampu)
COMMIT: https://review.gluster.org/17615 committed in release-3.11 by Shyamsundar Ranganathan (srangana) ------ commit 91c44b41654ae10a2278ee8c494910a46196077a Author: Sunil Kumar Acharya <sheggodu> Date: Wed Jun 21 16:37:09 2017 +0530 cluster/ec: Node uuid xattr support update for EC Backport of https://review.gluster.org/17594 Problem: The change in EC to return list of node uuids for GF_XATTR_NODE_UUID_KEY 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 features which were depending on this. BUG: 1463250 Change-Id: I2d9214a9658d4a41a3d6de08600884d2bda5f3eb Signed-off-by: Sunil Kumar Acharya <sheggodu> Reviewed-on: https://review.gluster.org/17615 Tested-by: Pranith Kumar Karampuri <pkarampu> Smoke: Gluster Build System <jenkins.org> NetBSD-regression: NetBSD Build System <jenkins.org> Reviewed-by: Xavier Hernandez <xhernandez> CentOS-regression: Gluster Build System <jenkins.org> Reviewed-by: Pranith Kumar Karampuri <pkarampu>
REVIEW: https://review.gluster.org/17627 (cluster/rebalance: Use GF_XATTR_LIST_NODE_UUIDS_KEY to figure out local subvols.) posted (#1) for review on release-3.11 by N Balachandran (nbalacha)
COMMIT: https://review.gluster.org/17627 committed in release-3.11 by Shyamsundar Ranganathan (srangana) ------ commit f67dbab74a3d0a13a8f7ed31b8cc840c05042913 Author: Susant Palai <spalai> Date: Wed Jun 21 17:52:45 2017 +0530 cluster/rebalance: Use GF_XATTR_LIST_NODE_UUIDS_KEY to figure out local subvols. Afr has introduced a new key GF_XATTR_LIST_NODE_UUIDS_KEY, through which rebalance will figure out its local subvolumes.(Reference bugid=1463250) key: GF_XATTR_NODE_UUID_KEY will continue to serve it's old purpose of returning the first afr chiild. test: prove tests/basic/distribute/rebal-all-nodes-migrate.t > BUG: 1463648 > Signed-off-by: Susant Palai <spalai> Change-Id: I4d602feda2a05b29d2210c712a07a4ac6b8bc112 BUG: 1463250 Signed-off-by: N Balachandran <nbalacha> Reviewed-on: https://review.gluster.org/17627 Smoke: Gluster Build System <jenkins.org> CentOS-regression: Gluster Build System <jenkins.org> Reviewed-by: Shyamsundar Ranganathan <srangana>
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.11.1, please open a new bug report. glusterfs-3.11.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/announce/2017-June/000074.html [2] https://www.gluster.org/pipermail/gluster-users/