On Mon, 2008-11-10 at 22:47 -0700, Steven Dake wrote:
> On Tue, 2008-11-11 at 05:55 +0100, Fabio M. Di Nitto wrote:
> > On Mon, 2008-11-10 at 17:46 -0700, Steven Dake wrote:
> > > I disagree with a global debug keyword.
> > > At one time I thought it was a
> > > good idea but that time has long since passed. The idea of turning
> > > debug to on and then having all debug output go to syslog is frightening
> > > and will result in lost messages. While it appears this proposal
> > > includes the selectable log output filtering per output medium as was
> > > discussed already, it is unclear how the debug keyword affects this. It
> > > would simply make sense to change the file's log priority or the
> > > syslog's log priority if that is the behavior desired and then no need
> > > for any extra keyword.
> >
> > You have these two situations:
> >
> > print_log(LOG_DEBUG, "doing this and that....\n");
> >
> > if (debug) { /*
> > gather_some_data_that_is_very_expensive_operation_to_do_all_the_time();
> > print_log(LOG_DEBUG, "print those extra data\n");
> > }
> >
> > as it is now, it would basically be an alias to set logpriority to DEBUG
> > but enables people to execute debugging code conditionally and as I
> > wrote it is an easy keyword to remember compared to
> > syslog_priority/logpriority.
> >
> > Fabio
> >
>
> The second situation doesn't exist in any code I have written and never
> would.
Clearly you haven't read what I wrote in the debugging note.
> Turning debug on for all of the entire stack to be output to syslog is
> not satisfactory because messages would be lost in overload conditions.
> Logging to file is only a slight bit better solution but if you really
> must have debug output in a persistent store that doesn't occur as a
> result of a failure, logging to file is the only suitable answer.
Please point me to where I wrote that it should go to syslog as I only
mentioned logfile_priority so far.
Fabio
--
Linux-cluster mailing list
Linux-cluster@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/linux-cluster
