fix: change enqueue error to debug log level (#19686)

spikecurtis · web-flow · commit 04dfda8a0edb · 2025-09-03T13:42:02.000+04:00
fixes coder/internal#958 Logging was being done at error level, but most likely any errors are from simple races between an update triggered around the same time as a client disconnecting. Debug is fine for these.
diff --git a/enterprise/tailnet/pgcoord.go b/enterprise/tailnet/pgcoord.go
@@ -693,8 +693,9 @@ func (m *mapper) run() {
 			m.logger.Debug(m.ctx, "skipping nil node update")
 			continue
 		}
-		if err := m.c.Enqueue(update); err != nil && !xerrors.Is(err, context.Canceled) {
-			m.logger.Error(m.ctx, "failed to enqueue node update", slog.Error(err))
+		if err := m.c.Enqueue(update); err != nil {
+			// lots of reasons this could happen, most usually, the peer has disconnected.
+			m.logger.Debug(m.ctx, "failed to enqueue node update", slog.Error(err))
 		}
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -693,8 +693,9 @@ func (m *mapper) run() {`
`693`	`693`	`m.logger.Debug(m.ctx, "skipping nil node update")`
`694`	`694`	`continue`
`695`	`695`	`}`
`696`		`- if err := m.c.Enqueue(update); err != nil && !xerrors.Is(err, context.Canceled) {`
`697`		`- m.logger.Error(m.ctx, "failed to enqueue node update", slog.Error(err))`
	`696`	`+ if err := m.c.Enqueue(update); err != nil {`
	`697`	`+ // lots of reasons this could happen, most usually, the peer has disconnected.`
	`698`	`+ m.logger.Debug(m.ctx, "failed to enqueue node update", slog.Error(err))`
`698`	`699`	`}`
`699`	`700`	`}`
`700`	`701`	`}`