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

[renayama19661014 at ybb.ne.jp]

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 <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
>