Closed Bug 1328469 Opened 7 years ago Closed 7 years ago

Remove the logslice API now that the log viewer downloads the logs directly

Categories

(Tree Management :: Treeherder: API, defect, P3)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: emorley, Assigned: emorley)

References

Details

Attachments

(1 file)

Comment on attachment 8823478 [details] [review]
[treeherder] mozilla:rm-logslice-api > mozilla:master

I'll check for usage prior to landing in a couple of days.
Attachment #8823478 - Flags: review?(wlachance)
Comment on attachment 8823478 [details] [review]
[treeherder] mozilla:rm-logslice-api > mozilla:master

Very happy to see this go.
Attachment #8823478 - Flags: review?(wlachance) → review+
Commit pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/422cd3390b7204d3b11e6c88754cbba37625c294
Bug 1328469 - Remove the now unused logslice API

The new log viewer loads the logs directly in the client from wherever
they are stored, so doesn't need Treeherder's API to proxy them.

The logslice API was the only user of the Django filesystem cache, so
that has also been removed.
Aside from a stray logviewer page that had been left open (judging by other accesses from that IP), there's no other usage of the API, no code hits on GitHub - so I've landed this.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Zero hits on the prod API now.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: