Another "known failure" error to allow in tests (#2379)

michaeldwan · web-flow · commit 72fc99da6c7a · 2025-05-30T16:58:11.000Z
diff --git a/pkg/docker/errors.go b/pkg/docker/errors.go
@@ -51,6 +51,7 @@ func isMissingDeviceDriverError(err error) bool {
 		strings.Contains(msg, "nvidia-container-cli: initialization error")
 }
 
+// isNetworkError checks if the error is a network error. This is janky and intended for use in tests only
 func isNetworkError(err error) bool {
 	// for both CLI and API clients, network errors are wrapped and lose the net.Error interface
 	// CLI client: wrapped by exec.Command as exec.ExitError
@@ -65,6 +66,7 @@ func isNetworkError(err error) bool {
 		"EOF",
 		"no route to host",
 		"network is unreachable",
+		"server closed",
 	}
 
 	for _, errStr := range networkErrorStrings {

Original file line number	Diff line number	Diff line change
`@@ -51,6 +51,7 @@ func isMissingDeviceDriverError(err error) bool {`
`51`	`51`	`strings.Contains(msg, "nvidia-container-cli: initialization error")`
`52`	`52`	`}`
`53`	`53`
	`54`	`+// isNetworkError checks if the error is a network error. This is janky and intended for use in tests only`
`54`	`55`	`func isNetworkError(err error) bool {`
`55`	`56`	`// for both CLI and API clients, network errors are wrapped and lose the net.Error interface`
`56`	`57`	`// CLI client: wrapped by exec.Command as exec.ExitError`
`@@ -65,6 +66,7 @@ func isNetworkError(err error) bool {`
`65`	`66`	`"EOF",`
`66`	`67`	`"no route to host",`
`67`	`68`	`"network is unreachable",`
	`69`	`+ "server closed",`
`68`	`70`	`}`
`69`	`71`
`70`	`72`	`for _, errStr := range networkErrorStrings {`