diff options
| author | Kaushal M <kaushal@redhat.com> | 2012-08-13 18:57:20 +0530 | 
|---|---|---|
| committer | Anand Avati <avati@redhat.com> | 2012-08-20 00:16:38 -0700 | 
| commit | 7bf3723fba2dc7d2896fc2a6e5b5f65db0b8c179 (patch) | |
| tree | a632e5506bc672a86e18cedd848461c6f20aa92c /cli/src/cli-rpc-ops.c | |
| parent | 2f2e3bfb5ef89b5ba266a3df7496f95b11fb93e1 (diff) | |
cli: Proper xml output for "gluster peer status"
Change-Id: I5d72d3844aba0417652498f4dc706b4a68d36bd8
BUG: 847760
Signed-off-by: Kaushal M <kaushal@redhat.com>
Reviewed-on: http://review.gluster.org/3814
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Krishnan Parthasarathi <krishnan.parthasarathi@gmail.com>
Reviewed-by: Anand Avati <avati@redhat.com>
Diffstat (limited to 'cli/src/cli-rpc-ops.c')
| -rw-r--r-- | cli/src/cli-rpc-ops.c | 35 | 
1 files changed, 30 insertions, 5 deletions
diff --git a/cli/src/cli-rpc-ops.c b/cli/src/cli-rpc-ops.c index a1a243b7332..a7de5bb59b1 100644 --- a/cli/src/cli-rpc-ops.c +++ b/cli/src/cli-rpc-ops.c @@ -306,6 +306,7 @@ gf_cli_list_friends_cbk (struct rpc_req *req, struct iovec *iov,          int32_t                    port = 0;          int32_t                    connected = 0;          char                       *connected_str = NULL; +        char                       msg[1024] = {0,};          if (-1 == req->rpc_status) {                  goto out; @@ -327,7 +328,21 @@ gf_cli_list_friends_cbk (struct rpc_req *req, struct iovec *iov,          if (!rsp.op_ret) {                  if (!rsp.friends.friends_len) { -                        cli_err ("peer status: No peers present"); +                        snprintf (msg, sizeof (msg), +                                  "peer status: No peers present"); +#if (HAVE_LIB_XML) +                        if (global_state->mode & GLUSTER_MODE_XML) { +                                ret = cli_xml_output_peer_status (dict, +                                                                  rsp.op_ret, +                                                                  rsp.op_errno, +                                                                  msg); +                                if (ret) +                                        gf_log ("cli", GF_LOG_ERROR, +                                                "Error outputting to xml"); +                                goto out; +                        } +#endif +                        cli_err ("%s", msg);                          ret = 0;                          goto out;                  } @@ -351,9 +366,8 @@ gf_cli_list_friends_cbk (struct rpc_req *req, struct iovec *iov,  #if (HAVE_LIB_XML)                  if (global_state->mode & GLUSTER_MODE_XML) { -                        ret = cli_xml_output_dict ("peerStatus", dict, -                                                   rsp.op_ret, rsp.op_errno, -                                                   NULL); +                        ret = cli_xml_output_peer_status (dict, rsp.op_ret, +                                                          rsp.op_errno, msg);                          if (ret)                                  gf_log ("cli", GF_LOG_ERROR,                                          "Error outputting to xml"); @@ -411,7 +425,18 @@ gf_cli_list_friends_cbk (struct rpc_req *req, struct iovec *iov,                          i++;                  }          } else { -                ret = -1; +#if (HAVE_LIB_XML) +                if (global_state->mode & GLUSTER_MODE_XML) { +                        ret = cli_xml_output_peer_status (dict, rsp.op_ret, +                                                          rsp.op_errno, NULL); +                        if (ret) +                                gf_log ("cli", GF_LOG_ERROR, +                                        "Error outputting to xml"); +                } else +#endif +                { +                        ret = -1; +                }                  goto out;          }  | 
