1656244 - taskgraph: Use routes that populate a different Treeherder project for Github pull-requests

Status: RESOLVED FIXED

(Firefox Build System :: Task Configuration, enhancement)

Description

[Johan Lorenzo [:jlorenzo]]

We've been populating regular Treeherder projects by using tc-treeherder.v2.{project}.{commit_hash}.0. This pollutes the pulse queues of Treeherder and causes an inconvenience to Treeherder maintainers.

Tom suggested to create a seperate TH project for PRs, just like the try projects we use for HG. For instance: https://treeherder.mozilla.org/#/jobs?repo=ci-admin-try

Comment 1

[Armen [:armenzg]]

IIRC I've seen both tasks for the push to the branch and the tasks for PR.

Thank you!

Comment 2

[Armen [:armenzg]]

If curious, this is what I had to do:
https://github.com/mozilla/treeherder/compare/ebf32746...59fbd290 (ignore Docker and doc changes)

Comment 3

[Tom Prince [:tomprince]]

Attachment: Bug 1656244: Use a separate treeherder route on github PRs; r?jlorenzo

Comment 4

[Tom Prince [:tomprince]]

Attachment: Bug 1656244: Use a separate treeherder route on github PRs; r?jlorenzo

Comment 5

[Tom Prince [:tomprince]]

Attachment: Bug 1656244: Remove on-push treeherder route scopes from github PRs; r?aki!

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/ci/ci-configuration/rev/e890517f304c
Use a separate treeherder route on github PRs; r=jlorenzo

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/ci/taskgraph/rev/6b424c5c2fe2
Use a separate treeherder route on github PRs; r=jlorenzo

Comment 8

[Armen [:armenzg]]

I had to add this logic to not ignore nightly tasks:
https://github.com/mozilla/treeherder/pull/6697/files#diff-609378e93352a69d9c31bfe55b0a8d64R143-R148

Should I go back to listening to all tasks that use "tc-treeherder" ?

Comment 9

[Mihai Tabara [:mtabara]⌚️GMT]

Attachment: Bug 1656244 - fix typo in PR treeherder routes.r=tomprince

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/ci/taskgraph/rev/d40a892a7c80
fix typo in PR treeherder routes.r=bustage

Comment 11

[BugBot [:suhaib / :marco/ :calixte]]

The bug assignee didn't login in Bugzilla in the last 7 months.
:ahal, could you have a look please?
For more information, please visit auto_nag documentation.

Comment 12

[Andrew Halberstadt [:ahal]]

Rebased and will land the last patch here (all affected consumers have been updated).

Comment 13

[BugBot [:suhaib / :marco/ :calixte]]

The bug assignee didn't login in Bugzilla in the last 7 months.
:ahal, could you have a look please?
For more information, please visit auto_nag documentation.

Comment 14

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/ci/ci-configuration/rev/bb432990178c
Remove on-push treeherder route scopes from github PRs; r=jlorenzo,aki