Configure logfile path via 'logConfig.path' property

Signed-off-by: YanzheL <[email protected]>

Author: YanzheL

cmd/dfdaemon/app/init.go

@@ -80,15 +80,13 @@ func initLogger(cfg config.Properties) error {
 		cfg.WorkHome = filepath.Join(current.HomeDir, ".small-dragonfly")
 	}
 
-	logFilePath := filepath.Join(cfg.WorkHome, "logs", "dfdaemon.log")
-
 	opts := []dflog.Option{
-		dflog.WithLogFile(logFilePath, cfg.LogConfig.MaxSize, cfg.LogConfig.MaxBackups),
+		dflog.WithLogFile(cfg.LogConfig.Path, cfg.LogConfig.MaxSize, cfg.LogConfig.MaxBackups),
 		dflog.WithSign(fmt.Sprintf("%d", os.Getpid())),
 		dflog.WithDebug(cfg.Verbose),
 	}
 
-	logrus.Debugf("use log file %s", logFilePath)
+	logrus.Debugf("use log file %s", cfg.LogConfig.Path)
 
 	return errors.Wrap(dflog.Init(logrus.StandardLogger(), opts...), "init log")
 }

pkg/dflog/log.go

@@ -20,6 +20,7 @@ import (
 	"bytes"
 	"fmt"
 	"os"
+	"path/filepath"
 	"strings"
 
 	"github.com/pkg/errors"
@@ -35,6 +36,10 @@ type LogConfig struct {
 	// MaxBackups is the maximum number of old log files to retain.
 	// The default value is 1.
 	MaxBackups int `yaml:"maxBackups" json:"maxBackups"`
+
+	// Path is the location of log file
+	// The default value is logs/dfdaemon.log
+	Path string `yaml:"path" json:"path"`
 }
 
 // DefaultLogTimeFormat defines the timestamp format.
@@ -72,7 +77,7 @@ func getLumberjack(l *logrus.Logger) *lumberjack.Logger {
 func WithLogFile(f string, maxSize, maxBackups int) Option {
 	return func(l *logrus.Logger) error {
 		if f == "" {
-			return nil
+			f = filepath.Join("logs", "dfdaemon.log")
 		}
 		if maxSize <= 0 {
 			maxSize = 40