On Fri, 2009-02-20 at 10:49 -0600, David Teigland wrote: > On Fri, Feb 20, 2009 at 10:00:03AM -0600, David Teigland wrote: > > I suggest the following, notice the final "corosync" entry, > > > > <logging to_syslog="yes" to_logfile="yes" syslog_facility="daemon" > > syslog_priority="info" logfile_priority="info"> > > <logging_daemon="qdiskd" > > logfile="/var/log/cluster/qdisk.log"/> > > <logging_daemon="groupd" > > logfile="/var/log/cluster/groupd.log"/> > > <logging_daemon="fenced" > > logfile="/var/log/cluster/fenced.log"/> > > <logging_daemon="dlm_controld" > > logfile="/var/log/cluster/dlm_controld.log"/> > > <logging_daemon="gfs_controld" > > logfile="/var/log/cluster/gfs_controld.log"/> > > <logging_daemon="rgmanager" > > logfile="/var/log/cluster/rgmanager.log"/> > > <logging_daemon="corosync" > > logfile="/var/log/cluster/corosync.log"/> > > </logging> > > > > the corosync entry would apply to *all* corosync subsystems by default. > > We can still allow per-subsystem configuration, > > > > <logging_daemon="corosync" subsys="QUORUM" > > logfile="/var/log/cluster/corosync-quorum.log"/> > > <logging_daemon="corosync" subsys="TOTEM" > > logfile="/var/log/cluster/corosync-totem.log"/> > > ... > > Correcting the XML, > > <logging_daemon name="qdiskd" > logfile="/var/log/cluster/qdisk.log"/> > ... > > <logging_daemon name="corosync" subsys="QUORUM" > logfile="/var/log/cluster/corosync-quorum.log"/>
Since I haven't finished to convert logsys to our standard, I'd happy to do this change. It sounds sane to me. Of course this is only when we explicitly add info for the subsystem. Otherwise we go default like we already implemented. Fabio
