Closed
Bug 1455448
Opened 6 years ago
Closed 6 years ago
Don't include `firefox` in the decision task index.
Categories
(Firefox Build System :: Task Configuration, task)
Tracking
(firefox61 fixed)
RESOLVED
FIXED
mozilla61
Tracking | Status | |
---|---|---|
firefox61 | --- | fixed |
People
(Reporter: tomprince, Assigned: tomprince)
Details
Attachments
(2 files)
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•6 years ago
|
||
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•6 years ago
|
||
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
Attachment #8969454 -
Flags: review+
Comment 3•6 years ago
|
||
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
Attachment #8969454 -
Flags: review+
Comment 4•6 years ago
|
||
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
Attachment #8969454 -
Flags: review+
Comment 5•6 years ago
|
||
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
Attachment #8969454 -
Flags: review+
Comment 6•6 years ago
|
||
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
Attachment #8969454 -
Flags: review+
Pushed by mozilla@hocat.ca: 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•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/28db2c96ac69
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox61:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
Assignee | ||
Comment 9•6 years ago
|
||
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•6 years ago
|
||
Pushed by mozilla@hocat.ca: https://hg.mozilla.org/integration/mozilla-inbound/rev/305addd57798 [taskcluster] Use `taskgraph` indexed decision tasks for finding parameters; r=me
Comment 11•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/305addd57798
Updated•6 years ago
|
Assignee: nobody → mozilla
Comment 12•6 years ago
|
||
Pushed by mozilla@hocat.ca: https://hg.mozilla.org/comm-central/rev/31ad23dc587d Don't include `thunderbird` in the decision task index; r=me
Comment 13•6 years ago
|
||
Comment 14•6 years ago
|
||
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)
Updated•5 years ago
|
Version: Version 3 → 3 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•