Commit a257c753 authored by Brad Fitzpatrick's avatar Brad Fitzpatrick

misc/dashboard: don't email about netbsd failures

Too flaky.

R=iant, rsc
CC=golang-dev
https://golang.org/cl/6925056
parent 08ce7f1d
...@@ -21,6 +21,13 @@ const ( ...@@ -21,6 +21,13 @@ const (
domain = "build.golang.org" 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 // notifyOnFailure checks whether the supplied Commit or the subsequent
// Commit (if present) breaks the build for this builder. // Commit (if present) breaks the build for this builder.
// If either of those commits break the build an email notification is sent // If either of those commits break the build an email notification is sent
...@@ -30,6 +37,10 @@ const ( ...@@ -30,6 +37,10 @@ const (
// This must be run in a datastore transaction, and the provided *Commit must // This must be run in a datastore transaction, and the provided *Commit must
// have been retrieved from the datastore within that transaction. // have been retrieved from the datastore within that transaction.
func notifyOnFailure(c appengine.Context, com *Commit, builder string) error { func notifyOnFailure(c appengine.Context, com *Commit, builder string) error {
if failIgnore[builder] {
return
}
// TODO(adg): implement notifications for packages // TODO(adg): implement notifications for packages
if com.PackagePath != "" { if com.PackagePath != "" {
return nil return nil
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment