diff options
author | Avra Sengupta <asengupt@redhat.com> | 2012-12-19 11:58:15 +0530 |
---|---|---|
committer | Vijay Bellur <vbellur@redhat.com> | 2012-12-19 00:14:35 -0800 |
commit | 09e4d0861c8df392dc19dc6c9656dd33571f260d (patch) | |
tree | 643b6e9601b2a43215af97d86de5e4d35e092da4 /cli | |
parent | c9c0fe3f7d737b175dea3833071f3e54d2f412fa (diff) |
cli: Fixing the xml output in failure cases for gluster peer detach
Change-Id: I87aadec1949fccadde396b95f31eeb6832e461b4
BUG: 879494
Signed-off-by: Avra Sengupta <asengupt@redhat.com>
Reviewed-on: http://review.gluster.org/4336
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'cli')
-rw-r--r-- | cli/src/cli-rpc-ops.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/cli/src/cli-rpc-ops.c b/cli/src/cli-rpc-ops.c index a5314378c25..dcbe074cbf6 100644 --- a/cli/src/cli-rpc-ops.c +++ b/cli/src/cli-rpc-ops.c @@ -270,8 +270,10 @@ gf_cli_deprobe_cbk (struct rpc_req *req, struct iovec *iov, } if (global_state->mode & GLUSTER_MODE_XML) { - ret = cli_xml_output_str ("peerDetach", msg, rsp.op_ret, - rsp.op_errno, NULL); + ret = cli_xml_output_str (NULL, + (rsp.op_ret)? NULL : msg, + rsp.op_ret, rsp.op_errno, + (rsp.op_ret)? msg : NULL); if (ret) gf_log ("cli", GF_LOG_ERROR, "Error outputting to xml"); |