From 435aeef7d61a4cff6e6d918216b9124a6d44e061 Mon Sep 17 00:00:00 2001
From: Aleksandr Nogikh <nogikh@google.com>
Date: Wed, 7 Sep 2022 10:43:53 +0000
Subject: dashboard: don't retest repros for bugs with commits

This just wastes resources -- those bugs are marked as fixed on our
dashboard anyway.
---
 dashboard/app/jobs.go | 5 +++++
 1 file changed, 5 insertions(+)

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
-- 
cgit mrf-deployment