metropolis/test/launch: fix data race
Change-Id: Id427d73cf32ff06a074fce5903d66050dd9e28cb
Reviewed-on: https://review.monogon.dev/c/monogon/+/3688
Reviewed-by: Lorenz Brun <lorenz@monogon.tech>
Tested-by: Jenkins CI
diff --git a/metropolis/test/launch/cluster.go b/metropolis/test/launch/cluster.go
index b46cc68..bb9f2f7 100644
--- a/metropolis/test/launch/cluster.go
+++ b/metropolis/test/launch/cluster.go
@@ -875,13 +875,14 @@
go func() {
var serialPort io.ReadWriter
+ var err error
if opts.NodeLogsToFiles {
- path := path.Join(ld, "nanoswitch.txt")
- serialPort, err = NewSerialFileLogger(path)
+ loggerPath := path.Join(ld, "nanoswitch.txt")
+ serialPort, err = NewSerialFileLogger(loggerPath)
if err != nil {
- launch.Log("Could not open log file for nanoswitch: %v", err)
+ launch.Fatal("Could not open log file for nanoswitch: %v", err)
}
- launch.Log("Nanoswitch logs at %s", path)
+ launch.Log("Nanoswitch logs at %s", loggerPath)
} else {
serialPort = newPrefixedStdio(99)
}