andy/glusterfs
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

CLI: Show warning message only for remove-brick commit

Browse Source 
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>
This commit is contained in:
Susant Palai 2014-10-28 02:53:39 -04:00 committed by Kaushal M
parent 22e18a4c0c
commit c6e6b43b16
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
cli/src/cli-rpc-ops.c
View File

@ -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 "