Fix bug with issues [17464] fix with zap logger redirection

Signed-off-by: Colin <jay_jieliu@outlook.com>

client/pkg/logutil/zap.go

@@ -15,6 +15,7 @@
 package logutil
 
 import (
+	"os"
 	"sort"
 	"time"
 
@@ -33,6 +34,32 @@ func CreateDefaultZapLogger(level zapcore.Level) (*zap.Logger, error) {
 	return c, nil
 }
 
+// CreateDefaultZapLogger creates a logger with default zap configuration can redirect log to /dev/null
+func CreateUtilZapLogger(level zapcore.Level) *zap.Logger {
+	infoLevel := zap.LevelEnablerFunc(func(level zapcore.Level) bool {
+		return level == zapcore.InfoLevel
+	})
+	errorFatalLevel := zap.LevelEnablerFunc(func(level zapcore.Level) bool {
+		return level == zapcore.ErrorLevel || level == zapcore.FatalLevel
+	})
+	stdoutSyncer := zapcore.Lock(os.Stdout)
+	stderrSyncer := zapcore.Lock(os.Stderr)
+	core := zapcore.NewTee(
+		zapcore.NewCore(
+			zapcore.NewJSONEncoder(DefaultZapLoggerConfig.EncoderConfig),
+			stdoutSyncer,
+			infoLevel,
+		),
+		zapcore.NewCore(
+			zapcore.NewJSONEncoder(DefaultZapLoggerConfig.EncoderConfig),
+			stderrSyncer,
+			errorFatalLevel,
+		),
+	)
+	logger := zap.New(core)
+	return logger
+}
+
 // DefaultZapLoggerConfig defines default zap logger configuration.
 var DefaultZapLoggerConfig = zap.Config{
 	Level: zap.NewAtomicLevelAt(ConvertToZapLevel(DefaultLogLevel)),

etcdctl/ctlv3/command/snapshot_command.go

@@ -50,10 +50,7 @@ func snapshotSaveCommandFunc(cmd *cobra.Command, args []string) {
 		cobrautl.ExitWithError(cobrautl.ExitBadArgs, err)
 	}
 
-	lg, err := logutil.CreateDefaultZapLogger(zap.InfoLevel)
-	if err != nil {
-		cobrautl.ExitWithError(cobrautl.ExitError, err)
-	}
+	lg := logutil.CreateUtilZapLogger(zap.InfoLevel)
 	cfg := mustClientCfgFromCmd(cmd)
 
 	// if user does not specify "--command-timeout" flag, there will be no timeout for snapshot save command