diff options
| author | Mohit Agrawal <moagrawal@redhat.com> | 2019-07-03 15:22:38 +0530 |
|---|---|---|
| committer | Atin Mukherjee <amukherj@redhat.com> | 2019-07-04 08:25:23 +0000 |
| commit | af989db23d1db00e087f2b9d3dfc43b13ef17153 (patch) | |
| tree | 52a621c083eebc4e41206ea126541db9a836eb5f /xlators/mgmt/glusterd/src/glusterd-handler.c | |
| parent | 7d270d05f835d41e32572501246b50181dc9be56 (diff) | |
glusterd: Show the correct brick status in get-state
Problem: get-state does not show correct brick status if brick
status is not Started, it always shows started if any value
is set brickinfo->status
Solution: Check the value of brickinfo->status to show correct status
in get-state
Change-Id: I12a79619024c2cf59f338220d144f2f034059b3b
fixes: bz#1726906
Signed-off-by: Mohit Agrawal <moagrawal@redhat.com>
Diffstat (limited to 'xlators/mgmt/glusterd/src/glusterd-handler.c')
| -rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-handler.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-handler.c b/xlators/mgmt/glusterd/src/glusterd-handler.c index a0bf409d872..9002ef55b03 100644 --- a/xlators/mgmt/glusterd/src/glusterd-handler.c +++ b/xlators/mgmt/glusterd/src/glusterd-handler.c @@ -5335,7 +5335,9 @@ glusterd_get_state(rpcsvc_request_t *req, dict_t *dict) char vol_status_str[STATUS_STRLEN] = { 0, }; - + char brick_status_str[STATUS_STRLEN] = { + 0, + }; this = THIS; GF_VALIDATE_OR_GOTO(THIS->name, this, out); @@ -5585,8 +5587,9 @@ glusterd_get_state(rpcsvc_request_t *req, dict_t *dict) brickinfo->rdma_port); fprintf(fp, "Volume%d.Brick%d.port_registered: %d\n", count_bkp, count, brickinfo->port_registered); + glusterd_brick_get_status_str(brickinfo, brick_status_str); fprintf(fp, "Volume%d.Brick%d.status: %s\n", count_bkp, count, - brickinfo->status ? "Started" : "Stopped"); + brick_status_str); ret = sys_statvfs(brickinfo->path, &brickstat); if (ret) { |