diff options
author | Aravinda VK <avishwan@redhat.com> | 2013-12-02 15:19:17 +0530 |
---|---|---|
committer | Vijay Bellur <vbellur@redhat.com> | 2013-12-02 21:47:47 -0800 |
commit | 0e5837a4495028e56c75e95a694a0d6123623eb2 (patch) | |
tree | e5da8e9ac7c2a4f29a5b2ac82ae0fb3bcb8cf3db | |
parent | be380f35a2c9c356a8adcb9118e4f77ff9d1fec6 (diff) |
cli: xml: Rebalance status(xml) was empty when a glusterd down
When a glusterd is down in cluster rebalance/remove-brick status
--xml will fail to get status and returns null.
This patch skips collecting status if glusterd is down, and
collects status from all the other up nodes.
Change-Id: I6df0feef41b5cc817cc8d7820ee2acac95176a98
BUG: 1036564
Signed-off-by: Aravinda VK <avishwan@redhat.com>
Reviewed-on: http://review.gluster.org/6391
Reviewed-by: Krishnan Parthasarathi <kparthas@redhat.com>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
-rw-r--r-- | cli/src/cli-xml-output.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/cli/src/cli-xml-output.c b/cli/src/cli-xml-output.c index 6c7bc175237..68d69111410 100644 --- a/cli/src/cli-xml-output.c +++ b/cli/src/cli-xml-output.c @@ -3128,8 +3128,11 @@ cli_xml_output_vol_rebalance_status (xmlTextWriterPtr writer, dict_t *dict, memset (key, 0, sizeof (key)); snprintf (key, sizeof (key), "status-%d", i); ret = dict_get_int32 (dict, key, &status_rcd); + + /* If glusterd is down it fails to get the status, try + getting status from other nodes */ if (ret) - goto out; + continue; if (GF_DEFRAG_STATUS_NOT_STARTED == status_rcd) continue; |