refactor: remove template lifecycle update changes

coder · ssncferreira · Aug 8, 2025 · Aug 8, 2025 · Aug 8, 2025 · Aug 8, 2025
commit 2761b7e5526d7c062648d99c3c3873f0631c3c5d
diff --git a/coderd/database/queries.sql.go b/coderd/database/queries.sql.go
diff --git a/coderd/database/queries/workspaces.sql b/coderd/database/queries/workspaces.sql
@@ -567,11 +567,7 @@ UPDATE
 SET
 	ttl = $2
 WHERE
-	template_id = $1
-	-- Prebuilt workspaces (identified by having the prebuilds system user as owner_id)
-	-- should not have their TTL updated, as they are handled by the prebuilds
-	-- reconciliation loop.
-	AND workspaces.owner_id != 'c42fdf75-3097-471c-8c33-fb52454d81c0'::UUID;
+	template_id = $1;
 
 -- name: UpdateWorkspaceLastUsedAt :exec
 UPDATE
@@ -821,11 +817,8 @@ SET
     dormant_at = CASE WHEN @dormant_at::timestamptz > '0001-01-01 00:00:00+00'::timestamptz THEN @dormant_at::timestamptz ELSE dormant_at END
 WHERE
     template_id = @template_id
-	AND dormant_at IS NOT NULL
-	-- Prebuilt workspaces (identified by having the prebuilds system user as owner_id)
-	-- should not have their dormant or deleting at set, as these are handled by the
-    -- prebuilds reconciliation loop.
-	AND workspaces.owner_id != 'c42fdf75-3097-471c-8c33-fb52454d81c0'::UUID
+AND
+    dormant_at IS NOT NULL
 RETURNING *;
 
 -- name: UpdateTemplateWorkspacesLastUsedAt :exec

diff --git a/enterprise/coderd/schedule/template.go b/enterprise/coderd/schedule/template.go
@@ -242,10 +242,6 @@ func (s *EnterpriseTemplateScheduleStore) Set(ctx context.Context, db database.S
 		nextStartAts := []time.Time{}
 
 		for _, workspace := range workspaces {
-			// Skip prebuilt workspaces
-			if workspace.IsPrebuild() {
-				continue
-			}
 			nextStartAt := time.Time{}
 			if workspace.AutostartSchedule.Valid {
 				next, err := agpl.NextAllowedAutostart(s.now(), workspace.AutostartSchedule.String, templateSchedule)
@@ -258,7 +254,7 @@ func (s *EnterpriseTemplateScheduleStore) Set(ctx context.Context, db database.S
 			nextStartAts = append(nextStartAts, nextStartAt)
 		}
 
-		//nolint:gocritic // We need to be able to update information about regular workspaces.
+		//nolint:gocritic // We need to be able to update information about all workspaces.
 		if err := db.BatchUpdateWorkspaceNextStartAt(dbauthz.AsSystemRestricted(ctx), database.BatchUpdateWorkspaceNextStartAtParams{
 			IDs:          workspaceIDs,
 			NextStartAts: nextStartAts,
@@ -338,11 +334,6 @@ func (s *EnterpriseTemplateScheduleStore) updateWorkspaceBuild(ctx context.Conte
 		return xerrors.Errorf("get workspace %q: %w", build.WorkspaceID, err)
 	}
 
-	// Skip lifecycle updates for prebuilt workspaces
-	if workspace.IsPrebuild() {
-		return nil
-	}
-
 	job, err := db.GetProvisionerJobByID(ctx, build.JobID)
 	if err != nil {
 		return xerrors.Errorf("get provisioner job %q: %w", build.JobID, err)