removed the 'started' constraint from unhanger

ibetitsmike · ibetitsmike · commit 6243ff01ae88 · 2025-05-09T13:17:22.000Z
diff --git a/coderd/database/queries.sql.go b/coderd/database/queries.sql.go
diff --git a/coderd/database/queries/provisionerjobs.sql b/coderd/database/queries/provisionerjobs.sql
@@ -263,7 +263,6 @@ FROM
 	provisioner_jobs
 WHERE
 	updated_at < $1
-	AND started_at IS NOT NULL
 	AND completed_at IS NULL;
 
 -- name: InsertProvisionerJobTimings :many
diff --git a/coderd/unhanger/detector.go b/coderd/unhanger/detector.go
@@ -229,14 +229,6 @@ func unhangJob(ctx context.Context, log slog.Logger, db database.Store, pub pubs
 			return xerrors.Errorf("get provisioner job: %w", err)
 		}
 
-		// Check if we should still unhang it.
-		if !job.StartedAt.Valid {
-			// This shouldn't be possible to hit because the query only selects
-			// started and not completed jobs, and a job can't be "un-started".
-			return jobIneligibleError{
-				Err: xerrors.New("job is not started"),
-			}
-		}
 		if job.CompletedAt.Valid {
 			return jobIneligibleError{
 				Err: xerrors.Errorf("job is completed (status %s)", job.JobStatus),
