Re: [PATCH] log: move "Corosync started" log messages

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Patch looks good, ACK (+ pushed).

Thanks,
  Honza

Kazunori INOUE napsal(a):
> Hi All,
> 
> I want corosync to send "Corosync Cluster Engine ... started" message
> to syslog after the syslog_facility parameter in corosync.conf became
> effective.
> Since this message is useful for us.
> 
> ---
>  exec/main.c |    6 +++---
>  1 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/exec/main.c b/exec/main.c
> index 6a5dc45..eeb295e 100644
> --- a/exec/main.c
> +++ b/exec/main.c
> @@ -1085,9 +1085,6 @@ int main (int argc, char **argv, char **envp)
>  
>  	corosync_mlockall ();
>  
> -	log_printf (LOGSYS_LEVEL_NOTICE, "Corosync Cluster Engine ('%s'): started and ready to provide service.", VERSION);
> -	log_printf (LOGSYS_LEVEL_INFO, "Corosync built-in features:" PACKAGE_FEATURES "");
> -
>  	/*
>  	 * Other signals are registered later via qb_loop_signal_add
>  	 */
> @@ -1129,6 +1126,9 @@ int main (int argc, char **argv, char **envp)
>  		corosync_exit_error (COROSYNC_DONE_LOGCONFIGREAD);
>  	}
>  
> +	log_printf (LOGSYS_LEVEL_NOTICE, "Corosync Cluster Engine ('%s'): started and ready to provide service.", VERSION);
> +	log_printf (LOGSYS_LEVEL_INFO, "Corosync built-in features:" PACKAGE_FEATURES "");
> +
>  	/*
>  	 * Make sure required directory is present
>  	 */
> ----
> Best Regards,
> Kazunori INOUE
> _______________________________________________
> discuss mailing list
> discuss@xxxxxxxxxxxx
> http://lists.corosync.org/mailman/listinfo/discuss

_______________________________________________
discuss mailing list
discuss@xxxxxxxxxxxx
http://lists.corosync.org/mailman/listinfo/discuss


[Index of Archives]     [Linux Clusters]     [Corosync Project]     [Linux USB Devel]     [Linux Audio Users]     [Photo]     [Yosemite News]    [Yosemite Photos]    [Linux Kernel]     [Linux SCSI]     [X.Org]

  Powered by Linux