1455448 - Don't include `firefox` in the decision task index.

Status: RESOLVED FIXED

(Firefox Build System :: Task Configuration, task)

Description

[Tom Prince [:tomprince]]

The decision task is used for everything built as part of a push (for mozilla-central, this is firefox, devedition and fennec, as well as tasks that aren't strictly part of any product). Thus, having `firefox` encoded as part of the decision task doesn't make sense.

comm-central currently uses `thunderbird` instead of `firefox`. This means that tools that want to support both need to encode that knowledge (as well as the `gecko.` vs. `comm.` prefix to the routes, but that has security implications).

Comment 1

[Phabricator Automation]

Attachment: Bug 1455448: Don't include `firefox` in the decision task index.

The decision task is used for everything built as part of a push (for
mozilla-central, this is firefox, devedition and fennec, as well as tasks that
aren't strictly part of any product). Thus, having `firefox` encoded as part of
the decision task doesn't make sense.

This changes the route from
    index.gecko.v2.${repository.project}.latest.firefox.decision
to
    index.gecko.v2.${repository.project}.latest.taskgraph.decision
while leaving the latter for backwards compatability with tools that expect it.

Comment 2

[Phabricator Automation]

Comment on attachment 8969454 [details]
Bug 1455448: Don't include `firefox` in the decision task index.

Rail Aliiev [:rail] ⌚️ET has approved the revision.

https://phabricator.services.mozilla.com/D996

Comment 3

[Phabricator Automation]

Comment on attachment 8969454 [details]
Bug 1455448: Don't include `firefox` in the decision task index.

Dustin J. Mitchell [:dustin] pronoun: he has approved the revision.

https://phabricator.services.mozilla.com/D996

Comment 4

[Phabricator Automation]

Comment on attachment 8969454 [details]
Bug 1455448: Don't include `firefox` in the decision task index.

Michael Shal [:mshal] has approved the revision.

https://phabricator.services.mozilla.com/D996

Comment 5

[Phabricator Automation]

Comment on attachment 8969454 [details]
Bug 1455448: Don't include `firefox` in the decision task index.

Flag set by Rail Aliiev [:rail] ⌚️ET is no longer active.
Flag set by Dustin J. Mitchell [:dustin] pronoun: he is no longer active.
Flag set by Michael Shal [:mshal] is no longer active.

https://phabricator.services.mozilla.com/D996

Comment 6

[Phabricator Automation]

Comment on attachment 8969454 [details]
Bug 1455448: Don't include `firefox` in the decision task index.

Dustin J. Mitchell [:dustin] pronoun: he has approved the revision.
Michael Shal [:mshal] has approved the revision.
Rail Aliiev [:rail] ⌚️ET has approved the revision.
Justin Wood (:Callek) has approved the revision.

https://phabricator.services.mozilla.com/D996

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/mozilla-central/rev/28db2c96ac69
Don't include `firefox` in the decision task index; r=dustin,mshal,rail,Callek a=release

Comment 8

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/28db2c96ac69

Comment 9

[Tom Prince [:tomprince]]

https://hg.mozilla.org/releases/mozilla-beta/rev/82e0d69ecde1d708ba327f125e32d7478b7d16b5
https://hg.mozilla.org/releases/mozilla-release/rev/6c1e7178ca23655d400b4895858354fb2b9fb509
https://hg.mozilla.org/releases/mozilla-esr52/rev/f36ff7dcaaebe1336c8526331dc67af895f110de

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/305addd57798
[taskcluster] Use `taskgraph` indexed decision tasks for finding parameters; r=me

Comment 11

[Eliza Balazs [:ebalazs_]]

https://hg.mozilla.org/mozilla-central/rev/305addd57798

Comment 12

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/31ad23dc587d
Don't include `thunderbird` in the decision task index; r=me

Comment 13

[GitHub Bugzilla PR Linker]

Attachment: Link to GitHub pull-request: https://github.com/mozilla/treeherder/pull/3510

Comment 14

[Treeherder GitHub Bugbot]

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

https://github.com/mozilla/treeherder/commit/7bb04c879fcd83d8c2f5eff5e995495e98c4e7bf
Bug 1455448 - Use new index for decision tasks (#3510)