Closed
Bug 1301477
Opened 8 years ago
Closed 8 years ago
Get rid of the artifact model
Categories
(Tree Management :: Treeherder, defect)
Tree Management
Treeherder
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.
Comment 2•8 years ago
|
||
Assignee | ||
Comment 3•8 years ago
|
||
Comment on attachment 8805713 [details]
[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)
Updated•8 years ago
|
Assignee: nobody → wlachance
Updated•8 years ago
|
Attachment #8805713 -
Flags: review?(james) → review+
Comment 4•8 years ago
|
||
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 5•8 years ago
|
||
Assignee | ||
Comment 6•8 years ago
|
||
Comment on attachment 8816510 [details]
[treeherder] wlach:1301477-followup > mozilla:master
This does away with the remainder of the artifacts model.
Attachment #8816510 -
Flags: review?(james)
Assignee | ||
Comment 7•8 years ago
|
||
Comment on attachment 8816510 [details]
[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)
Updated•8 years ago
|
Attachment #8816510 -
Flags: review?(cdawson) → review+
Comment 8•8 years ago
|
||
Commit pushed to master at https://github.com/mozilla/treeherder
https://github.com/mozilla/treeherder/commit/0f10dff26aa922259a16e0f1dffb0ea15a23ac94
Bug 1301477 - Convert artifact model into a set of etl methods (#2013)
Assignee | ||
Updated•8 years ago
|
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•