1231361 - Use the django orm instead of RefDataModel

Status: RESOLVED FIXED

(Tree Management :: Treeherder: Data Ingestion, defect)

Description

[Mauro Doglio [:mdoglio]]

Using RefDataModel is not necessary anymore, let's get rid of it.

Comment 2

[William Lachance (:wlach)]

Bug 1239185 won't remove the refdatamodel altogether, just some (most) of it.

Comment 3

[GitHub Autolander Bot]

Attachment: [treeherder] wlach:1231361 > mozilla:master

Comment 4

[William Lachance (:wlach)]

Comment on attachment 8736477 [details] [review]
[treeherder] wlach:1231361 > mozilla:master

I figured I'd split the remaining work on this bug into smaller chunks so it's easier for people to review incrementally, instead of doing a monster patch. Cam, could you take a quick look at this? It's pretty much just one big piece of code deletion.

Comment 5

[Treeherder GitHub Bugbot]

Commit pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/e2715170c8e41401dab4c04b43b67a1a15e64cbb
Bug 1231361 - Remove refdata model repository methods

Comment 6

[GitHub Autolander Bot]

Attachment: [treeherder] wlach:1231361-2 > mozilla:master

Comment 7

[William Lachance (:wlach)]

Comment on attachment 8737516 [details] [review]
[treeherder] wlach:1231361-2 > mozilla:master

Hey Ed, do you want to take a look at this? I tried to just make this a straight refactoring, there's lots I noticed that could be improved/changed but I tried to keep my eye on the prize of removing datasource. ;)

Comment 8

[GitHub Autolander Bot]

Attachment: [treeherder] mozilla:onhold-repo-fix > mozilla:master

Comment 9

[Cameron Dawson [:camd]]

Comment on attachment 8737952 [details] [review]
[treeherder] mozilla:onhold-repo-fix > mozilla:master

For your reading pleasure, sir.  :)

Comment 10

[Treeherder GitHub Bugbot]

Commit pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/08da3867aba441434393ae100b58d071f01ee62f
Bug 1231361 - fixes for skipping onhold repos

Comment 11

[Ed Morley [:emorley]]

Comment on attachment 8737516 [details] [review]
[treeherder] wlach:1231361-2 > mozilla:master

Looks great - nice work! :-)

Comment 12

[Treeherder GitHub Bugbot]

Commits pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/15ba5b10b94ce89cc6d11c092b9dacc14e061acf
Bug 1231361 - Remove bugscache methods from refdata model

Instead, just use the ORM directly where needed.

https://github.com/mozilla/treeherder/commit/649e6bfcabf9d6d93c70118b7d595e0585af7a6c
Bug 1231361 - Remove reference data signature methods from refdata model

https://github.com/mozilla/treeherder/commit/ea0697b13504953f9df5d52c4116d3e400caad8b
Bug 1231361 - Remove option collection method from refdata model

https://github.com/mozilla/treeherder/commit/92f02ccc25b0c6ae8c81686f8af22d713be9dc11
Bug 1231361 - Remove refdata model dependency from unit tests

https://github.com/mozilla/treeherder/commit/5e24106af844b1a38452b951a4ca8925a18c0274
Bug 1231361 - Remove the last of the refdata model

The only thing we were still using it for was to get a database name,
which we can just as easily grab from settings.

https://github.com/mozilla/treeherder/commit/53420b81910a4e82770c0605181227ccc6750065
Merge pull request #1385 from wlach/1231361-2

Bug 1231361 - Remove last of refdata model

Comment 14

[William Lachance (:wlach)]

Goodbye refdata model, we hardly knew ye.

Comment 15

[GitHub Autolander Bot]

Attachment: [treeherder] wlach:1231361 > mozilla:master

Comment 16

[Treeherder GitHub Bugbot]

Commit pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/fce6c1c342bb196303ae023fe18239df1c848209
Bug 1231361 - Fix performance artifact ingestion

We weren't taking the repository into account when getting the reference data
signature.

Comment 17

[GitHub Autolander Bot]

Attachment: [treeherder] wlach:1231361-3 > mozilla:master

Comment 18

[Treeherder GitHub Bugbot]

Commit pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/464c82754ed947e7f5f4216027f2eb121ddd2e19
Bug 1231361 - Fix performance artifact ingestion more robustly

Comment 19

[GitHub Autolander Bot]

Attachment: [treeherder] wlach:1231361-4 > mozilla:master

Comment 20

[Treeherder GitHub Bugbot]

Commit pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/50af0d9a0230708037394669e84701d10524509b
Bug 1231361 - Remove one last piece of the old refdata model (#1723)