Open davhdavh opened 1 week ago
Hmm, guessing without looking up documentation, do we have command lines for configuring log sampling in zap?
Hmm, guessing without looking up documentation, do we have command lines for configuring log sampling in zap?
Unless I am missing something there is nothing in server logging config currently for zap sampling rates: https://etcd.io/docs/v3.6/op-guide/configuration/#logging
--logger 'zap'
Currently only supports 'zap' for structured logging.
--log-outputs 'default'
Specify 'stdout' or 'stderr' to skip journald logging even when running under systemd, or list of comma separated output targets.
--log-level 'info'
Configures log level. Only supports debug, info, warn, error, panic, or fatal.
--enable-log-rotation 'false'
Enable log rotation of a single log-outputs file target.
--log-rotation-config-json '{"maxsize": 100, "maxage": 0, "maxbackups": 0, "localtime": false, "compress": false}'
Configures log rotation if enabled with a JSON logger config. MaxSize(MB), MaxAge(days,0=no limit), MaxBackups(0=no limit), LocalTime(use computers local time), Compress(gzip)".
We could consider adding https://pkg.go.dev/go.uber.org/zap#SamplingConfig for rafthttp
?
~ I would be careful about overcomplicating logging, my question came from idea whether it can be solved now. If we have a problem with a single log line, we should look into it first, before we start supporting a new feature.
What would you like to be added?
im fairly certain that giving me the exact same log message hundreds of times per seconds provide no actual benefit.
Why is this needed?
because having to write 10s of megabytes per min of log on every single control-plane while a member is temporarily missing is insane.