Merge pull request #158649 from msbutler/blathers/backport-release-25.2-158637

msbutler · web-flow · commit df7c75ac79bd · 2025-12-03T08:17:23.000-05:00
release-25.2: roachtest: send more schema change flakes to foundations
diff --git a/pkg/cmd/roachtest/tests/backup_restore_roundtrip.go b/pkg/cmd/roachtest/tests/backup_restore_roundtrip.go
@@ -50,7 +50,7 @@ func handleSchemaChangeWorkloadError(err error) error {
 	// crash.
 	if err != nil {
 		flattenedErr := errors.FlattenDetails(err)
-		if strings.Contains(flattenedErr, "UNEXPECTED ERROR") || strings.Contains(flattenedErr, "UNEXPECTED COMMIT ERROR") {
+		if strings.Contains(flattenedErr, "workload run error: ***") || strings.Contains(flattenedErr, "UNEXPECTED ERROR") || strings.Contains(flattenedErr, "UNEXPECTED COMMIT ERROR") {
 			return registry.ErrorWithOwner(registry.OwnerSQLFoundations, errors.Wrapf(err, "schema change workload failed"))
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,7 @@ func handleSchemaChangeWorkloadError(err error) error {`
`50`	`50`	`// crash.`
`51`	`51`	`if err != nil {`
`52`	`52`	`flattenedErr := errors.FlattenDetails(err)`
`53`		`- if strings.Contains(flattenedErr, "UNEXPECTED ERROR") \|\| strings.Contains(flattenedErr, "UNEXPECTED COMMIT ERROR") {`
	`53`	`+ if strings.Contains(flattenedErr, "workload run error: ***") \|\| strings.Contains(flattenedErr, "UNEXPECTED ERROR") \|\| strings.Contains(flattenedErr, "UNEXPECTED COMMIT ERROR") {`
`54`	`54`	`return registry.ErrorWithOwner(registry.OwnerSQLFoundations, errors.Wrapf(err, "schema change workload failed"))`
`55`	`55`	`}`
`56`	`56`	`}`