summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorXavi Hernandez <xhernandez@redhat.com>2018-03-28 11:34:49 +0200
committerXavi Hernandez <xhernandez@redhat.com>2018-03-28 20:32:44 +0200
commitabb20b91136c2d03e54ab94e52bb88997396fdfb (patch)
tree23a10c55fcfa3c812f2a0e0dce1362e18a94bc2e
parent78a180afc67404f0937692290e1f54816136ad42 (diff)
cluster/ec: send list-node-uuids request to all subvolumes
The xattr trusted.glusterfs.list-node-uuids was only sent to a single subvolume. This was returning null uuids from the other subvolumes as if they were down. This fix forces that xattr to be requested from all subvolumes. Backport of: > BUG: 1561406 Change-Id: If62eb39a6857258923ba625e153d4ad79018ea2f BUG: 1561721 Signed-off-by: Xavi Hernandez <xhernandez@redhat.com>
-rw-r--r--tests/basic/ec/ec-rebalance.t1
-rw-r--r--xlators/cluster/ec/src/ec.c2
2 files changed, 2 insertions, 1 deletions
diff --git a/tests/basic/ec/ec-rebalance.t b/tests/basic/ec/ec-rebalance.t
index b5c3072..6cda3a3 100644
--- a/tests/basic/ec/ec-rebalance.t
+++ b/tests/basic/ec/ec-rebalance.t
@@ -14,6 +14,7 @@ cleanup
TEST glusterd
TEST pidof glusterd
TEST $CLI volume create $V0 disperse 3 redundancy 1 $H0:$B0/${V0}{0..2}
+TEST $CLI volume set $V0 lookup-optimize on
TEST $CLI volume start $V0
#Mount the volume
diff --git a/xlators/cluster/ec/src/ec.c b/xlators/cluster/ec/src/ec.c
index 429cac9..ebf15f7 100644
--- a/xlators/cluster/ec/src/ec.c
+++ b/xlators/cluster/ec/src/ec.c
@@ -931,7 +931,7 @@ ec_gf_getxattr (call_frame_t *frame, xlator_t *this, loc_t *loc,
if (name &&
((fnmatch (GF_XATTR_STIME_PATTERN, name, 0) == 0) ||
- (XATTR_IS_NODE_UUID(name)))) {
+ XATTR_IS_NODE_UUID(name) || XATTR_IS_NODE_UUID_LIST(name))) {
minimum = EC_MINIMUM_ALL;
}