Closed
Bug 1418058
Opened 6 years ago
Closed 6 years ago
Fix action tasks to get proper repo and revisions for comm-central trees.
Categories
(Thunderbird :: Build Config, enhancement, P2)
Thunderbird
Build Config
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 61.0
People
(Reporter: tomprince, Assigned: tomprince)
References
Details
Attachments
(8 files)
59 bytes,
text/x-review-board-request
|
Details | |
45 bytes,
text/x-phabricator-request
|
dustin
:
review+
|
Details | Review |
45 bytes,
text/x-phabricator-request
|
dustin
:
review+
|
Details | Review |
45 bytes,
text/x-phabricator-request
|
dustin
:
review+
|
Details | Review |
45 bytes,
text/x-phabricator-request
|
dustin
:
review+
|
Details | Review |
45 bytes,
text/x-phabricator-request
|
Details | Review | |
1.00 KB,
patch
|
Details | Diff | Splinter Review | |
46 bytes,
text/x-phabricator-request
|
dustin
:
review+
|
Details | Review |
No description provided.
Comment 1•6 years ago
|
||
I think fixing this one will also allow me to add tasks to builds via treeherder. Right now it doesn't work because actions.json contains repository.url = mozilla-central and therefore treeherder breaks on permrissions. If that is not the case please let me know so I can file a new bug.
Assignee | ||
Updated•6 years ago
|
Priority: -- → P2
Target Milestone: --- → Thunderbird 59.0
Comment hidden (mozreview-request) |
Pushed by mozilla@hocat.ca: https://hg.mozilla.org/comm-central/rev/9ce6d19a9dca Update .taskcluster.yml in preperation for supporting actions tasks.
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•6 years ago
|
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 4•6 years ago
|
||
Comment 5•6 years ago
|
||
Comment 6•6 years ago
|
||
Comment 7•6 years ago
|
||
Comment 8•6 years ago
|
||
Assignee | ||
Comment 9•6 years ago
|
||
(using needinfo to mirror phabricator review request)
Flags: needinfo?(dustin)
Comment 10•6 years ago
|
||
Comment on attachment 8967119 [details] Bug 1418058: [taskcluster] Add the prefix for the graph parameters of the primary repo to the graph config. Dustin J. Mitchell [:dustin] pronoun: he has approved the revision. https://phabricator.services.mozilla.com/D913
Attachment #8967119 -
Flags: review+
Comment 11•6 years ago
|
||
Comment on attachment 8967120 [details] Bug 1418058: [taskcluster] Load graph configuration for the correct project when running actions. Dustin J. Mitchell [:dustin] pronoun: he has approved the revision. https://phabricator.services.mozilla.com/D914
Attachment #8967120 -
Flags: review+
Comment 12•6 years ago
|
||
Comment on attachment 8967118 [details] Bug 1418058: [taskcluster] Pass a graph_config through action task generation. Dustin J. Mitchell [:dustin] pronoun: he has approved the revision. https://phabricator.services.mozilla.com/D912
Attachment #8967118 -
Flags: review+
Updated•6 years ago
|
Flags: needinfo?(dustin)
Comment 13•6 years ago
|
||
Comment on attachment 8967121 [details] Bug 1418058: [taskcluster] Get the `.taskcluster.yml` from the current project. Dustin J. Mitchell [:dustin] pronoun: he has approved the revision. https://phabricator.services.mozilla.com/D915
Attachment #8967121 -
Flags: review+
Comment 14•6 years ago
|
||
Pushed by mozilla@hocat.ca: https://hg.mozilla.org/integration/mozilla-inbound/rev/9fbb232ef1f1 [taskcluster] Pass a graph_config through action task generation; r=dustin https://hg.mozilla.org/integration/mozilla-inbound/rev/f23ff7f39f9a [taskcluster] Add the prefix for the graph parameters of the primary repo to the graph config; r=dustin https://hg.mozilla.org/integration/mozilla-inbound/rev/5b90e2614663 [taskcluster] Load graph configuration for the correct project when running actions; r=dustin https://hg.mozilla.org/integration/mozilla-inbound/rev/9f9c19355f64 [taskcluster] Get the `.taskcluster.yml` from the current project; r=dustin https://hg.mozilla.org/integration/mozilla-inbound/rev/c165de963471 [taskcluster] Update actions to take and use graph configuration; r=dustin
Assignee | ||
Comment 15•6 years ago
|
||
This patch will need to land on c-c once the rest merges to m-c.
Flags: needinfo?(jorgk)
Comment 16•6 years ago
|
||
Thanks, will do.
Comment 17•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/9fbb232ef1f1 https://hg.mozilla.org/mozilla-central/rev/f23ff7f39f9a https://hg.mozilla.org/mozilla-central/rev/5b90e2614663 https://hg.mozilla.org/mozilla-central/rev/9f9c19355f64 https://hg.mozilla.org/mozilla-central/rev/c165de963471
Status: REOPENED → RESOLVED
Closed: 6 years ago → 6 years ago
Resolution: --- → FIXED
Target Milestone: Thunderbird 59.0 → Thunderbird 61.0
Comment 18•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/9fbb232ef1f1 https://hg.mozilla.org/mozilla-central/rev/f23ff7f39f9a https://hg.mozilla.org/mozilla-central/rev/5b90e2614663 https://hg.mozilla.org/mozilla-central/rev/9f9c19355f64 https://hg.mozilla.org/mozilla-central/rev/c165de963471
Comment 19•6 years ago
|
||
Pushed by mozilla@jorgk.com: https://hg.mozilla.org/comm-central/rev/8da2ba3d4cac Add project-repository-prefix config for comm-central; r=me
Updated•6 years ago
|
Flags: needinfo?(jorgk)
Comment 20•6 years ago
|
||
I think you missed at least the cancel all action, see https://tools.taskcluster.net/groups/KLDupjsHRWa8FOMEHIjMqA/tasks/QfvjB14hS1e4B4W0Ts_eNw/runs/0/logs/public%2Flogs%2Flive_backing.log
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 21•6 years ago
|
||
Comment 22•6 years ago
|
||
Comment on attachment 8970302 [details] Bug 1418058: [taskcluster] Add `graph_config` to some actions that were originally missed; r?dustin Dustin J. Mitchell [:dustin] pronoun: he has approved the revision. https://phabricator.services.mozilla.com/D1025
Attachment #8970302 -
Flags: review+
Comment 23•6 years ago
|
||
It would be great to uplift this to esr60, as otherwise it will conflict pretty badly with bug 1415868. It sounds like that's already more-or-less the plan.
Comment 24•6 years ago
|
||
Pushed by mozilla@hocat.ca: https://hg.mozilla.org/integration/mozilla-inbound/rev/51c27d9d25ea [taskcluster] Add `graph_config` to some actions that were originally missed; r=dustin
Comment 25•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/51c27d9d25ea
Status: REOPENED → RESOLVED
Closed: 6 years ago → 6 years ago
Resolution: --- → FIXED
Comment 26•6 years ago
|
||
Ported the hunk from comment #19 to comm-esr60: https://hg.mozilla.org/releases/comm-esr60/rev/647db0270eae4e2d2ec577f1d82495ba0bb20c67
You need to log in
before you can comment on or make changes to this bug.
Description
•