Skip to content

Commit

Permalink
Merge #123983
Browse files Browse the repository at this point in the history
123983: githubpost: assign backupccl test failures to disaster-recovery r=herkolategan a=renatolabs

Temporary workaround for `backupccl` failures, by far the most common occurrence of #107885.

Informs: #107885

Release note: None

Co-authored-by: Renato Costa <[email protected]>
  • Loading branch information
craig[bot] and renatolabs committed May 13, 2024
2 parents c8e257e + feab208 commit 2f739e8
Showing 1 changed file with 14 additions and 2 deletions.
16 changes: 14 additions & 2 deletions pkg/cmd/bazci/githubpost/githubpost.go
Original file line number Diff line number Diff line change
Expand Up @@ -690,8 +690,20 @@ func getOwner(ctx context.Context, packageName, testName string) (_teams []team.
if testEng.Name() == "" {
log.Fatalf("test-eng team could not be found in TEAMS.yaml")
}
log.Printf("assigning %s.%s to 'test-eng' as catch-all", packageName, testName)
match = []team.Team{testEng}

// Workaround for #107885.
if strings.Contains(packageName, "backupccl") {
dr := co.GetTeamForAlias("cockroachdb/disaster-recovery")
if dr.Name() == "" {
log.Fatalf("disaster-recovery team could not be found in TEAMS.yaml")
}

log.Printf("assigning %s.%s to 'disaster-recovery' due to #107885", packageName, testName)
match = []team.Team{dr}
} else {
log.Printf("assigning %s.%s to 'test-eng' as catch-all", packageName, testName)
match = []team.Team{testEng}
}
}
return match
}
Expand Down

0 comments on commit 2f739e8

Please sign in to comment.