From ccb906c8a8139b66130192080020709bfc6d0dc3 Mon Sep 17 00:00:00 2001 From: Dan Lambright Date: Thu, 19 May 2016 10:43:53 -0400 Subject: cluster/tier: fix detach tier error message Do not refer to obsolete syntax when throwing an error on detach tier. Throw a warning if the user tries to commit a detach tier before starting (or completing) the decommission process. Change-Id: I9df28c1b42b8ab1790e1db5c0cd518432146c1d1 BUG: 1337227 Signed-off-by: Dan Lambright Reviewed-on: http://review.gluster.org/14438 Smoke: Gluster Build System NetBSD-regression: NetBSD Build System CentOS-regression: Gluster Build System Reviewed-by: Atin Mukherjee --- cli/src/cli-cmd-parser.c | 2 +- cli/src/cli-cmd-volume.c | 2 +- cli/src/cli-rpc-ops.c | 10 +++++----- 3 files changed, 7 insertions(+), 7 deletions(-) (limited to 'cli') diff --git a/cli/src/cli-cmd-parser.c b/cli/src/cli-cmd-parser.c index b062adfab0f..f60e5fb04a0 100644 --- a/cli/src/cli-cmd-parser.c +++ b/cli/src/cli-cmd-parser.c @@ -1848,7 +1848,7 @@ cli_cmd_volume_detach_tier_parse (const char **words, int wordcount, ret = 0; out: if (ret) { - gf_log ("cli", GF_LOG_ERROR, "Unable to parse detach-tier CLI"); + gf_log ("cli", GF_LOG_ERROR, "Unable to parse detach tier CLI"); if (dict) dict_unref (dict); } diff --git a/cli/src/cli-cmd-volume.c b/cli/src/cli-cmd-volume.c index 689eba6d281..b09a6ccbe3d 100644 --- a/cli/src/cli-cmd-volume.c +++ b/cli/src/cli-cmd-volume.c @@ -1023,7 +1023,7 @@ out: if (ret) { cli_cmd_sent_status_get (&sent); if ((sent == 0) && (parse_error == 0)) - cli_out ("Volume detach-tier failed"); + cli_out ("Volume detach tier failed"); } CLI_STACK_DESTROY (frame); diff --git a/cli/src/cli-rpc-ops.c b/cli/src/cli-rpc-ops.c index 67f26991419..12776065128 100644 --- a/cli/src/cli-rpc-ops.c +++ b/cli/src/cli-rpc-ops.c @@ -2396,7 +2396,7 @@ gf_cli_detach_tier_cbk (struct rpc_req *req, struct iovec *iov, break; } - gf_log ("cli", GF_LOG_INFO, "Received resp to detach-tier"); + gf_log ("cli", GF_LOG_INFO, "Received resp to detach tier"); if (rsp.op_ret && strcmp (rsp.op_errstr, "")) snprintf (msg, sizeof (msg), "%s", rsp.op_errstr); @@ -2418,10 +2418,10 @@ gf_cli_detach_tier_cbk (struct rpc_req *req, struct iovec *iov, } if (rsp.op_ret) { - cli_err ("volume detach-tier %s: failed: %s", cmd_str, + cli_err ("volume detach tier %s: failed: %s", cmd_str, msg); } else { - cli_out ("volume detach-tier %s: success", cmd_str); + cli_out ("volume detach tier %s: success", cmd_str); if (GF_OP_CMD_DETACH_START == cmd && task_id_str != NULL) cli_out ("ID: %s", task_id_str); if (GF_OP_CMD_DETACH_COMMIT == cmd) @@ -2558,10 +2558,10 @@ xml_output: } if ((cmd == GF_OP_CMD_STOP_DETACH_TIER) && (rsp.op_ret == 0)) { - cli_out ("'detach-tier' process may be in the middle of a " + cli_out ("'detach tier' process may be in the middle of a " "file migration.\nThe process will be fully stopped " "once the migration of the file is complete.\nPlease " - "check detach-tier process for completion before " + "check detach tier process for completion before " "doing any further brick related tasks on the " "volume."); } -- cgit