fix: check for io.EOF error in derpmap to resolve flake (coder#14125)

kylecarbs · web-flow · commit e2cec454bc75 · 2024-08-02T17:08:47.000Z
See: https://github.com/coder/coder/actions/runs/10218717887/job/28275465405?pr=14045
diff --git a/codersdk/workspacesdk/connector.go b/codersdk/workspacesdk/connector.go
@@ -267,7 +267,9 @@ func (tac *tailnetAPIConnector) derpMap(client proto.DRPCTailnetClient) error {
 			if xerrors.Is(err, context.Canceled) || xerrors.Is(err, context.DeadlineExceeded) {
 				return nil
 			}
-			tac.logger.Error(tac.ctx, "error receiving DERP Map", slog.Error(err))
+			if !xerrors.Is(err, io.EOF) {
+				tac.logger.Error(tac.ctx, "error receiving DERP Map", slog.Error(err))
+			}
 			return err
 		}
 		tac.logger.Debug(tac.ctx, "got new DERP Map", slog.F("derp_map", dmp))

Original file line number	Diff line number	Diff line change
`@@ -267,7 +267,9 @@ func (tac *tailnetAPIConnector) derpMap(client proto.DRPCTailnetClient) error {`
`267`	`267`	`if xerrors.Is(err, context.Canceled) \|\| xerrors.Is(err, context.DeadlineExceeded) {`
`268`	`268`	`return nil`
`269`	`269`	`}`
`270`		`- tac.logger.Error(tac.ctx, "error receiving DERP Map", slog.Error(err))`
	`270`	`+ if !xerrors.Is(err, io.EOF) {`
	`271`	`+ tac.logger.Error(tac.ctx, "error receiving DERP Map", slog.Error(err))`
	`272`	`+ }`
`271`	`273`	`return err`
`272`	`274`	`}`
`273`	`275`	`tac.logger.Debug(tac.ctx, "got new DERP Map", slog.F("derp_map", dmp))`