Merge pull request #73277 from smarterclayton/ignore_goaway

Add the http2 GOAWAY error to IsProbableEOF for ignoring in watch
This commit is contained in:
Kubernetes Prow Robot 2019-01-24 14:55:22 -08:00 committed by GitHub
commit 4a44df3fef
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -68,14 +68,17 @@ func IsProbableEOF(err error) bool {
if uerr, ok := err.(*url.Error); ok { if uerr, ok := err.(*url.Error); ok {
err = uerr.Err err = uerr.Err
} }
msg := err.Error()
switch { switch {
case err == io.EOF: case err == io.EOF:
return true return true
case err.Error() == "http: can't write HTTP request on broken connection": case msg == "http: can't write HTTP request on broken connection":
return true return true
case strings.Contains(err.Error(), "connection reset by peer"): case strings.Contains(msg, "http2: server sent GOAWAY and closed the connection"):
return true return true
case strings.Contains(strings.ToLower(err.Error()), "use of closed network connection"): case strings.Contains(msg, "connection reset by peer"):
return true
case strings.Contains(strings.ToLower(msg), "use of closed network connection"):
return true return true
} }
return false return false