misc/dashboard: don't email about netbsd failures

Too flaky.

R=iant, rsc
CC=golang-dev
https://golang.org/cl/6925056

diff --git a/misc/dashboard/app/build/notify.go b/misc/dashboard/app/build/notify.go
index afcc7b2db54c1631a5727ef0532d1d14558e8a64..0eadb909f36009c80f899af31eb0709ed2acf31a 100644 (file)
--- 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