|
 |
f338ef |
From 039a3f81209706261fc809eac94564e81a3377da Mon Sep 17 00:00:00 2001
|
|
|
f338ef |
From: Sanju Rakonde <srakonde@redhat.com>
|
|
|
f338ef |
Date: Wed, 25 Sep 2019 14:55:19 +0530
|
|
|
f338ef |
Subject: [PATCH 300/302] cli: remove the warning displayed when remove brick
|
|
|
f338ef |
start issued
|
|
|
f338ef |
|
|
|
f338ef |
remove-brick start command gives displays below error:
|
|
|
f338ef |
|
|
|
f338ef |
It is recommended that remove-brick be run with cluster.force-migration
|
|
|
f338ef |
option disabled to prevent possible data corruption. Doing so will ensure
|
|
|
f338ef |
that files that receive writes during migration will not be migrated and
|
|
|
f338ef |
will need to be manually copied after the remove-brick commit operation.
|
|
|
f338ef |
Please check the value of the option and update accordingly.
|
|
|
f338ef |
Do you want to continue with your current cluster.force-migration settings? (y/n)
|
|
|
f338ef |
|
|
|
f338ef |
As we are not qualifying cluster.force-migration for 3.5.0,
|
|
|
f338ef |
we should not display this message. So, removing it.
|
|
|
f338ef |
|
|
|
f338ef |
Label: DOWNSTREAM ONLY
|
|
|
f338ef |
|
|
|
f338ef |
BUG: 1755227
|
|
|
f338ef |
Change-Id: I409f2059d43c5e867788f19d2ccb8d6d839520f7
|
|
|
f338ef |
fixes: bz#1755227
|
|
|
f338ef |
Signed-off-by: Sanju Rakonde <srakonde@redhat.com>
|
|
|
f338ef |
Reviewed-on: https://code.engineering.redhat.com/gerrit/182009
|
|
|
f338ef |
Tested-by: RHGS Build Bot <nigelb@redhat.com>
|
|
|
f338ef |
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
|
|
|
f338ef |
---
|
|
|
f338ef |
cli/src/cli-cmd-parser.c | 2 --
|
|
|
f338ef |
cli/src/cli-cmd-volume.c | 11 -----------
|
|
|
f338ef |
2 files changed, 13 deletions(-)
|
|
|
f338ef |
|
|
|
f338ef |
diff --git a/cli/src/cli-cmd-parser.c b/cli/src/cli-cmd-parser.c
|
|
|
f338ef |
index 92ceb8e..4456a7b 100644
|
|
|
f338ef |
--- a/cli/src/cli-cmd-parser.c
|
|
|
f338ef |
+++ b/cli/src/cli-cmd-parser.c
|
|
|
f338ef |
@@ -2101,8 +2101,6 @@ cli_cmd_volume_remove_brick_parse(struct cli_state *state, const char **words,
|
|
|
f338ef |
wordcount--;
|
|
|
f338ef |
if (!strcmp("start", w)) {
|
|
|
f338ef |
command = GF_OP_CMD_START;
|
|
|
f338ef |
- if (question)
|
|
|
f338ef |
- *question = 1;
|
|
|
f338ef |
} else if (!strcmp("commit", w)) {
|
|
|
f338ef |
command = GF_OP_CMD_COMMIT;
|
|
|
f338ef |
} else if (!strcmp("stop", w)) {
|
|
|
f338ef |
diff --git a/cli/src/cli-cmd-volume.c b/cli/src/cli-cmd-volume.c
|
|
|
f338ef |
index a42e663..6b958bd 100644
|
|
|
f338ef |
--- a/cli/src/cli-cmd-volume.c
|
|
|
f338ef |
+++ b/cli/src/cli-cmd-volume.c
|
|
|
f338ef |
@@ -2088,17 +2088,6 @@ cli_cmd_volume_remove_brick_cbk(struct cli_state *state,
|
|
|
f338ef |
"Remove-brick force will not migrate files from the "
|
|
|
f338ef |
"removed bricks, so they will no longer be available"
|
|
|
f338ef |
" on the volume.\nDo you want to continue?";
|
|
|
f338ef |
- } else if (command == GF_OP_CMD_START) {
|
|
|
f338ef |
- question =
|
|
|
f338ef |
- "It is recommended that remove-brick be run with"
|
|
|
f338ef |
- " cluster.force-migration option disabled to prevent"
|
|
|
f338ef |
- " possible data corruption. Doing so will ensure that"
|
|
|
f338ef |
- " files that receive writes during migration will not"
|
|
|
f338ef |
- " be migrated and will need to be manually copied"
|
|
|
f338ef |
- " after the remove-brick commit operation. Please"
|
|
|
f338ef |
- " check the value of the option and update accordingly."
|
|
|
f338ef |
- " \nDo you want to continue with your current"
|
|
|
f338ef |
- " cluster.force-migration settings?";
|
|
|
f338ef |
}
|
|
|
f338ef |
|
|
|
f338ef |
if (!brick_count) {
|
|
|
f338ef |
--
|
|
|
f338ef |
1.8.3.1
|
|
|
f338ef |
|