queue: Claim Task response to include Task Definition

RESOLVED FIXED

Status

RESOLVED FIXED
4 years ago
3 years ago

People

(Reporter: pmoore, Unassigned)

Tracking

Details

(Reporter)

Description

4 years ago
Currently when a worker claims a task, it needs to make a separate call to get the Task Definition.

We could return the Task Definition as part of the Claim Task response.

Note, this is not needed for the Reclaim Task response, since when reclaiming, the Task Definition should not have changed, and you should have already received it with the initial claim.
This would reduce requests...

The only objection I can come up with is that it's not pretty :)
 - I'm pretty sure that's not a valid reason.
(Reporter)

Updated

3 years ago
Component: TaskCluster → Queue
Product: Testing → Taskcluster
Will be done in bug 1145802
Depends on: 1134342
(Reporter)

Comment 3

3 years ago
The remaining part after bug 1145802 is resolved is updating workers to use it.

There is no need to have a coordinated rollout, as far as I understand, as the task definition is still available via the old mechanics.

@jonasfj
Please advise if there will be any breaking changes to the Claim Task response (i.e. existing fields being removed or changed). Thanks!
(Reporter)

Updated

3 years ago
Depends on: 1219612
Landed an in production... sorry this thing shouldn't have taken so long...
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.