Closed Bug 1276268 Opened 9 years ago Closed 5 years ago

We don't expire old reference data (eg reference data signatures)

Categories

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

defect

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: emorley, Unassigned)

References

(Blocks 1 open bug)

Details

Bug 1268593 added data cycling for groups/machines, however there are still a few other reference data tables that are not purged. This bloats the sheriff exclusion profile editor and makes it annoying to use (bug 1266116, bug 1276234 comment 0).
Depends on: 1329694
No longer depends on: 1329694
Component: Treeherder → Treeherder: Infrastructure

I am interested to work on this. Kindly let me know how to proceed further for the same!

Flags: needinfo?(armenzg)

I'm looking at the data and I have no clue on how to determine what needs expiring.
I think if you take some time trying to figure out if those models are used anywhere and how they're used would be a good start.
Also look at the bugs Ed mentioned for background about this problem.

Aryx, do you have any pointers or context for this bug? Is it worth trying to fix this?

I believe investigating bug 1598433 would be a better use of your energy.

As reference, these are the models:
https://github.com/mozilla/treeherder/blob/master/treeherder/model/models.py#L1096-L1115
https://github.com/mozilla/treeherder/blob/master/treeherder/model/models.py#L284-L287

Flags: needinfo?(armenzg) → needinfo?(aryx.bugmail)

Exclusion profiles got removed in bug 1387640. Have never seen that admin.html page.

Cam might have pointers if anything still needs to be removed, else WONTFIXing sounds sane.

Flags: needinfo?(aryx.bugmail) → needinfo?(cdawson)

Yeah, I don't think this bug is really valid anymore. Certainly for the reasons Aryx mentioned. But I also think we are data-cycling fairly appropriately now.

I will close this one. But I think Bug 1578195 is a better use of time. Don't remove the table, just be sure it's getting cycled properly.

Flags: needinfo?(cdawson)
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.