Closed
Bug 1370428
Opened 8 years ago
Closed 5 years ago
fennec 54.0b14 - data.routes should NOT have more than 10 items
Categories
(Release Engineering :: Release Automation, enhancement, P2)
Release Engineering
Release Automation
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mozilla, Unassigned)
References
Details
IIRC we reduced this back to max 10 routes.
We should probably be chunking according to max # of locales per chunk, rather than some hardcoded number.
Reporter | ||
Comment 1•8 years ago
|
||
Reporter | ||
Comment 2•8 years ago
|
||
The `mach taskgraph morphed` logic appears to only be in mozilla-central. We likely either need to uplift that to all release repos, or raise the route limit again.
Comment 3•8 years ago
|
||
Taskcluster increased their limit again to unblock the release.
Comment #2 refers to bug 1333255, which is only in version 55. We could wait 7 weeks for 54 to exit mozilla-release before lowering the limit again, at the risk of another job sneaking in > 10 routes in the meantime (ie new bustage next time). Or look at uplifting the changes, which could be interesting as they do a fair bit of refactoring which might need porting. What do you think Callek ?
Flags: needinfo?(bugspam.Callek)
Comment 4•8 years ago
|
||
(In reply to Nick Thomas [:nthomas] from comment #3)
> Taskcluster increased their limit again to unblock the release.
>
> Comment #2 refers to bug 1333255, which is only in version 55. We could wait
> 7 weeks for 54 to exit mozilla-release before lowering the limit again, at
> the risk of another job sneaking in > 10 routes in the meantime (ie new
> bustage next time). Or look at uplifting the changes, which could be
> interesting as they do a fair bit of refactoring which might need porting.
> What do you think Callek ?
I've discussed options with the #taskcluster team, and its pretty clear uplifting the changes is not tenable in the short-term.
Flags: needinfo?(bugspam.Callek)
Comment 5•8 years ago
|
||
And were they happy to wait until 55 ships before reducing the route limit again ? Seems to me our only alternative is to have more chunks for Fennec single locale, and see if there is anything we can remove from the main opt routes.
Updated•8 years ago
|
Severity: normal → major
Priority: -- → P2
Reporter | ||
Comment 6•7 years ago
|
||
- we now chunk by locales-per-chunk rather than a hardcoded number
- Fx55 has ridden the trains
We can potentially reduce this limit again, with the caveat that we haven't touched ESR52. At worst, we can reduce the number in August when ESR52 goes away.
Reporter | ||
Comment 7•5 years ago
|
||
Found in triage. We have index tasks now, and we do chunk locales by # of locales rather than # of chunks.
Dustin, do you know if we still have an index limit in Taskcluster that we're looking to lower?
Flags: needinfo?(dustin)
Comment 8•5 years ago
|
||
Per https://firefox-ci-tc.services.mozilla.com/schemas/queue/v1/task.json, the limit is 64. That seems to not be causing problems, so let's just call it fine.
Flags: needinfo?(dustin)
Reporter | ||
Comment 9•5 years ago
|
||
Thanks!
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•