1353456 - index-tasks should use task.scopes = scopes from decision task

Status: RESOLVED FIXED

(Firefox Build System :: Task Configuration, task)

Description

[Jonas Finnemann Jensen (:jonasfj)]

basically, task.scopes is too long.

Comment 1

[Jonas Finnemann Jensen (:jonasfj)]

If you find the code it should be pretty easy... We can also try to do smarter things by identifying the index scopes...

Comment 2

[Jonas Finnemann Jensen (:jonasfj)]

This is at:
https://dxr.mozilla.org/mozilla-central/source/taskcluster/taskgraph/morph.py#105-106

Comment 3

[Dustin J. Mitchell [:dustin] (he/him)]

I'm going to try something a little different, summarizing the index scopes.  The decision role is a little hard to get to (parameters aren't passed in here), and it seems no less hacky than summarizing index scopes.

Comment 4

[Dustin J. Mitchell [:dustin] (he/him)]

Attachment: Bug 1353456: summarize index routes to avoid scope bloat;

Review commit: https://reviewboard.mozilla.org/r/126512/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/126512/

Comment 5

[Jonas Finnemann Jensen (:jonasfj)]

Comment on attachment 8854550 [details]
Bug 1353456: summarize index routes to avoid scope bloat;

https://reviewboard.mozilla.org/r/126512/#review130142

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b66ff847e1c5
summarize index routes to avoid scope bloat; r=jonasfj

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/b66ff847e1c5

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/b66ff847e1c5