fix: check if context is nil when logging coordination delete errors

kylecarbs · kylecarbs · commit 8e41ada8f4c9 · 2024-05-06T22:56:17.000Z
I'm not exactly sure how to handle this idiomatically... See: https://github.com/coder/coder/actions/runs/8976564938/job/24653672411
diff --git a/enterprise/tailnet/pgcoord.go b/enterprise/tailnet/pgcoord.go
@@ -1654,7 +1654,11 @@ func (h *heartbeats) sendDelete() {
 	ctx := dbauthz.As(context.Background(), pgCoordSubject)
 	err := h.store.DeleteCoordinator(ctx, h.self)
 	if err != nil {
-		h.logger.Error(h.ctx, "failed to send coordinator delete", slog.Error(err))
+		// If this errors and the context is no longer active, this
+		// most likely is a database connection error.
+		if h.ctx.Err() == nil {
+			h.logger.Error(h.ctx, "failed to send coordinator delete", slog.Error(err))
+		}
 		return
 	}
 	h.logger.Debug(h.ctx, "deleted coordinator")
