This change updates thetask_workspace_apps table structure for
improved linking to workspace builds and adds queries to manage tasks
and a view to expose task status.

Updatescoder/internal#948
Supersedes#20212
Supersedes#19773

feat(coderd/database): add task status and status view

1faf470

This change updates the `task_workspace_apps` table structure forimproved linking to workspace builds and adds queries to manage tasksand a view to expose task status.Updatescoder/internal#948Supersedes#20212Supersedes#19773

github-actionsbot assignedmafredri

Oct 9, 2025

mafredri mentioned this pull request

Oct 9, 2025

feat(coderd): add tasks rbac object#20234

Open

Copy link

MemberAuthor

mafredri commentedOct 9, 2025•
edited
Loading

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stackon Graphite.
Learn more

This stack of pull requests is managed byGraphite. Learn more aboutstacking.

This was referencedOct 9, 2025

fix(coderd/database): ensure task name uniqueness#20236

Open

feat(coderd): implement task to app linking#20237

Open

mafredri changed the base branch frommafredri/feat-coderd-tasks-data-model-split4 tographite-base/20235

October 9, 2025 10:40

mafredri marked this pull request as ready for review

October 9, 2025 10:41

mafredri requested review fromjohnstcn andDanielleMaywood

October 9, 2025 10:41

johnstcn approved these changes

Oct 9, 2025

View reviewed changes

coderd/database/querier_test.go

		agentState:database.WorkspaceAgentLifecycleStateReady,
		appHealths: []database.WorkspaceAppHealth{database.WorkspaceAppHealthHealthy,database.WorkspaceAppHealthHealthy},
		expectedStatus:database.TaskStatusActive,
		description:"Running job with ready agent and multiple healthy apps should be active",

Copy link

Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

One healthy and one unhealthy app will be error, correct?

Copy link

MemberAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Ah, looks like I've missed the removal of a multi-app tests. We don't care about non-task app health so they're pointless. I'll make the second one unhealthy though and that way we test that health is still OK for task.