Commit4b252eb

committed

filling out started_at when unhanging not started jobs

1 parent2f3d606 commit4b252ebCopy full SHA for 4b252eb

File tree

+17

-3

lines changed

+17

-3

lines changed

Lines changed: 4 additions & 1 deletion

Some generated files are not rendered by default. Learn more aboutcustomizing how changed files appear on GitHub.

Lines changed: 2 additions & 1 deletion

Lines changed: 11 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -295,11 +295,21 @@ func unhangJob(ctx context.Context, log slog.Logger, db database.Store, pub pubs`
`295`	`295`	`}`
`296`	`296`	`lowestLogID=newLogs[0].ID`
`297`	`297`
`298`		`-// Mark the job as failed.`
`299`	`298`	`now=dbtime.Now()`
	`299`	`+// If we are unhanging a job that was never picked up by the`
	`300`	`+// provisioner, we need to set the started_at time to the current`
	`301`	`+// time so that the build duration is correct.`
	`302`	`+if!job.StartedAt.Valid {`
	`303`	`+job.StartedAt= sql.NullTime{`
	`304`	`+Time:now,`
	`305`	`+Valid:true,`
	`306`	`+}`
	`307`	`+}`
	`308`	`+// Mark the job as failed.`
`300`	`309`	`err=db.UpdateProvisionerJobWithCompleteByID(ctx, database.UpdateProvisionerJobWithCompleteByIDParams{`
`301`	`310`	`ID:job.ID,`
`302`	`311`	`UpdatedAt:now,`
	`312`	`+StartedAt:job.StartedAt,`
`303`	`313`	`CompletedAt: sql.NullTime{`
`304`	`314`	`Time:now,`
`305`	`315`	`Valid:true,`

Comments

(0)