Closed Bug 1370428 Opened 7 years ago Closed 5 years ago

fennec 54.0b14 - data.routes should NOT have more than 10 items

Categories

(Release Engineering :: Release Automation: Other, enhancement, P2)

enhancement

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.
Depends on: 1333255
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.
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)
(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)
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.
Severity: normal → major
Priority: -- → P2
- 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.

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)

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)

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.