Closed Bug 1347655 Opened 7 years ago Closed 7 years ago

Getting decision task id shouldn't use jobdetail api

Categories

(Tree Management :: Treeherder, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: wlach, Assigned: wlach)

References

Details

Attachments

(1 file)

Right now we get the "inspect task" job detail (and munge the link via a regular expression) to get the task id. We'd like to be able to get rid of these links at some point, so we'll need to fetch the task id in some other way.

Instead of hitting the job details api, we should be able to just hit the jobs api directly and the get taskcluster_metadata dict:

/api/project/try/jobs/1234/
Comment on attachment 8847754 [details] [review]
[treeherder] wlach:1347655 > mozilla:master

With this fixed, I think we should finally be able to stop storing all those silly taskcluster links in the job details table.
Attachment #8847754 - Flags: review?(emorley)
Comment on attachment 8847754 [details] [review]
[treeherder] wlach:1347655 > mozilla:master

Looks good!
Attachment #8847754 - Flags: review?(emorley) → review+
Assignee: nobody → wlachance
Commit pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/9ff7e98c78a48a958e9c9ef34c13a932aa29d81d
Bug 1347655 - Don't use "inspect task" link to get tc id of decision task (#2257)
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: