From 1222e2ae2d42cf12569a0cf00bb1f1504ef4e7b7 Mon Sep 17 00:00:00 2001 From: shishir gowda Date: Wed, 4 Apr 2012 16:04:10 +0530 Subject: remove-brick: Abort should revert volume changes Also, enhanced gluster cli output. Signed-off-by: shishir gowda Change-Id: I520ccfb63b3568a2e6654d8a5fbf265b9b903af1 BUG: 788524 Reviewed-on: http://review.gluster.com/3082 Tested-by: Gluster Build System Reviewed-by: Vijay Bellur --- cli/src/cli-rpc-ops.c | 43 ++++++++++++++++++++++++++++++++++++++++++- 1 file changed, 42 insertions(+), 1 deletion(-) (limited to 'cli') diff --git a/cli/src/cli-rpc-ops.c b/cli/src/cli-rpc-ops.c index 70c9a3d80..779508ad7 100644 --- a/cli/src/cli-rpc-ops.c +++ b/cli/src/cli-rpc-ops.c @@ -1557,6 +1557,9 @@ gf_cli3_1_remove_brick_cbk (struct rpc_req *req, struct iovec *iov, gf_cli_rsp rsp = {0,}; int ret = -1; char msg[1024] = {0,}; + gf1_op_commands cmd = GF_OP_CMD_NONE; + dict_t *dict = NULL; + char *cmd_str = "unknown"; if (-1 == req->rpc_status) { goto out; @@ -1568,12 +1571,47 @@ gf_cli3_1_remove_brick_cbk (struct rpc_req *req, struct iovec *iov, goto out; } + dict = dict_new (); + + ret = dict_unserialize (rsp.dict.dict_val, rsp.dict.dict_len, &dict); + if (ret) { + gf_log ("", GF_LOG_ERROR, "failed to unserialize rsp to dict"); + goto out; + } + ret = dict_get_int32 (dict, "command", (int32_t *)&cmd); + if (ret) { + gf_log ("", GF_LOG_ERROR, "failed to get command"); + goto out; + } + + switch (cmd) { + + case GF_OP_CMD_START: + cmd_str = "start"; + break; + case GF_OP_CMD_COMMIT: + cmd_str = "commit"; + break; + case GF_OP_CMD_ABORT: + cmd_str = "abort"; + break; + case GF_OP_CMD_PAUSE: + cmd_str = "pause"; + break; + case GF_OP_CMD_COMMIT_FORCE: + cmd_str = "commit force"; + break; + default: + cmd_str = "unkown"; + break; + } + gf_log ("cli", GF_LOG_INFO, "Received resp to remove brick"); if (rsp.op_ret && strcmp (rsp.op_errstr, "")) snprintf (msg, sizeof (msg), "%s", rsp.op_errstr); else - snprintf (msg, sizeof (msg), "Remove Brick %s", + snprintf (msg, sizeof (msg), "Remove Brick %s %s", cmd_str, (rsp.op_ret) ? "unsuccessful": "successful"); #if (HAVE_LIB_XML) @@ -1595,6 +1633,9 @@ out: free (rsp.dict.dict_val); if (rsp.op_errstr) free (rsp.op_errstr); + if (dict) + dict_unref(dict); + return ret; } -- cgit