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?
- 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 <[email protected]> wrote:
> On Fri, Aug 6, 2010 at 3:47 AM, <[email protected]> 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: [email protected]
> 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: [email protected]
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
