summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPranith Kumar K <pkarampu@redhat.com>2013-12-23 11:56:03 +0530
committerVijay Bellur <vbellur@redhat.com>2013-12-23 02:35:58 -0800
commitd6f687084d94a17abf505b8d0bf315d18bf937ee (patch)
tree110e1fadb5c552aabe67a8c55461d01a5d7e4798
parente19add96b3906c2911d8b568a0ebcfa44c5a6950 (diff)
cli: Throw a warning during replace-brick
Change-Id: Iae59365f09bf64a5927edeeb4c3c052e237eee38 BUG: 1039954 Signed-off-by: Pranith Kumar K <pkarampu@redhat.com> Reviewed-on: http://review.gluster.org/6560 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Vijay Bellur <vbellur@redhat.com>
-rw-r--r--cli/src/cli-cmd-volume.c14
1 files changed, 14 insertions, 0 deletions
diff --git a/cli/src/cli-cmd-volume.c b/cli/src/cli-cmd-volume.c
index e693f11..528cd5e 100644
--- a/cli/src/cli-cmd-volume.c
+++ b/cli/src/cli-cmd-volume.c
@@ -1150,6 +1150,11 @@ cli_cmd_volume_replace_brick_cbk (struct cli_state *state,
int sent = 0;
int parse_error = 0;
cli_local_t *local = NULL;
+ int replace_op = 0;
+ char *q = "All replace-brick commands except "
+ "commit force are deprecated. "
+ "Do you want to continue?";
+ gf_answer_t answer = GF_ANSWER_NO;
#ifdef GF_SOLARIS_HOST_OS
cli_out ("Command not supported on Solaris");
@@ -1169,6 +1174,15 @@ cli_cmd_volume_replace_brick_cbk (struct cli_state *state,
goto out;
}
+ ret = dict_get_int32 (options, "operation", &replace_op);
+ if (replace_op != GF_REPLACE_OP_COMMIT_FORCE) {
+ answer = cli_cmd_get_confirmation (state, q);
+ if (GF_ANSWER_NO == answer) {
+ ret = 0;
+ goto out;
+ }
+ }
+
if (state->mode & GLUSTER_MODE_SCRIPT) {
ret = dict_set_int32 (options, "force", _gf_true);
if (ret) {