1455632 - Remove deprecated Queue API methods

Status: RESOLVED FIXED

(Taskcluster :: Services, enhancement, P5)

Description

[Dustin J. Mitchell [:dustin] (he/him)]

We have a few deprecated API methods:

    queue.defineTask
    queue.rerunTask

as well as some deprecated ways of using existing APIs, notably

    queue.createTask without priority

We should measure the usage of each of these APIs, set a date for removal, and work with any remaining users to stop using them. Note that queue.defineTask is still used by the scheduler service, so will require bug 1399437 first.

Comment 1

[John Ford [:jhford] CET/CEST Berlin Time]

Marking as P5 because it depends on uncompleted work in bug 1399437

Comment 2

[Vishakha Nihore]

@dustin , We need to remove them completely or there is some replacement query method for this ?

Comment 3

[Dustin J. Mitchell [:dustin] (he/him)]

There are already replacement methods for all of these. This is more of a "research" question -- we know the methods are still used right now, so they can't be removed immediately.

Comment 4

[Tom Prince [:tomprince]]

Note that run-task is heavily used and should not be removed.

Comment 5

[Dustin J. Mitchell [:dustin] (he/him)]

ITYM "rerunTask"? If so, yes, agreed.

Comment 6

[Dustin J. Mitchell [:dustin] (he/him)]

So the remaining method to remove is queue.defineTask (its associated scopes are already removed).

I'll make a quick PR to mark rerunTask as not deprecated -- I thought we had done so long ago.

Comment 7

[Dustin J. Mitchell [:dustin] (he/him)]

https://github.com/taskcluster/taskcluster/pull/2451