Commit8d9157d

authored

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.

1 parent50575e1 commit8d9157dCopy full SHA for 8d9157d

File tree

-1

lines changed

-1

lines changed

Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -274,7 +274,8 @@ func (s *server) AcquireJobWithCancel(stream proto.DRPCProvisionerDaemon_Acquire`
`274`	`274`	`// in the database. We need to mark this job as failed so the end user can retry if they want to.`
`275`	`275`	`now:=dbtime.Now()`
`276`	`276`	`err:=s.Database.UpdateProvisionerJobWithCompleteByID(`
`277`		`-context.Background(),`
	`277`	`+//nolint:gocritic // Provisionerd has specific authz rules.`
	`278`	`+dbauthz.AsProvisionerd(context.Background()),`
`278`	`279`	`database.UpdateProvisionerJobWithCompleteByIDParams{`
`279`	`280`	`ID:je.job.ID,`
`280`	`281`	`CompletedAt: sql.NullTime{`

Comments

(0)