1251845 - We should only generate performance alerts on specific repositories

Status: RESOLVED FIXED

(Tree Management :: Perfherder, defect)

Description

[William Lachance (:wlach)]

Right now we generate alerts on any repository not marked as a try repository, but there are some cases (e.g. b2g-inbound) where we don't want to do this. We should probably add a property to the repository model on whether we should generate performance alerts for it (default False).

Relevant code in:

treeherder/model/models.py
treeherder/etl/perf.py

Comment 1

[Joel Maher ( :jmaher ) (UTC -8)]

do we have more work to do here?  I thought we had turned this off everywhere we don't want it.

Comment 2

[William Lachance (:wlach)]

We have no formal support for this yet, so this is probably still relevant (albeit low priority).

Comment 3

[GitHub Autolander Bot]

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

Comment 5

[Joel Maher ( :jmaher ) (UTC -8)]

Comment on attachment 8740483 [details] [review]
[treeherder] wlach:1251845 > mozilla:master

nice!

Comment 6

[GitHub Autolander Bot]

Pull request has landed in master: https://github.com/mozilla/treeherder/commit/c552b4774f9e6eceefd5f326652e7368aeb858ca

Comment 7

[Treeherder GitHub Bugbot]

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

https://github.com/mozilla/treeherder/commit/3bf0fefd0fe4187c517c29a3f160af3e68678266
Bug 1251845 - Only generate performance alerts on specific repositories

Make a repository property on whether we should generate performance alerts,
default to false.