Skip to content

Commit

Permalink
Added logging when we remove a queue’s concurrency limit
Browse files Browse the repository at this point in the history
  • Loading branch information
matt-aitken committed Dec 10, 2024
1 parent 9970b9b commit 9065e64
Show file tree
Hide file tree
Showing 2 changed files with 19 additions and 0 deletions.
9 changes: 9 additions & 0 deletions apps/webapp/app/v3/services/createBackgroundWorker.server.ts
Original file line number Diff line number Diff line change
Expand Up @@ -208,6 +208,15 @@ export async function createBackgroundTasks(
taskQueue.concurrencyLimit
);
} else {
logger.debug("CreateBackgroundWorkerService: removing concurrency limit", {
workerId: worker.id,
taskQueue,
orgId: environment.organizationId,
projectId: environment.projectId,
environmentId: environment.id,
concurrencyLimit,
taskidentifier: task.id,
});
await marqs?.removeQueueConcurrencyLimits(environment, taskQueue.name);
}
} catch (error) {
Expand Down
10 changes: 10 additions & 0 deletions apps/webapp/app/v3/services/triggerTask.server.ts
Original file line number Diff line number Diff line change
Expand Up @@ -474,6 +474,16 @@ export class TriggerTaskService extends BaseService {
taskQueue.concurrencyLimit
);
} else {
logger.debug("TriggerTaskService: removing concurrency limit", {
runId: taskRun.id,
friendlyId: taskRun.friendlyId,
taskQueue,
orgId: environment.organizationId,
projectId: environment.projectId,
existingConcurrencyLimit,
concurrencyLimit,
queueOptions: body.options?.queue,
});
await marqs?.removeQueueConcurrencyLimits(environment, taskQueue.name);
}
}
Expand Down

0 comments on commit 9065e64

Please sign in to comment.