feat: parameter to write per-run logs to directory

cmd/pew.go

@@ -12,6 +12,7 @@ import (
 	"math/rand"
 	"os"
 	"os/exec"
+	"path/filepath"
 	"strings"
 	"time"
 
@@ -90,6 +91,8 @@ type PewCmd struct {
 	OutputOnSuccess bool `help:"show output on success"`
 	RealtimeOutput  bool `help:"show realtime output"`
 
+	LogDir string `help:"write logs to directory"`
+
 	Endless        bool          `help:"endless tests"`
 	EndlessTimeout time.Duration `help:"timeout between tests" default:"1m"`
 	EndlessStress  string        `help:"endless stress script" type:"existingfile"`
@@ -403,39 +406,54 @@ func (cmd PewCmd) testArtifact(swg *sizedwaitgroup.SizedWaitGroup,
 
 	defer swg.Done()
 
-	logdir := "logs/" + cmd.Tag
+	var logDirWriter *zerolog.ConsoleWriter
-	err := os.MkdirAll(logdir, os.ModePerm)
+	if cmd.LogDir != "" {
-	if err != nil {
+		logdir := filepath.Join(cmd.LogDir, cmd.Tag)
-		log.Error().Err(err).Msgf("mkdir %s", logdir)
+		err := os.MkdirAll(logdir, os.ModePerm)
-		return
+		if err != nil {
-	}
+			log.Error().Err(err).Msgf("mkdir %s", logdir)
+			return
+		}
 
-	logfile := fmt.Sprintf("logs/%s/%s-%s-%s.log",
+		logfile := fmt.Sprintf("logs/%s/%s-%s-%s.log",
-		cmd.Tag,
+			cmd.Tag,
-		ki.Distro.ID.String(),
+			ki.Distro.ID.String(),
-		ki.Distro.Release,
+			ki.Distro.Release,
-		ki.KernelRelease,
+			ki.KernelRelease,
-	)
+		)
-	f, err := os.Create(logfile)
+		f, err := os.Create(logfile)
-	if err != nil {
+		if err != nil {
-		log.Error().Err(err).Msgf("create %s", logfile)
+			log.Error().Err(err).Msgf("create %s", logfile)
-		return
+			return
-	}
+		}
-	defer f.Close()
+		defer f.Close()
 
-	slog := zerolog.New(zerolog.MultiLevelWriter(
+		logDirWriter = &zerolog.ConsoleWriter{
-		&ConsoleWriter,
-		&FileWriter,
-		&zerolog.ConsoleWriter{
 			Out: f,
 			FieldsExclude: []string{
 				"distro_release",
 				"distro_type",
 				"kernel",
+				"command",
+				"workdir",
 			},
 			NoColor: true,
-		},
+		}
-	))
+	}
+
+	var slog zerolog.Logger
+	if logDirWriter != nil {
+		slog = zerolog.New(zerolog.MultiLevelWriter(
+			&ConsoleWriter,
+			&FileWriter,
+			logDirWriter,
+		))
+	} else {
+		slog = zerolog.New(zerolog.MultiLevelWriter(
+			&ConsoleWriter,
+			&FileWriter,
+		))
+	}
 
 	switch LogLevel {
 	case zerolog.TraceLevel, zerolog.DebugLevel:

container/container.go

@@ -466,7 +466,7 @@ func (c Container) build(imagePath string) (output string, err error) {
 	return
 }
 
-func (c Container) Run(workdir string, cmds []string) (out string, err error) {
+func (c *Container) Run(workdir string, cmds []string) (out string, err error) {
 	flog := c.Log.With().
 		Str("workdir", workdir).
 		Str("command", fmt.Sprintf("%v", cmds)).
@@ -535,7 +535,7 @@ func (c Container) Run(workdir string, cmds []string) (out string, err error) {
 			}
 			c.handleCommandsOutput(m)
 			out += m + "\n"
-			flog.Trace().Str("stdout", m).Msg("")
+			flog.Trace().Str("container stdout", m).Msg("")
 		}
 	}()