From 93cae76df54096bbfeb39ea9a57dd24ed7076f9f Mon Sep 17 00:00:00 2001 From: Brad Fitzpatrick Date: Wed, 12 Dec 2012 11:33:03 -0800 Subject: misc/dashboard: don't email about netbsd failures Too flaky. R=iant, rsc CC=golang-dev https://codereview.appspot.com/6925056 --- misc/dashboard/app/build/notify.go | 11 +++++++++++ 1 file changed, 11 insertions(+) (limited to 'misc/dashboard') diff --git a/misc/dashboard/app/build/notify.go b/misc/dashboard/app/build/notify.go index afcc7b2db..0eadb909f 100644 --- a/misc/dashboard/app/build/notify.go +++ b/misc/dashboard/app/build/notify.go @@ -21,6 +21,13 @@ const ( domain = "build.golang.org" ) +// failIgnore is a set of builders that we don't email about because +// they're too flaky. +var failIgnore = map[string]bool{ + "netbsd-386-bsiegert": true, + "netbsd-amd64-bsiegert": true, +} + // notifyOnFailure checks whether the supplied Commit or the subsequent // Commit (if present) breaks the build for this builder. // If either of those commits break the build an email notification is sent @@ -30,6 +37,10 @@ const ( // This must be run in a datastore transaction, and the provided *Commit must // have been retrieved from the datastore within that transaction. func notifyOnFailure(c appengine.Context, com *Commit, builder string) error { + if failIgnore[builder] { + return + } + // TODO(adg): implement notifications for packages if com.PackagePath != "" { return nil -- cgit v1.2.1