Closed
Bug 1275972
Opened 9 years ago
Closed 9 years ago
Plug Linux64 PGO builds to use coalescer for queue collapsing whilst under load
Categories
(Firefox Build System :: Task Configuration, task)
Firefox Build System
Task Configuration
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mtabara, Assigned: mtabara)
References
Details
Attachments
(1 file)
891 bytes,
patch
|
dustin
:
review+
|
Details | Diff | Splinter Review |
The initial conversation on this took place in bug 1274310. I filed this bug to separately track the work on this.
Assignee | ||
Comment 1•9 years ago
|
||
Changed the coalescer route and supersederUrl to be consistent with the build name used in the index before.
:dustin - do I need to add another entry besides mozilla-inbound in https://github.com/mozilla/tc-coalesce/pull/14/files ?
I will follow-up soon with a patch to tweak the taskcluster/taskgraph/kind/legacy.py to remove the route and supersederUrl for try jobs.
Attachment #8756922 -
Flags: review?(dustin)
Updated•9 years ago
|
Attachment #8756922 -
Flags: review?(dustin) → review+
Assignee | ||
Comment 3•9 years ago
|
||
Thanks :dividehex for merging PR https://github.com/mozilla/tc-coalesce/pull/14/
New threshold is in production now - https://coalesce.mozilla-releng.net/v1/threshold
The patch is now landed in mozilla-inbound:
* https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=bb43b2d9d79f
Not sure yet on how to check that the patch is working but am working on it :)
Comment 4•9 years ago
|
||
(In reply to Mihai Tabara [:mtabara] from comment #3)
> Thanks :dividehex for merging PR
> https://github.com/mozilla/tc-coalesce/pull/14/
> New threshold is in production now -
> https://coalesce.mozilla-releng.net/v1/threshold
>
> The patch is now landed in mozilla-inbound:
> *
> https://treeherder.mozilla.org/#/jobs?repo=mozilla-
> inbound&revision=bb43b2d9d79f
>
> Not sure yet on how to check that the patch is working but am working on it
> :)
I checked the logs and it looks like it is happily hitting the coalesce service.
Assignee | ||
Comment 5•9 years ago
|
||
:dividehex - this is great news, thanks!
I'll close the bug then. Will reopen if we need to tweak the treshold or bring any other changes to its config.
Thanks again for all the assistance here.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment 6•9 years ago
|
||
bugherder |
Updated•7 years ago
|
Product: TaskCluster → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•