From 8aae1381312cc7ae8fb1c1f7d2561e3d551f74ce Mon Sep 17 00:00:00 2001 From: Andrew Gerrand Date: Tue, 28 Jan 2014 17:59:29 +1100 Subject: [PATCH] go.tools/dashboard/app: update bad builder list LGTM=r R=r, minux.ma CC=golang-codereviews https://golang.org/cl/57580043 --- dashboard/app/build/notify.go | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/dashboard/app/build/notify.go b/dashboard/app/build/notify.go index 63d19ca190..424119ee96 100644 --- a/dashboard/app/build/notify.go +++ b/dashboard/app/build/notify.go @@ -30,11 +30,13 @@ const ( gobotBase = "http://research.swtch.com/gobot_codereview" ) -// 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, +// ignoreFailure is a set of builders that we don't email about because +// they are not yet production-ready. +var ignoreFailure = map[string]bool{ + "dragonfly-amd64": true, "netbsd-amd64-bsiegert": true, + "plan-386-cnielsen": true, + "solaris-amd64-smartos": true, } // notifyOnFailure checks whether the supplied Commit or the subsequent @@ -46,7 +48,7 @@ var failIgnore = map[string]bool{ // 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] { + if ignoreFailure[builder] { return nil }