From 83f945db79eb902458b40875fa8a10b0dbb49172 Mon Sep 17 00:00:00 2001 From: Dmitry Vyukov Date: Fri, 8 Jun 2018 10:00:42 +0200 Subject: dashboard/app: fix old crash purging logic NumRepro includes failed repro attempts, so we should not look at it when purging old crashes. --- dashboard/app/api.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dashboard/app/api.go b/dashboard/app/api.go index f690ca348..1b5f016ca 100644 --- a/dashboard/app/api.go +++ b/dashboard/app/api.go @@ -597,7 +597,7 @@ func reportCrash(c context.Context, ns string, req *dashapi.Crash) (*Bug, error) } func purgeOldCrashes(c context.Context, bug *Bug, bugKey *datastore.Key) { - if bug.NumCrashes-bug.NumRepro <= maxCrashes || (bug.NumCrashes-1)%10 != 0 { + if bug.NumCrashes <= maxCrashes || (bug.NumCrashes-1)%10 != 0 { return } var crashes []*Crash -- cgit mrf-deployment