From 8d9157dc35138dd99202d30f5df946c551735a38 Mon Sep 17 00:00:00 2001 From: Spike Curtis Date: Mon, 11 Dec 2023 14:52:44 +0400 Subject: [PATCH] fix: use provisionerd context when failing job on canceled acquire (#11118) Spotted during code read. We need to use the provisionerd auth context when failing a job due to a lost provisioner daemon. --- coderd/provisionerdserver/provisionerdserver.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/coderd/provisionerdserver/provisionerdserver.go b/coderd/provisionerdserver/provisionerdserver.go index 250308bf93..91d0ec6d50 100644 --- a/coderd/provisionerdserver/provisionerdserver.go +++ b/coderd/provisionerdserver/provisionerdserver.go @@ -274,7 +274,8 @@ func (s *server) AcquireJobWithCancel(stream proto.DRPCProvisionerDaemon_Acquire // in the database. We need to mark this job as failed so the end user can retry if they want to. now := dbtime.Now() err := s.Database.UpdateProvisionerJobWithCompleteByID( - context.Background(), + //nolint:gocritic // Provisionerd has specific authz rules. + dbauthz.AsProvisionerd(context.Background()), database.UpdateProvisionerJobWithCompleteByIDParams{ ID: je.job.ID, CompletedAt: sql.NullTime{