good for merge

On Fri, 2010-02-19 at 11:56 -0600, David Teigland wrote:
> The corosync logs are so full of these messages that they end up being
> unhelpful.  I think they could be made very helpful, though, if they were
> printed when the quorum state changed.
> 
> Dave
> 
> Index: exec/vsf_quorum.c
> ===================================================================
> --- exec/vsf_quorum.c (revision 2662)
> +++ exec/vsf_quorum.c (working copy)
> @@ -135,11 +135,12 @@
>                                 size_t view_list_entries,
>                                 int quorum, struct memb_ring_id *ring_id)
>  {
> +     int old_quorum = primary_designated;
>       primary_designated = quorum;
>  
> -     if (primary_designated) {
> +     if (primary_designated && !old_quorum) {
>               log_printf (LOGSYS_LEVEL_NOTICE, "This node is within the 
> primary component and will provide service.\n");
> -     } else {
> +     } else if (!primary_designated && old_quorum) {
>               log_printf (LOGSYS_LEVEL_NOTICE, "This node is within the 
> non-primary component and will NOT provide any services.\n");
>       }
>  
> _______________________________________________
> Openais mailing list
> [email protected]
> https://lists.linux-foundation.org/mailman/listinfo/openais

_______________________________________________
Openais mailing list
[email protected]
https://lists.linux-foundation.org/mailman/listinfo/openais

Reply via email to