diff options
author | Sanju Rakonde <srakonde@redhat.com> | 2019-01-04 15:49:53 +0530 |
---|---|---|
committer | Sanju Rakonde <srakonde@redhat.com> | 2019-01-04 18:13:46 +0530 |
commit | 95ec7391d8db26b933e542694add2b419dddf89f (patch) | |
tree | fb5bb5b3817e11972d23a643f4814b7015823b09 /NEWS | |
parent | f6d0141b47ac36f6d2b100252241ed64bc92f913 (diff) | |
download | glusterfs-95ec7391d8db26b933e542694add2b419dddf89f.tar.gz glusterfs-95ec7391d8db26b933e542694add2b419dddf89f.tar.xz glusterfs-95ec7391d8db26b933e542694add2b419dddf89f.zip |
glusterd: rebalance status should display information of localhost
Problem:
https://review.gluster.org/#/c/glusterfs/+/21762/ has migrated
rebalance commands from op-sm framework to mgmt v3 framework.
In a heterogenous cluster, if rebalance commands follow op-sm
framework, localhost information is not displayed in the
output of "gluster v rebalance <volname> status".
Cause:
Previously without https://review.gluster.org/#/c/glusterfs/+/21762/
rebalance commands were following op-sm framework.
In glusterd_volume_rebalance_use_rsp_dict() current_index variable
keeps track of number/count of peers in trusted storage pool.
In op-sm, glusterd_volume_rebalance_use_rsp_dict() will be called
only for the peers. So the current index should start from 2
assuming local host as node 1.
With the above patch, rebalance commands are following mgmt v3
framework. In mgmt v3, glusterd_volume_rebalance_use_rsp_dict()
is called for all nodes. For localhost it is called from
brick-op function and for peers it is called from brick-op
call back function. So the current index value should start
from 1.
https://review.gluster.org/#/c/glusterfs/+/21762/ has changed the
value of current index to 1. Because of this, In heterogenous cluster,
local host's information is overwritten by one of the peers information.
And rebalance status will not display localhost's information in
the output.
Solution: assign a value to current index based on a op-version
check.
Change-Id: I2dfba1f007e908cf160acc4a4a5d8ef672572e4d
fixes: bz#1663243
Signed-off-by: Sanju Rakonde <srakonde@redhat.com>
Diffstat (limited to 'NEWS')
0 files changed, 0 insertions, 0 deletions