-
Notifications
You must be signed in to change notification settings - Fork 1.1k
feat(coderd/database): add task status and status view #20235
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(coderd/database): add task status and status view #20235
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
| 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", |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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.
a05d8dc to
f805f37
Compare
38e1300 to
e9c6164
Compare
e9c6164 to
a98abac
Compare
f805f37 to
2bcc488
Compare
This change updates the `task_workspace_apps` table structure for improved linking to workspace builds and adds queries to manage tasks and a view to expose task status. Updates coder/internal#948 Supersedes #20212 Supersedes #19773
2bcc488 to
b3c188e
Compare
a98abac to
299a54a
Compare

This change updates the
task_workspace_appstable structure forimproved linking to workspace builds and adds queries to manage tasks
and a view to expose task status.
Updates coder/internal#948
Supersedes #20212
Supersedes #19773