Skip to content

feat(run-task): implement hook point before task payload - #1045

Open
ahal wants to merge 2 commits into
taskcluster:mainfrom
ahal:ahal/xlowmzzzqzyy
Open

ahal wants to merge 2 commits into
taskcluster:mainfrom
ahal:ahal/xlowmzzzqzyy

Conversation

@ahal

@ahal ahal commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Sometimes consumers might want to do some additional shared setup before executing the task payload. This implements an environment variable which points to a Python script which will be executed in-process.

Bug: 2066119

@ahal
ahal force-pushed the ahal/xlowmzzzqzyy branch from 979d83c to cbfc464 Compare September 18, 2026 16:10
@ahal

ahal commented Sep 18, 2026

Copy link
Copy Markdown
Contributor Author

This was tested over in:
https://treeherder.mozilla.org/jobs?repo=try&revision=f837a428c489247e8c7cdded60bd84ab601c1c59&selectedTaskRun=MbIoDGe0Tru5ZWII-k07ww.0

(note the green mpu task)

The other failures in that push are mostly due to me disabling strict hashing to point at this PR. Still investigating the ci failure, but that doesn't need to block this PR.

@ahal
ahal marked this pull request as ready for review September 18, 2026 16:16
@ahal
ahal requested a review from a team as a code owner September 18, 2026 16:16
@ahal
ahal requested a review from hneiva September 18, 2026 16:16
@ahal
ahal marked this pull request as draft September 18, 2026 18:38
@ahal
ahal force-pushed the ahal/xlowmzzzqzyy branch from cbfc464 to 6856760 Compare September 21, 2026 16:47
@ahal
ahal marked this pull request as ready for review September 21, 2026 18:51

@jcristau jcristau left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like a reasonable way to let firefox / other consumers customize what run-task does.

Comment thread src/taskgraph/run-task/run-task Outdated
Sometimes consumers might want to do some additional shared setup before
executing the task payload. This implements an environment variable
which points to a Python script which will be executed in-process.

Bug: 2066119
@ahal
ahal force-pushed the ahal/xlowmzzzqzyy branch from 089658b to b717fe9 Compare September 22, 2026 13:54
@ahal
ahal force-pushed the ahal/xlowmzzzqzyy branch from b717fe9 to 4d2b119 Compare September 22, 2026 16:24

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants