diff options
| author | hari <hgowtham@redhat.com> | 2016-10-24 16:24:23 +0530 |
|---|---|---|
| committer | Dan Lambright <dlambrig@redhat.com> | 2016-10-25 04:55:39 -0700 |
| commit | 62238f9fb3511dc854651e694bb27b9ebce1e19e (patch) | |
| tree | 64d7e134e75a39729d35cd3899690b2245844d3c /cli/src/cli-cmd-volume.c | |
| parent | 054c294936845b1818d6e9b66514a832904e592e (diff) | |
CLI/TIER: throw warning regarding the removal of the older commands.
The older tier commands for attach tier and detach tier have to be
removed from code. This patch sends a warning asking to use new
command as older ones are depricated and will be removed.
Change-Id: Ie1c62947bad6ff106f40331ff6134838a6c72a7a
BUG: 1388062
Signed-off-by: hari <hgowtham@redhat.com>
Reviewed-on: http://review.gluster.org/15713
Tested-by: hari gowtham <hari.gowtham005@gmail.com>
Smoke: Gluster Build System <jenkins@build.gluster.org>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Dan Lambright <dlambrig@redhat.com>
Diffstat (limited to 'cli/src/cli-cmd-volume.c')
| -rw-r--r-- | cli/src/cli-cmd-volume.c | 28 |
1 files changed, 28 insertions, 0 deletions
diff --git a/cli/src/cli-cmd-volume.c b/cli/src/cli-cmd-volume.c index f7ccbf9b6c4..c47c002c3f0 100644 --- a/cli/src/cli-cmd-volume.c +++ b/cli/src/cli-cmd-volume.c @@ -1272,6 +1272,22 @@ cli_cmd_volume_tier_cbk (struct cli_state *state, int i = 0; eventtypes_t event = EVENT_LAST; gf_boolean_t aborted = _gf_false; + gf_answer_t answer = GF_ANSWER_NO; + + const char *detach_question = "gluster volume detach-tier <VOLNAME> " + "<start|stop|status|commit|force> is " + "deprecated. Use the new command \'" + "gluster volume tier <VOLNAME> detach <start|" + "stop|status|commit|force>\'\n" + "Do you want to Continue?"; + + const char *attach_question = "gluster volume attach-tier <VOLNAME> " + "[<replica COUNT>] <NEW-BRICK>... is " + "deprecated. Use the new command \'" + "gluster volume tier <VOLNAME> attach [<replica" + " COUNT>] <NEW-BRICK>... [force]\'\n" + "Do you want to Continue?"; + if (wordcount < 4) { cli_usage_out (word->pattern); @@ -1281,6 +1297,12 @@ cli_cmd_volume_tier_cbk (struct cli_state *state, } if (!strcmp(words[1], "detach-tier")) { + /* we need to ask question when older command is used */ + answer = cli_cmd_get_confirmation (state, detach_question); + if (GF_ANSWER_NO == answer) { + ret = 0; + goto out; + } ret = do_cli_cmd_volume_detach_tier (state, word, words, wordcount, &aborted); @@ -1306,6 +1328,12 @@ cli_cmd_volume_tier_cbk (struct cli_state *state, goto out; } else if (!strcmp(words[1], "attach-tier")) { + /* we need to ask question when the older command is used */ + answer = cli_cmd_get_confirmation (state, attach_question); + if (GF_ANSWER_NO == answer) { + ret = 0; + goto out; + } ret = do_cli_cmd_volume_attach_tier (state, word, words, wordcount); goto out; |