Closed
Bug 1278597
Opened 8 years ago
Closed 8 years ago
job info artifact refactoring broke mozscreenshots
Categories
(Tree Management :: Treeherder: API, defect)
Tree Management
Treeherder: API
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: wlach, Assigned: wlach)
References
Details
Attachments
(2 files)
Oops. It appears that the recent changes to job details broke https://screenshots.mattn.ca/compare/
Fortunately it shouldn't be too hard to add back the functionality it needs (job_id__in and repository filters to the new /jobdetail/ endpoint) and fix the tool.
Comment 1•8 years ago
|
||
Assignee | ||
Comment 2•8 years ago
|
||
Comment on attachment 8760831 [details] [review]
[treeherder] wlach:1278597 > mozilla:master
I think this should do the job. We can change the meaning of `job_id` to reference the jobs table in the main database when we finish the datasource migration.
Attachment #8760831 -
Flags: review?(james)
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → wlachance
Updated•8 years ago
|
Attachment #8760831 -
Flags: review?(james) → review+
Comment 3•8 years ago
|
||
Pull request has landed in master: https://github.com/mozilla/treeherder/commit/8fdd6f0492e8481a6c897639fe6f783d22e2bf1c
Comment 4•8 years ago
|
||
Commit pushed to master at https://github.com/mozilla/treeherder
https://github.com/mozilla/treeherder/commit/a5caf03b757f91eabac93134fff0e7e5b47848de
Bug 1278597 - Add job_id__in and repository filtering to /jobdetail/ endpoint
Comment 5•8 years ago
|
||
Comment 6•8 years ago
|
||
Commits pushed to master at https://github.com/mozilla/treeherder
https://github.com/mozilla/treeherder/commit/d49451c55d767cc78bc04006f425c54de1281818
Bug 1278597 - Increase the limit / count of jobdetail objects returned
https://github.com/mozilla/treeherder/commit/e17b5101a3190035e5267cd60502936ddf0cb043
Merge pull request #1571 from wlach/1278597-2
Bug 1278597 - Increase the limit / count of jobdetail objects returned
You need to log in
before you can comment on or make changes to this bug.
Description
•