From 46d78f4deb682ac280e65fbc8d7d37f4f23d083e Mon Sep 17 00:00:00 2001 From: Dmitry Vyukov Date: Mon, 15 Jan 2018 15:26:30 +0100 Subject: dashboard/app: don't warn about emails on closed bugs --- dashboard/app/reporting.go | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) (limited to 'dashboard') diff --git a/dashboard/app/reporting.go b/dashboard/app/reporting.go index 5c694cdc6..675138c3e 100644 --- a/dashboard/app/reporting.go +++ b/dashboard/app/reporting.go @@ -396,7 +396,12 @@ func incomingCommandTx(c context.Context, now time.Time, cmd *dashapi.BugUpdate, // but this is confusing and non-actionable for users. // So now we fail the update, but give empty reason, // which means "don't notify user". - log.Warningf(c, "Dup bug is already closed") + if cmd.Status == dashapi.BugStatusUpdate { + // This happens when people discuss old bugs. + log.Infof(c, "Dup bug is already closed") + } else { + log.Warningf(c, "Dup bug is already closed") + } return false, "", nil } case BugStatusFixed, BugStatusInvalid: -- cgit mrf-deployment