From 0ffbfc130e3811c0ba997ed3386958dc214c3395 Mon Sep 17 00:00:00 2001
From: Roman Khimov <roman@nspcc.ru>
Date: Wed, 13 Nov 2024 12:58:47 +0300
Subject: [PATCH] ir/node: disable log sampling

The default Zap config is:

> Sampling is enabled at 100:100 by default, meaning that after the first 100 log entries with the same level and message in the same second, it will log every 100th entry with the same level and message in the same second. You may disable this behavior by setting Sampling to nil.

It leads to message loss too easily and we care a lot about each and every
object. Notice that it was disabled in NeoGo as well
in a52e016c22de2269fcc7362f33317cebfa7e8e4c (https://github.com/nspcc-dev/neo-go/pull/598).

Signed-off-by: Roman Khimov <roman@nspcc.ru>
---
 CHANGELOG.md             | 1 +
 cmd/neofs-ir/main.go     | 1 +
 cmd/neofs-node/config.go | 1 +
 3 files changed, 3 insertions(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 6818f3b3c5..771f8cdbe5 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -40,6 +40,7 @@ attribute, which is used for container domain name in NNS contracts (#2954)
 - Metabase is refilled if an object exists in write-cache but is missing in metabase (#2977)
 - Pprof and metrics services stop at the end of SN's application lifecycle (#2976)
 - Reject configuration with unknown fields (#2981)
+- Log sampling is disabled by default now (#3011)
 
 ### Removed
 - Support for node.key configuration (#2959)
diff --git a/cmd/neofs-ir/main.go b/cmd/neofs-ir/main.go
index 860ffa31bc..bc2ea713f2 100644
--- a/cmd/neofs-ir/main.go
+++ b/cmd/neofs-ir/main.go
@@ -66,6 +66,7 @@ func main() {
 	c := zap.NewProductionConfig()
 	c.Level = logLevel
 	c.Encoding = cfg.GetString("logger.encoding")
+	c.Sampling = nil
 	if term.IsTerminal(int(os.Stdout.Fd())) {
 		c.EncoderConfig.EncodeTime = zapcore.ISO8601TimeEncoder
 	} else {
diff --git a/cmd/neofs-node/config.go b/cmd/neofs-node/config.go
index 958d40947c..202be30da5 100644
--- a/cmd/neofs-node/config.go
+++ b/cmd/neofs-node/config.go
@@ -570,6 +570,7 @@ func initCfg(appCfg *config.Config) *cfg {
 	logCfg := zap.NewProductionConfig()
 	logCfg.Level = c.internals.logLevel
 	logCfg.Encoding = c.logger.encoding
+	logCfg.Sampling = nil
 	if term.IsTerminal(int(os.Stdout.Fd())) {
 		logCfg.EncoderConfig.EncodeTime = zapcore.ISO8601TimeEncoder
 	} else {