[Pacemaker] [PATCH]The changing of the log level of pengine process.

[Andrew Beekhof]

On Fri, Aug 27, 2010 at 3:03 AM,  <renayama19661014 at ybb.ne.jp> wrote:
> Hi Andrew,
>
> Thank you for comment.
>
>> Why not simply remove the if(was_processing_error)  block?
>> Its just a summary message, the place that set was_processing_error
>> will also have logged an error.
>
> Is this meaning to abolish the next code?

Right.
Though I still don't see what good it does to remove it :-)

2 errors vs. 1.

>
> -               if(was_processing_error) {
> -                       crm_err("Transition %d:"
> -                               " ERRORs found during PE processing."
> -                               " PEngine Input stored in: %s",
> -                               transition_id, filename);
> -
>                } else if(was_processing_warning) {
>
> Best Regards,
> Hideo Yamauchi.
>
>
> --- Andrew Beekhof <andrew at beekhof.net> wrote:
>
>> On Fri, Aug 6, 2010 at 3:47 AM,  <renayama19661014 at ybb.ne.jp> wrote:
>> > Hi Andrew,
>> >
>> > Thank you for comment.
>> >
>> >> np :-)
>> >>
>> >> Maybe it would be easier to show the logs and/or crm_mon output with
>> >> and without the patch.
>> >
>> > However, our many users watch error log.
>> > And some users do not like trouble to be notified of in error log in this situation.
>> >
>> > After all is this patch too special?
>>
>> I'm not sure, I'm still trying to figure out what it does.
>> It does seem like a lot of effort to go to reduce the number of logs
>> containing ERROR: by 1.
>>
>> Why not simply remove the if(was_processing_error)  block?
>> Its just a summary message, the place that set was_processing_error
>> will also have logged an error.
>>
>> _______________________________________________
>> Pacemaker mailing list: Pacemaker at oss.clusterlabs.org
>> http://oss.clusterlabs.org/mailman/listinfo/pacemaker
>>
>> Project Home: http://www.clusterlabs.org
>> Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
>> Bugs: http://developerbugs.linux-foundation.org/enter_bug.cgi?product=Pacemaker
>>
>
>
> _______________________________________________
> Pacemaker mailing list: Pacemaker at oss.clusterlabs.org
> http://oss.clusterlabs.org/mailman/listinfo/pacemaker
>
> Project Home: http://www.clusterlabs.org
> Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
> Bugs: http://developerbugs.linux-foundation.org/enter_bug.cgi?product=Pacemaker
>