|
 |
50dc83 |
From be925e84edcecd879e953bdb68c10f98825dba53 Mon Sep 17 00:00:00 2001
|
|
|
50dc83 |
From: Shwetha K Acharya <sacharya@redhat.com>
|
|
|
50dc83 |
Date: Mon, 3 Jun 2019 18:05:24 +0530
|
|
|
50dc83 |
Subject: [PATCH 186/192] cli: Remove-brick warning seems unnecessary
|
|
|
50dc83 |
|
|
|
50dc83 |
As force-migration option is disabled by default,
|
|
|
50dc83 |
the warning seems unnessary.
|
|
|
50dc83 |
|
|
|
50dc83 |
Rephrased the warning to make best sense out of it.
|
|
|
50dc83 |
|
|
|
50dc83 |
>fixes: bz#1712668
|
|
|
50dc83 |
>Change-Id: Ia18c3c5e7b3fec808fce2194ca0504a837708822
|
|
|
50dc83 |
>Signed-off-by: Shwetha K Acharya <sacharya@redhat.com>
|
|
|
50dc83 |
|
|
|
50dc83 |
backport of https://review.gluster.org/#/c/glusterfs/+/22805/
|
|
|
50dc83 |
|
|
|
50dc83 |
Bug: 1708183
|
|
|
50dc83 |
Change-Id: Ia18c3c5e7b3fec808fce2194ca0504a837708822
|
|
|
50dc83 |
Signed-off-by: Shwetha K Acharya <sacharya@redhat.com>
|
|
|
50dc83 |
Reviewed-on: https://code.engineering.redhat.com/gerrit/173447
|
|
|
50dc83 |
Tested-by: RHGS Build Bot <nigelb@redhat.com>
|
|
|
50dc83 |
Reviewed-by: Sunil Kumar Heggodu Gopala Acharya <sheggodu@redhat.com>
|
|
|
50dc83 |
---
|
|
|
50dc83 |
cli/src/cli-cmd-volume.c | 17 +++++++++--------
|
|
|
50dc83 |
1 file changed, 9 insertions(+), 8 deletions(-)
|
|
|
50dc83 |
|
|
|
50dc83 |
diff --git a/cli/src/cli-cmd-volume.c b/cli/src/cli-cmd-volume.c
|
|
|
50dc83 |
index 564aef7..a42e663 100644
|
|
|
50dc83 |
--- a/cli/src/cli-cmd-volume.c
|
|
|
50dc83 |
+++ b/cli/src/cli-cmd-volume.c
|
|
|
50dc83 |
@@ -2090,14 +2090,15 @@ cli_cmd_volume_remove_brick_cbk(struct cli_state *state,
|
|
|
50dc83 |
" on the volume.\nDo you want to continue?";
|
|
|
50dc83 |
} else if (command == GF_OP_CMD_START) {
|
|
|
50dc83 |
question =
|
|
|
50dc83 |
- "Running remove-brick with cluster.force-migration"
|
|
|
50dc83 |
- " enabled can result in data corruption. It is safer"
|
|
|
50dc83 |
- " to disable this option so that files that receive "
|
|
|
50dc83 |
- "writes during migration are not migrated.\nFiles "
|
|
|
50dc83 |
- "that are not migrated can then be manually copied "
|
|
|
50dc83 |
- "after the remove-brick commit operation.\nDo you "
|
|
|
50dc83 |
- "want to continue with your current "
|
|
|
50dc83 |
- "cluster.force-migration settings?";
|
|
|
50dc83 |
+ "It is recommended that remove-brick be run with"
|
|
|
50dc83 |
+ " cluster.force-migration option disabled to prevent"
|
|
|
50dc83 |
+ " possible data corruption. Doing so will ensure that"
|
|
|
50dc83 |
+ " files that receive writes during migration will not"
|
|
|
50dc83 |
+ " be migrated and will need to be manually copied"
|
|
|
50dc83 |
+ " after the remove-brick commit operation. Please"
|
|
|
50dc83 |
+ " check the value of the option and update accordingly."
|
|
|
50dc83 |
+ " \nDo you want to continue with your current"
|
|
|
50dc83 |
+ " cluster.force-migration settings?";
|
|
|
50dc83 |
}
|
|
|
50dc83 |
|
|
|
50dc83 |
if (!brick_count) {
|
|
|
50dc83 |
--
|
|
|
50dc83 |
1.8.3.1
|
|
|
50dc83 |
|