diff options
author | Brad Fitzpatrick <bradfitz@golang.org> | 2012-12-12 11:33:03 -0800 |
---|---|---|
committer | Brad Fitzpatrick <bradfitz@golang.org> | 2012-12-12 11:33:03 -0800 |
commit | 93cae76df54096bbfeb39ea9a57dd24ed7076f9f (patch) | |
tree | 4856c3cb03044d7af061ed480d6d2e9b765d2593 | |
parent | 6aa84f8b61234804a4e554a08973b3de87171f95 (diff) | |
download | go-93cae76df54096bbfeb39ea9a57dd24ed7076f9f.tar.gz |
misc/dashboard: don't email about netbsd failures
Too flaky.
R=iant, rsc
CC=golang-dev
https://codereview.appspot.com/6925056
-rw-r--r-- | misc/dashboard/app/build/notify.go | 11 |
1 files changed, 11 insertions, 0 deletions
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 |