Closed Bug 1278597 Opened 8 years ago Closed 8 years ago

job info artifact refactoring broke mozscreenshots

Categories

(Tree Management :: Treeherder: API, defect)

defect
Not set
normal

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 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: nobody → wlachance
Attachment #8760831 - Flags: review?(james) → review+
Keywords: autoland
Status: NEW → RESOLVED
Closed: 8 years ago
Keywords: autoland
Resolution: --- → FIXED
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
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.

Attachment

General

Created:
Updated:
Size: