From 530e80f8ee7c0e39b3e98ed0839a557af2d6192e Mon Sep 17 00:00:00 2001
From: Aleksandr Nogikh <nogikh@google.com>
Date: Thu, 12 Dec 2024 13:16:11 +0100
Subject: syz-ci: rename bench file on all syz-manager restarts

ManagerCmd transparently restarts the instance in case of crashes, so we
should better be cleaning up the bench file within its loop, rather than
in manager.go.
---
 syz-ci/managercmd.go | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

(limited to 'syz-ci/managercmd.go')

diff --git a/syz-ci/managercmd.go b/syz-ci/managercmd.go
index cc91c3020..8156282b5 100644
--- a/syz-ci/managercmd.go
+++ b/syz-ci/managercmd.go
@@ -19,6 +19,7 @@ import (
 type ManagerCmd struct {
 	name    string
 	log     string
+	bench   string
 	errorf  Errorf
 	bin     string
 	args    []string
@@ -31,10 +32,11 @@ type Errorf func(msg string, args ...interface{})
 // name - name for logging.
 // log - manager log file with stdout/stderr.
 // bin/args - process binary/args.
-func NewManagerCmd(name, log string, errorf Errorf, bin string, args ...string) *ManagerCmd {
+func NewManagerCmd(name, log, bench string, errorf Errorf, bin string, args ...string) *ManagerCmd {
 	mc := &ManagerCmd{
 		name:    name,
 		log:     log,
+		bench:   bench,
 		errorf:  errorf,
 		bin:     bin,
 		args:    args,
@@ -75,6 +77,7 @@ func (mc *ManagerCmd) loop() {
 			if time.Since(started) > restartPeriod {
 				started = time.Now()
 				osutil.Rename(mc.log, mc.log+".old")
+				osutil.Rename(mc.bench, mc.bench+".old") // or else syz-manager will complain
 				logfile, err := os.Create(mc.log)
 				if err != nil {
 					mc.errorf("failed to create manager log: %v", err)
-- 
cgit mrf-deployment