diff options
| author | Susant Palai <spalai@redhat.com> | 2014-10-28 02:53:39 -0400 |
|---|---|---|
| committer | Kaushal M <kaushal@redhat.com> | 2014-10-29 05:46:11 -0700 |
| commit | c6e6b43b169b8452ee26121ce1ad0b0f07b512cf (patch) | |
| tree | 1f8fd73084495f248e58ef9616e05d451721cda9 /cli | |
| parent | 22e18a4c0c07a41fcdcde1ce0e90961b347d7a84 (diff) | |
CLI: Show warning message only for remove-brick commit
Earlier warning message for checking the removed-bricks for
any unmigrated files was thrown both for remove-brick commit
and force.
With this change the warning message will be shown only for
remove-brick commit.
Change-Id: Ib1fa47d831d5189088c77c5cf709a25ff36d7379
BUG: 1157974
Signed-off-by: Susant Palai <spalai@redhat.com>
Reviewed-on: http://review.gluster.org/8983
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Shyamsundar Ranganathan <srangana@redhat.com>
Reviewed-by: Kaushal M <kaushal@redhat.com>
Diffstat (limited to 'cli')
| -rw-r--r-- | cli/src/cli-rpc-ops.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/cli/src/cli-rpc-ops.c b/cli/src/cli-rpc-ops.c index 50f2f687a3a..5fd465fdf2f 100644 --- a/cli/src/cli-rpc-ops.c +++ b/cli/src/cli-rpc-ops.c @@ -2088,7 +2088,7 @@ gf_cli_remove_brick_cbk (struct rpc_req *req, struct iovec *iov, cli_out ("volume remove-brick %s: success", cmd_str); if (GF_OP_CMD_START == cmd && task_id_str != NULL) cli_out ("ID: %s", task_id_str); - if (GF_OP_CMD_COMMIT_FORCE == cmd || GF_OP_CMD_COMMIT == cmd) + if (GF_OP_CMD_COMMIT == cmd) cli_out ("Check the removed bricks to ensure all files " "are migrated.\nIf files with data are " "found on the brick path, copy them via a " |