roachtest: fix live migration error message

DarrylWong · DarrylWong · commit c497157a60e6 · 2025-09-09T11:41:46.000-04:00
Previously the error reporting would list host error vms,
instead of live migration vms. This change fixes that.

Epic: none
Fixes: none
Release note: none
diff --git a/pkg/cmd/roachtest/test_runner.go b/pkg/cmd/roachtest/test_runner.go
@@ -1261,7 +1261,7 @@ func (r *testRunner) runTest(
 				if liveMigrationVMNames != "" {
 					failureMsg = fmt.Sprintf("VMs had live migrations during the test run: %s\n\n**Other Failures:**\n%s", liveMigrationVMNames, failureMsg)
 					t.resetFailures()
-					t.Error(liveMigrationError(hostErrorVMNames))
+					t.Error(liveMigrationError(liveMigrationVMNames))
 				}
 
 				output := fmt.Sprintf("%s\ntest artifacts and logs in: %s", failureMsg, t.ArtifactsDir())

Original file line number	Diff line number	Diff line change
`@@ -1261,7 +1261,7 @@ func (r *testRunner) runTest(`
`1261`	`1261`	`if liveMigrationVMNames != "" {`
`1262`	`1262`	`failureMsg = fmt.Sprintf("VMs had live migrations during the test run: %s\n\nOther Failures:\n%s", liveMigrationVMNames, failureMsg)`
`1263`	`1263`	`t.resetFailures()`
`1264`		`- t.Error(liveMigrationError(hostErrorVMNames))`
	`1264`	`+ t.Error(liveMigrationError(liveMigrationVMNames))`
`1265`	`1265`	`}`
`1266`	`1266`
`1267`	`1267`	`output := fmt.Sprintf("%s\ntest artifacts and logs in: %s", failureMsg, t.ArtifactsDir())`