From 9b0374043ebe59fa638b8af330ea1d403503c143 Mon Sep 17 00:00:00 2001 From: Amar Tumballi Date: Wed, 13 Apr 2011 00:29:16 +0000 Subject: cli: add log file for cli instead of '/dev/null' Signed-off-by: Amar Tumballi Signed-off-by: Anand Avati BUG: 2737 (there is no log file for gluster cli) URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=2737 --- cli/src/cli.c | 20 ++++++++++++-------- 1 file changed, 12 insertions(+), 8 deletions(-) (limited to 'cli/src/cli.c') diff --git a/cli/src/cli.c b/cli/src/cli.c index d6c801967..93fcd6204 100644 --- a/cli/src/cli.c +++ b/cli/src/cli.c @@ -225,10 +225,6 @@ glusterfs_ctx_defaults_init (glusterfs_ctx_t *ctx) cmd_args = &ctx->cmd_args; - /* parsing command line arguments */ - cmd_args->log_file = "/dev/null"; - cmd_args->log_level = GF_LOG_NONE; - INIT_LIST_HEAD (&cmd_args->xlator_options); lim.rlim_cur = RLIM_INFINITY; @@ -242,10 +238,18 @@ glusterfs_ctx_defaults_init (glusterfs_ctx_t *ctx) static int logging_init (glusterfs_ctx_t *ctx) { + int ret = 0; cmd_args_t *cmd_args = NULL; cmd_args = &ctx->cmd_args; + /* CLI should not have something to DEBUG after the release, + hence defaulting to INFO loglevel */ + cmd_args->log_level = GF_LOG_INFO; + + ret = gf_asprintf (&cmd_args->log_file, + DEFAULT_CLI_LOG_FILE_DIRECTORY "/cli.log"); + if (gf_log_init (cmd_args->log_file) == -1) { fprintf (stderr, "ERROR: failed to open logfile %s\n", cmd_args->log_file); @@ -615,14 +619,14 @@ main (int argc, char *argv[]) return EPERM; } - global_rpc = cli_rpc_init (&state); - if (!global_rpc) - goto out; - ret = logging_init (ctx); if (ret) goto out; + global_rpc = cli_rpc_init (&state); + if (!global_rpc) + goto out; + ret = cli_cmds_register (&state); if (ret) goto out; -- cgit