fix(coderd): use database.IsQueryCanceledError instead of xerrors.Is(err, context.Canceled) (coder#12325)

johnstcn · web-flow · commit 1465ee2ed1b7 · 2024-02-28T21:19:57.000+10:00
diff --git a/coderd/agentapi/lifecycle.go b/coderd/agentapi/lifecycle.go
@@ -112,7 +112,7 @@ func (a *LifecycleAPI) UpdateLifecycle(ctx context.Context, req *agentproto.Upda
 		ReadyAt:        readyAt,
 	})
 	if err != nil {
-		if !xerrors.Is(err, context.Canceled) {
+		if !database.IsQueryCanceledError(err) {
 			// not an error if we are canceled
 			logger.Error(ctx, "failed to update lifecycle state", slog.Error(err))
 		}

Original file line number	Diff line number	Diff line change
`@@ -112,7 +112,7 @@ func (a LifecycleAPI) UpdateLifecycle(ctx context.Context, req agentproto.Upda`
`112`	`112`	`ReadyAt: readyAt,`
`113`	`113`	`})`
`114`	`114`	`if err != nil {`
`115`		`- if !xerrors.Is(err, context.Canceled) {`
	`115`	`+ if !database.IsQueryCanceledError(err) {`
`116`	`116`	`// not an error if we are canceled`
`117`	`117`	`logger.Error(ctx, "failed to update lifecycle state", slog.Error(err))`
`118`	`118`	`}`