Closed Bug 1414924 Opened 2 years ago Closed 2 years ago

test-action-callback prints task definitions scrambled

Categories

(Firefox Build System :: Task Configuration, task)

task
Not set

Tracking

(Not tracked)

RESOLVED FIXED
mozilla58

People

(Reporter: rail, Assigned: rail)

Details

Attachments

(1 file)

I guess because we "submit" them in parallel.
Comment on attachment 8925631 [details]
Bug 1414924 - test-action-callback prints task definitions scrambled

https://reviewboard.mozilla.org/r/196740/#review201996

::: taskcluster/taskgraph/create.py:67
(Diff revision 1)
>                  task_def.setdefault('dependencies', []).append(decision_task_id)
>  
>          task_def['taskGroupId'] = task_group_id
>          task_def['schedulerId'] = scheduler_id
>  
>      with futures.ThreadPoolExecutor(CONCURRENCY) as e:

It might be better to set this to 1 here if `testing`, rather than setting `CONCURRENCY` elsewhere.
Attachment #8925631 - Flags: review?(dustin) → review+
Pushed by raliiev@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/47849caec7a2
test-action-callback prints task definitions scrambled r=dustin
Yeah, makes more sense and it'd be more obvious.
https://hg.mozilla.org/mozilla-central/rev/47849caec7a2
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
Product: TaskCluster → Firefox Build System
You need to log in before you can comment on or make changes to this bug.