acid-ant/frostfs-node
1
0
Fork 0
forked from TrueCloudLab/frostfs-node
Code Issues Pull requests Projects Releases Packages Wiki Activity

[#638] logger: Remove sampling from test loggers

Browse source 
Losing logs is always a bad idea, especially when we debug tests.

Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>
This commit is contained in:
Evgenii Stratonikov 2023-08-23 10:48:23 +03:00 committed by Evgenii Stratonikov
parent 82cc453be9
commit 02b03d9c4f
1 changed files with 0 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
pkg/util/logger/test/logger.go
View file

@ -9,8 +9,6 @@ import (
"go.uber.org/zap/zapcore" "go.uber.org/zap/zapcore"
) )
const sampling = 1000
// NewLogger creates a new logger. // NewLogger creates a new logger.
// //
// If debug, development logger is created. // If debug, development logger is created.
@ -20,11 +18,6 @@ func NewLogger(t testing.TB, debug bool) *logger.Logger {
if debug { if debug {
cfg := zap.NewDevelopmentConfig() cfg := zap.NewDevelopmentConfig()
cfg.Sampling = &zap.SamplingConfig{
Initial: sampling,
Thereafter: sampling,
}
cfg.EncoderConfig.EncodeLevel = zapcore.CapitalColorLevelEncoder cfg.EncoderConfig.EncodeLevel = zapcore.CapitalColorLevelEncoder
log, err := cfg.Build() log, err := cfg.Build()