Skip to content

Commit 30aea1f

Browse files
committed
Fail executing/retrying runs
1 parent adacc88 commit 30aea1f

File tree

1 file changed

+21
-3
lines changed

1 file changed

+21
-3
lines changed

apps/webapp/app/v3/taskRunHeartbeatFailed.server.ts

Lines changed: 21 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -50,8 +50,6 @@ export class TaskRunHeartbeatFailedService extends BaseService {
5050

5151
switch (taskRun.status) {
5252
case "PENDING":
53-
case "EXECUTING":
54-
case "RETRYING_AFTER_FAILURE":
5553
case "WAITING_TO_RESUME":
5654
case "PAUSED": {
5755
const backInQueue = await marqs?.nackMessage(taskRun.id);
@@ -82,9 +80,29 @@ export class TaskRunHeartbeatFailedService extends BaseService {
8280

8381
break;
8482
}
83+
case "EXECUTING":
84+
case "RETRYING_AFTER_FAILURE": {
85+
logger.debug(`[RequeueTaskRunService] ${taskRun.status} failing task run`, { taskRun });
86+
87+
await service.call(taskRun.friendlyId, {
88+
ok: false,
89+
id: taskRun.friendlyId,
90+
retry: undefined,
91+
error: {
92+
type: "INTERNAL_ERROR",
93+
code: TaskRunErrorCodes.TASK_RUN_HEARTBEAT_TIMEOUT,
94+
message: "Did not receive a heartbeat from the worker in time",
95+
},
96+
});
97+
98+
break;
99+
}
85100
case "DELAYED":
86101
case "WAITING_FOR_DEPLOY": {
87-
logger.debug("[TaskRunHeartbeatFailedService] Removing task run from queue", { taskRun });
102+
logger.debug(
103+
`[TaskRunHeartbeatFailedService] ${taskRun.status} Removing task run from queue`,
104+
{ taskRun }
105+
);
88106

89107
await marqs?.acknowledgeMessage(
90108
taskRun.id,

0 commit comments

Comments
 (0)