Closed Bug 1301477 Opened 8 years ago Closed 7 years ago

Get rid of the artifact model

Categories

(Tree Management :: Treeherder, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: wlach, Assigned: wlach)

References

Details

Attachments

(2 files)

After we've gotten rid of the last of the job artifacts, we should just get rid of the artifact model and replace it with a set of functions that do the things we need.
Depends on: 1303036
Blocks: 1178641
Depends on: 1313779
Comment on attachment 8805713 [details] [review]
[treeherder] wlach:1301477 > mozilla:master

This removes the datasource parts of the artifacts model, though the abstraction is still around. I'm in a bit of a rush to finish up the ds removal refactoring and move on to other things, so I might leave that last bit of cleanup to later.
Attachment #8805713 - Flags: review?(james)
Assignee: nobody → wlachance
Attachment #8805713 - Flags: review?(james) → review+
Commit pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/129adcfab3a5d5b353d4476b0422a55b33c27e34
Bug 1301477 - Disable external facing artifact endpoints and API's (#1965)

* Can no longer store raw artifacts (anything treeherder doesn't understand
  is ignored)
* Attempting to retrieve an artifact now returns a 405 (not allowed)
Comment on attachment 8816510 [details] [review]
[treeherder] wlach:1301477-followup > mozilla:master

This does away with the remainder of the artifacts model.
Attachment #8816510 - Flags: review?(james)
Comment on attachment 8816510 [details] [review]
[treeherder] wlach:1301477-followup > mozilla:master

Cam, could you take a look at this since James is going to be away for the rest of the year? It should be a relatively fast review (there are no functional changes).
Attachment #8816510 - Flags: review?(james) → review?(cdawson)
Attachment #8816510 - Flags: review?(cdawson) → review+
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: