summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorN Balachandran <nbalacha@redhat.com>2017-07-24 17:48:47 +0530
committerN Balachandran <nbalacha@redhat.com>2017-07-25 10:03:24 +0000
commit8c3e766fe0a473734e8eca0f70d0318a2b909e2e (patch)
tree15f86f47b6ba0057a3155832c32282f357946171
parent8dcf91660e0bd10eb75ef25a29ca02ec51c81be4 (diff)
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. Change-Id: Iee778547eb7370a8069e954b5d629fcedf54e59b BUG: 1474318 Signed-off-by: N Balachandran <nbalacha@redhat.com> Reviewed-on: https://review.gluster.org/17861 Reviewed-by: Susant Palai <spalai@redhat.com> Smoke: Gluster Build System <jenkins@build.gluster.org> CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
-rw-r--r--xlators/cluster/dht/src/dht-rebalance.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/xlators/cluster/dht/src/dht-rebalance.c b/xlators/cluster/dht/src/dht-rebalance.c
index 6a9153f794d..35e19e5af7c 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;