Can't cancel or repeat task on C-C treeherder
Categories
(Thunderbird :: Build Config, defect)
Tracking
(thunderbird_esr6066+ fixed)
People
(Reporter: jorgk-bmo, Unassigned)
Details
Attachments
(2 files)
Trying to repeat a task I get: Taskcluster: While firing hook: InterpreterError at template.payload.env[2]["ACTION_TASK"]: unknown context value task --- * method: triggerHook * errorCode: InputError * statusCode: 400 * time: 2018-09-18T17:21:24.776Z Tom said on IRC: 19:51:00 - tomprince: The error message you quote suggests the `.taskcluster.yml` may have diverged from firefox's in a way that impacts actions. 19:55:24 - tomprince: https://bugzilla.mozilla.org/show_bug.cgi?id=1491371 is also related (but not the issue you are seeing on c-c). Apparently also part of the problem was bug 1470622 which has been resolved.
Comment 1•3 years ago
|
||
> Apparently also part of the problem was bug 1470622 which has been resolved.
That bug is what caused the issue to become visible, as it caused treeherder to start using in-tree actions for things like retrigger/cancel, instead of ad-hoc implementations.
Comment 2•3 years ago
|
||
I talked to Tom about this briefly last night. I think I know what needs to be changed, but I don't have all of the necessary access to test. We'll be working on it today.
Comment 3•3 years ago
|
||
The idea is to make actions work again from treeherder
Comment 4•3 years ago
|
||
Comment on attachment 9010653 [details] Bug 1492219 - Update .taskcluster.yml with (some) changes from M-C Tom Prince [:tomprince] has approved the revision.
Pushed by mozilla@jorgk.com: https://hg.mozilla.org/comm-central/rev/66c355e529f5 Update .taskcluster.yml with (some) changes from M-C r=tomprince
Reporter | ||
Updated•3 years ago
|
Pushed by mozilla@jorgk.com: https://hg.mozilla.org/comm-central/rev/bdd596274fb1 Follow-up: Fixed indentation/white-space issues. rs=white-space-only DONTBUILD
Reporter | ||
Comment 8•3 years ago
|
||
Hmm, tried to retrigger a job on TreeHerder and got: Taskcluster: No such hook --- * method: triggerHook * errorCode: ResourceNotFound * statusCode: 404 * time: 2018-09-21T08:39:35.255Z
Comment 9•3 years ago
|
||
It will probably need to go to comm-beta and comm-esr60 once it works on central. Tom mentioned something about needing to make changes in the ci-admin repository I believe.
Comment 10•3 years ago
|
||
ci-admin needs to be run by someone with permissions after changes to `.taskcluster.yml`. You can ask in #taskcluster. There is Bug 1491371 about figuring out how to pin versions on release branches without changing that file (and thus requiring a ci-admin run to allow actions to work).
Reporter | ||
Comment 11•3 years ago
|
||
Cancel seems to work again on the C-C tree, I've just cancelled a bunch of stuff.
Reporter | ||
Comment 12•2 years ago
|
||
Repeat works, too.
Reporter | ||
Comment 13•2 years ago
|
||
It worked for a while, now I get this on try: Taskcluster: No such hook --- * method: triggerHook * errorCode: ResourceNotFound * statusCode: 404 * time: 2018-10-18T18:51:01.662Z
Comment 14•2 years ago
|
||
I just reran a couple jobs on try without any problems. Can you try again and if it's still not working I'll look at it more.
Comment 15•2 years ago
|
||
Modifying .taskcluster.yml breaks actions. See bug 1491371.
Comment 16•2 years ago
|
||
Is there any chance to uplift https://hg.mozilla.org/comm-central/rev/66c355e529f5 to ESR60 as well? Ship-it v2 relies on this if you need to cancel a release.
Reporter | ||
Comment 17•2 years ago
|
||
Yes, I can. The question is whether we can use the release we built
https://treeherder.mozilla.org/#/jobs?repo=comm-esr60&revision=ec1b1f6de68c18251ec3699b88dd4575ffdda628
or whether we need to build a new one.
In the former case I'd push that with DONTBUILD (which doesn't appear to be popular amongst releng people).
Reporter | ||
Comment 18•2 years ago
|
||
Part of that hunk here, BTW
https://hg.mozilla.org/comm-central/rev/66c355e529f5#l1.12
+ # Hardcode cron push info for now, so that we can transition to using real values without breaking callers of Chain of Trust
+ _pushId: {$if: 'tasks_for == "cron"', then: '-1', else: {$eval: 'push.pushlog_id'}}
+ # action tasks can fail because of no pushdate or push comment information in context, so include them in
+ # hardcodes (even though they don't use these variables)
+ _pushDate: {$if: 'tasks_for == "cron" || tasks_for == "action"', then: '0', else: {$eval: 'push.pushdate'}}
+ _pushComment: {$if: 'tasks_for == "cron" || tasks_for == "action"', then: '', else: {$eval: 'push.comment'}}
would have been removed when uplifting
https://hg.mozilla.org/releases/comm-beta/rev/80b01cce09596e7e664cb5e356bf8a8b061fe45f#l1.67
- # action tasks can fail because of no pushdate or push comment information in context, so include them in
- # hardcodes (even though they don't use these variables)
- _pushDate: {$if: 'tasks_for == "cron" || tasks_for == "action"', then: '0', else: {$eval: 'push.pushdate'}}
- _pushComment: {$if: 'tasks_for == "cron" || tasks_for == "action"', then: '', else: {$eval: 'push.comment'}}
but since it wasn't there, I didn't remove it ;-) - I guess we want the _pushId?
Reporter | ||
Comment 20•2 years ago
|
||
Comment 21•2 years ago
|
||
(In reply to Jorg K (GMT+1) from comment #17)
Yes, I can. The question is whether we can use the release we built
https://treeherder.mozilla.org/#/jobs?repo=comm-
esr60&revision=ec1b1f6de68c18251ec3699b88dd4575ffdda628
or whether we need to build a new one.
The patch won't affect the existing task group, so we can't cancel the existing release this way. I'll figure out a way to make the release disappear from the dashboard. :)
In the former case I'd push that with DONTBUILD (which doesn't appear to be
popular amongst releng people).
I'd land it without DONTBUILD in case we decide to use the new revision for the next release build.
Updated•2 years ago
|
Reporter | ||
Comment 22•2 years ago
|
||
https://hg.mozilla.org/releases/comm-esr60/rev/3d01fb2a6a9bb8f718eb3b309aeb05aca05639d4
As you wish :-) - Thanks for your help.
Comment 23•2 years ago
|
||
Thank you!
Description
•