<html><body>
<p><tt>andrew@beekhof.net wrote on 06/09/2009 00:00:20 AM:<br>
<br>
> You might find:<br>
> <br>
> + syslog (LOG_INFO, "Event id: "U64T"\n", event_id);<br>
> + syslog (LOG_INFO, "Log timestamp: %s\n", ctime (&(event->time_logged)));<br>
> + syslog (LOG_INFO, "Event timestamp: %s\n", ctime (&(event->time_event)));<br>
> <br>
> to be quite noisy. Perhaps LOG_DEBUG and/or combining them into a<br>
> single line would be a good idea.<br>
> <br>
> + if (execvp (attrd_updater_args[0], attrd_updater_args) == -1)<br>
> <br>
> I think instead of exec'ing attrd_updater, you might as well link<br>
> against libcrmcommon.so and use the attrd_update_no_mainloop() API<br>
> call.<br>
</tt><br>
<tt>Here is a new version of the patch with those issues addressed:</tt><br>
<br>
<i>(See attached file: pacemaker.mark.patch)</i><tt><br>
</tt><br>
Mark</body></html>