diff mbox series

ndctl, monitor: fix the sevrity of "daemon started" message

Message ID 20181005070124.6429-1-qi.fuli@jp.fujitsu.com (mailing list archive)
State New, archived
Headers show
Series ndctl, monitor: fix the sevrity of "daemon started" message | expand

Commit Message

QI Fuli Oct. 5, 2018, 7:01 a.m. UTC 
The "daemon started" message was printed as an error and this is improper.
This patch is used to chang it to info() meanwhile to chang the default log
level of monitor to "LOG_INFO".

Signed-off-by: QI Fuli <qi.fuli@jp.fujitsu.com>
---
 ndctl/monitor.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Verma, Vishal L Oct. 5, 2018, 8:01 p.m. UTC | #1 
On Fri, 2018-10-05 at 16:01 +0900, QI Fuli wrote:
> The "daemon started" message was printed as an error and this is improper.
> This patch is used to chang it to info() meanwhile to chang the default log
> level of monitor to "LOG_INFO".
> 
> Signed-off-by: QI Fuli <qi.fuli@jp.fujitsu.com>
> ---
>  ndctl/monitor.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Thanks Qi! Applied.

> 
> diff --git a/ndctl/monitor.c b/ndctl/monitor.c
> index d29e378..17465bb 100644
> --- a/ndctl/monitor.c
> +++ b/ndctl/monitor.c
> @@ -628,7 +628,7 @@ int cmd_monitor(int argc, const char **argv, void *ctx)
>  	if (monitor.verbose)
>  		ndctl_set_log_priority((struct ndctl_ctx *)ctx, LOG_DEBUG);
>  	else
> -		ndctl_set_log_priority((struct ndctl_ctx *)ctx, LOG_NOTICE);
> +		ndctl_set_log_priority((struct ndctl_ctx *)ctx, LOG_INFO);
>  
>  	rc = read_config_file((struct ndctl_ctx *)ctx, &monitor, &param);
>  	if (rc)
> @@ -660,7 +660,7 @@ int cmd_monitor(int argc, const char **argv, void *ctx)
>  			err((struct ndctl_ctx *)ctx, "daemon start failed\n");
>  			goto out;
>  		}
> -		err((struct ndctl_ctx *)ctx, "ndctl monitor daemon started\n");
> +		info((struct ndctl_ctx *)ctx, "ndctl monitor daemon started\n");
>  	}
>  
>  	if (parse_monitor_event(&monitor, (struct ndctl_ctx *)ctx))
diff mbox series

Patch

diff --git a/ndctl/monitor.c b/ndctl/monitor.c
index d29e378..17465bb 100644
--- a/ndctl/monitor.c
+++ b/ndctl/monitor.c
@@ -628,7 +628,7 @@  int cmd_monitor(int argc, const char **argv, void *ctx)
 	if (monitor.verbose)
 		ndctl_set_log_priority((struct ndctl_ctx *)ctx, LOG_DEBUG);
 	else
-		ndctl_set_log_priority((struct ndctl_ctx *)ctx, LOG_NOTICE);
+		ndctl_set_log_priority((struct ndctl_ctx *)ctx, LOG_INFO);
 
 	rc = read_config_file((struct ndctl_ctx *)ctx, &monitor, &param);
 	if (rc)
@@ -660,7 +660,7 @@  int cmd_monitor(int argc, const char **argv, void *ctx)
 			err((struct ndctl_ctx *)ctx, "daemon start failed\n");
 			goto out;
 		}
-		err((struct ndctl_ctx *)ctx, "ndctl monitor daemon started\n");
+		info((struct ndctl_ctx *)ctx, "ndctl monitor daemon started\n");
 	}
 
 	if (parse_monitor_event(&monitor, (struct ndctl_ctx *)ctx))