Re: [PATCH multipath-tools-0.4.7.rhel5.13 1/1] multipathd excessive path checker logging

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On Fri, 26 Jun 2009, Konrad Rzeszutek wrote:

On Fri, Jun 26, 2009 at 03:50:57PM -0500, Benjamin Marzinski wrote:
On Thu, Jun 25, 2009 at 01:50:51PM -0400, Charlie Brady wrote:

multipathd logs a path checker message whenever state changes. But it also
logs every non-changed message when verbosity is >=4 and newstate is
PATH_UP or PATH_GHOST, and every non-changed message when verbosity is >= 2
and newstate is PATH_DOWN.

I believe that the message should only be logged, once, when state changes.

I disagree.  If you look at the stuff that gets logged when verbosity >=
4, you'll see that it's a whole bunch of stuff that normal users would
never care about, but may be useful for debugging.  This clearly fits.

I also think that having a path down is something that shouldn't just
blip by once in the logs.  It's something that often requires the
sysadmin to intervene. I could see limitting it to, say, only happen
on every tenth pass through the checker loop, so that these messages
don't clog up the logs so bad. But since there is the option to set the
verbosity to 1 to avoid this, I think that printing the path down
messages repeatedly is useful.

Hmm.. What if you have 100 LUNs with four paths, meaning you get 400
messages saying: "Down! Down! Pants on fire!"

Or, in real life, three LUNS with 16paths each:

http://groups.google.am/group/open-iscsi/browse_thread/thread/95865e5da58a28f2

FWIW, I don't think there is any practical value in posting repeated "path down" messages to syslog, and it's wrong to do that at default verbosity level.

--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel

[Index of Archives]     [DM Crypt]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Packaging]     [Fedora SELinux]     [Yosemite Discussion]     [KDE Users]     [Fedora Docs]

  Powered by Linux