From e45e0037f6df6a0fab846a83fb2c99bb09417cf4 Mon Sep 17 00:00:00 2001 From: susant Date: Wed, 24 Jul 2013 14:11:55 +0530 Subject: cli :remove-brick process output leads to ambiguity The output of remove-brick status as "Not started" leads to ambiguity.We should not show the status of the Server nodes which do not participate in the remove-brick process. Change-Id: I85fea40deb15f3e2dd5487d881f48c9aff7221de BUG: 986896 Signed-off-by: susant Reviewed-on: http://review.gluster.org/5383 Tested-by: Gluster Build System Reviewed-by: Vijay Bellur --- cli/src/cli-rpc-ops.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/cli/src/cli-rpc-ops.c b/cli/src/cli-rpc-ops.c index fd58bf81..28b92ee2 100644 --- a/cli/src/cli-rpc-ops.c +++ b/cli/src/cli-rpc-ops.c @@ -1894,9 +1894,12 @@ xml_output: } size_str = gf_uint64_2human_readable(size); - cli_out ("%40s %16"PRIu64 " %13s" " %13"PRIu64 " %13"PRIu64 - " %14s %16.2f", node_uuid, files, size_str, lookup, - failures, status, elapsed); + + if (strcmp (status, "not started")) { + cli_out ("%40s %16"PRIu64 " %13s" " %13"PRIu64 " %13" + PRIu64 " %14s %16.2f", node_uuid, files, + size_str, lookup, failures, status, elapsed); + } GF_FREE(size_str); i++; -- cgit