aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAleksandr Nogikh <nogikh@google.com>2022-09-07 10:43:53 +0000
committerAleksandr Nogikh <wp32pw@gmail.com>2022-09-07 14:01:31 +0200
commit435aeef7d61a4cff6e6d918216b9124a6d44e061 (patch)
treea5f337a2f8d14f67610afdfb3650f8c5231568cb
parentc5b7bc5781ca8357a6a980b487235b67c108e24f (diff)
dashboard: don't retest repros for bugs with commits
This just wastes resources -- those bugs are marked as fixed on our dashboard anyway.
-rw-r--r--dashboard/app/jobs.go5
1 files changed, 5 insertions, 0 deletions
diff --git a/dashboard/app/jobs.go b/dashboard/app/jobs.go
index 51bbbd99f..4cff3493a 100644
--- a/dashboard/app/jobs.go
+++ b/dashboard/app/jobs.go
@@ -288,6 +288,11 @@ func updateRetestReproJobs(c context.Context, ns string) error {
func handleRetestForBug(c context.Context, now time.Time, bug *Bug, bugKey *db.Key,
managerHasJob map[string]bool) error {
+ if len(bug.Commits) > 0 {
+ // Let's save resources -- there's no point in retesting repros for bugs
+ // for which we were already given fixing commits.
+ return nil
+ }
crashes, crashKeys, err := queryCrashesForBug(c, bugKey, maxCrashes)
if err != nil {
return err