Closed Bug 1418058 Opened 2 years ago Closed Last year

Fix action tasks to get proper repo and revisions for comm-central trees.

Categories

(Thunderbird :: Build Config, enhancement, P2)

enhancement

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 61.0

People

(Reporter: tomprince, Assigned: tomprince)

References

Details

Attachments

(8 files)

No description provided.
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.
Priority: -- → P2
Target Milestone: --- → Thunderbird 59.0
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: 2 years ago
Resolution: --- → FIXED
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(using needinfo to mirror phabricator review request)
Flags: needinfo?(dustin)
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 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 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+
Flags: needinfo?(dustin)
Blocks: 1454034
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+
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
Attached patch cc.patch β€” β€” Splinter Review
This patch will need to land on c-c once the rest merges to m-c.
Flags: needinfo?(jorgk)
Thanks, will do.
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/8da2ba3d4cac
Add project-repository-prefix config for comm-central; r=me
Flags: needinfo?(jorgk)
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 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+
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.
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
https://hg.mozilla.org/mozilla-central/rev/51c27d9d25ea
Status: REOPENED → RESOLVED
Closed: 2 years agoLast year
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.