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)
Tree Management
Treeherder
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 1•7 years ago
|
||
Assignee | ||
Comment 2•7 years ago
|
||
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 3•7 years ago
|
||
Comment on attachment 8847754 [details] [review] [treeherder] wlach:1347655 > mozilla:master Looks good!
Attachment #8847754 -
Flags: review?(emorley) → review+
Updated•7 years ago
|
Assignee: nobody → wlachance
Comment 4•7 years ago
|
||
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)
Assignee | ||
Updated•7 years ago
|
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.
Description
•