From: Steven Dake <sdake@redhat.com>
To: cluster-devel.redhat.com
Subject: [Cluster-devel] logging: final call on configuration, output and implementation
Date: Mon, 10 Nov 2008 23:00:58 -0700 [thread overview]
Message-ID: <1226383258.14398.31.camel@balance> (raw)
In-Reply-To: <1226382840.2445.78.camel@daitarn-fedora.int.fabbione.net>
On Tue, 2008-11-11 at 06:54 +0100, Fabio M. Di Nitto wrote:
> 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.
>
I read it but don't agree you can have a discussion about logging and
flight recording without discussing how debugging fits into the log
system.
> > 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.
>
If syslog is configured it will go to syslog by default in your scheme.
Regards
-steve
> Fabio
>
next prev parent reply other threads:[~2008-11-11 6:00 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-11-10 19:09 [Cluster-devel] logging: final call on configuration, output and implementation Fabio M. Di Nitto
2008-11-10 19:27 ` David Teigland
2008-11-10 19:48 ` Fabio M. Di Nitto
2008-11-10 20:49 ` [Linux-cluster] " Joel Becker
2008-11-11 0:46 ` Steven Dake
2008-11-11 4:55 ` Fabio M. Di Nitto
2008-11-11 5:47 ` Steven Dake
2008-11-11 5:54 ` Fabio M. Di Nitto
2008-11-11 6:00 ` Steven Dake [this message]
2008-11-11 18:11 ` Fabio M. Di Nitto
2008-11-13 5:51 ` [Cluster-devel] Re: logging: more input Fabio M. Di Nitto
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1226383258.14398.31.camel@balance \
--to=sdake@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).