From ef169f5029bd31601d691cbee5ef5c0117bfcfef Mon Sep 17 00:00:00 2001 From: N Balachandran Date: Mon, 24 Jul 2017 17:48:47 +0530 Subject: [PATCH 572/576] cluster/dht: Correct iterator for decommissioned bricks Corrected the iterator for looping over the list of decommissioned bricks while checking if the new target determined because of min-free-disk values has been decommissioned. > BUG: 1474318 > Signed-off-by: N Balachandran > Reviewed-on: https://review.gluster.org/17861 > Reviewed-by: Susant Palai Change-Id: Iee778547eb7370a8069e954b5d629fcedf54e59b BUG: 1474284 Signed-off-by: N Balachandran Reviewed-on: https://code.engineering.redhat.com/gerrit/113511 Reviewed-by: Susant Palai Reviewed-by: Atin Mukherjee --- xlators/cluster/dht/src/dht-rebalance.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xlators/cluster/dht/src/dht-rebalance.c b/xlators/cluster/dht/src/dht-rebalance.c index c60ce27..9ae4062 100644 --- a/xlators/cluster/dht/src/dht-rebalance.c +++ b/xlators/cluster/dht/src/dht-rebalance.c @@ -1023,7 +1023,7 @@ find_new_subvol: * removed-brick, do not mark the error as failure */ if (conf->decommission_subvols_cnt) { *ignore_failure = _gf_true; - for (i = 0; i < conf->decommission_subvols_cnt; i++) { + for (i = 0; i < conf->subvolume_cnt; i++) { if (conf->decommissioned_bricks[i] == from) { *ignore_failure = _gf_false; break; -- 1.8.3.1