summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorVijay Bellur <vbellur@redhat.com>2013-04-23 00:58:22 +0530
committerAnand Avati <avati@redhat.com>2013-07-27 03:48:05 -0700
commitd16cdf141d449246b0b58870bd3d82efe232e5db (patch)
tree78543ba59f9d37f24ea5bfc4155b5125bdd18137
parent412940c56e203b16ebf027fe5b9cbf58cd3a144e (diff)
cli, glusterd: Cleanup logging of bd op commands.
This patch prevents messages of the form "bd op: %s : SUCCESS" from being logged in .cmd_log_history. Change-Id: Iebeb7e26d409bf99b9c8df0a5c1c5a5d30d78a61 BUG: 823081 Signed-off-by: Vijay Bellur <vbellur@redhat.com> Reviewed-on: http://review.gluster.org/4871 Reviewed-by: Krutika Dhananjay <kdhananj@redhat.com> Reviewed-by: M. Mohan Kumar <mohan@in.ibm.com> Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Anand Avati <avati@redhat.com>
-rw-r--r--cli/src/cli-rpc-ops.c8
-rw-r--r--xlators/mgmt/glusterd/src/glusterd-handler.c1
2 files changed, 4 insertions, 5 deletions
diff --git a/cli/src/cli-rpc-ops.c b/cli/src/cli-rpc-ops.c
index 0bab5d42b2b..959addf1a0c 100644
--- a/cli/src/cli-rpc-ops.c
+++ b/cli/src/cli-rpc-ops.c
@@ -3031,10 +3031,10 @@ gf_cli_bd_op (call_frame_t *frame, xlator_t *this,
&req.dict.dict_len);
- ret = cli_cmd_submit (&req, frame, cli_rpc_prog,
- GLUSTER_CLI_BD_OP, NULL,
- this, gf_cli_bd_op_cbk,
- (xdrproc_t) xdr_gf_cli_req);
+ ret = cli_to_glusterd (&req, frame, gf_cli_bd_op_cbk,
+ (xdrproc_t)xdr_gf_cli_req, dict,
+ GLUSTER_CLI_BD_OP, this, cli_rpc_prog,
+ NULL);
out:
if (dict)
diff --git a/xlators/mgmt/glusterd/src/glusterd-handler.c b/xlators/mgmt/glusterd/src/glusterd-handler.c
index 38427d82292..a74c52303be 100644
--- a/xlators/mgmt/glusterd/src/glusterd-handler.c
+++ b/xlators/mgmt/glusterd/src/glusterd-handler.c
@@ -1146,7 +1146,6 @@ __glusterd_handle_cli_bd_op (rpcsvc_request_t *req)
ret = glusterd_op_begin (req, GD_OP_BD_OP, dict, op_errstr,
sizeof (op_errstr));
- gf_cmd_log ("bd op: %s", ((ret == 0) ? "SUCCESS": "FAILED"));
out:
if (ret && dict)
dict_unref (dict);