From 71d01446358836b150768026f969e79d01493830 Mon Sep 17 00:00:00 2001 From: Brad Fitzpatrick Date: Sat, 20 Dec 2014 17:07:06 +1100 Subject: [PATCH] dashboard/retrybuilds: add more flaky phrases Change-Id: Id1a1b870a348a24040d94b157328b1632776711f Reviewed-on: https://go-review.googlesource.com/1877 Reviewed-by: Dave Cheney --- dashboard/retrybuilds/retrybuilds.go | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/dashboard/retrybuilds/retrybuilds.go b/dashboard/retrybuilds/retrybuilds.go index a2d97df0..c432df27 100644 --- a/dashboard/retrybuilds/retrybuilds.go +++ b/dashboard/retrybuilds/retrybuilds.go @@ -106,6 +106,7 @@ var flakePhrases = []string{ "Bseek: unknown state 0", "error exporting repository: exit status", "remote error: User Is Over Quota", + "fatal: remote did not send all necessary objects", } func isFlaky(failLog string) bool { @@ -121,6 +122,12 @@ func isFlaky(failLog string) bool { if numLines < 20 && strings.Contains(failLog, "error: exit status") { return true } + // e.g. fatal: destination path 'go.tools.TMP' already exists and is not an empty directory. + // To be fixed in golang.org/issue/9407 + if strings.Contains(failLog, "fatal: destination path '") && + strings.Contains(failLog, "' already exists and is not an empty directory.") { + return true + } return false }