fix: check for io.EOF error in derpmap to resolve flake

kylecarbs · kylecarbs · commit 58414c566c9c · 2024-08-02T16:49:26.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
@@ -264,7 +264,7 @@ func (tac *tailnetAPIConnector) derpMap(client proto.DRPCTailnetClient) error {
 	for {
 		dmp, err := s.Recv()
 		if err != nil {
-			if xerrors.Is(err, context.Canceled) || xerrors.Is(err, context.DeadlineExceeded) {
+			if xerrors.Is(err, context.Canceled) || xerrors.Is(err, context.DeadlineExceeded) || xerrors.Is(err, io.EOF) {
 				return nil
 			}
 			tac.logger.Error(tac.ctx, "error receiving DERP Map", slog.Error(err))

Original file line number	Diff line number	Diff line change
`@@ -264,7 +264,7 @@ func (tac *tailnetAPIConnector) derpMap(client proto.DRPCTailnetClient) error {`
`264`	`264`	`for {`
`265`	`265`	`dmp, err := s.Recv()`
`266`	`266`	`if err != nil {`
`267`		`- if xerrors.Is(err, context.Canceled) \|\| xerrors.Is(err, context.DeadlineExceeded) {`
	`267`	`+ if xerrors.Is(err, context.Canceled) \|\| xerrors.Is(err, context.DeadlineExceeded) \|\| xerrors.Is(err, io.EOF) {`
`268`	`268`	`return nil`
`269`	`269`	`}`
`270`	`270`	`tac.logger.Error(tac.ctx, "error receiving DERP Map", slog.Error(err))`