Closed
Bug 976104
Opened 10 years ago
Closed 10 years ago
raise slaverebooter's parallelization
Categories
(Release Engineering :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: bhearsum, Assigned: bhearsum)
Details
Attachments
(3 files)
721 bytes,
patch
|
jhopkins
:
review+
bhearsum
:
checked-in+
|
Details | Diff | Splinter Review |
1017 bytes,
patch
|
jhopkins
:
review+
bhearsum
:
checked-in+
|
Details | Diff | Splinter Review |
475 bytes,
patch
|
jhopkins
:
review+
bhearsum
:
checked-in+
|
Details | Diff | Splinter Review |
Since fixing a bug in slaverebooter where we weren't actually waiting for graceful shutdowns, slaverebooter takes a lot longer to run. One way to improve this is to make it parallelize more. I had to reduce it to 4 threads when it initially landed because of issues with buildapi getting overloaded. But now that we have a better buildapi, I think we can increase it more. I'm thinking 8 threads for now, and maybe more if things continue to look good. I plan to land this early in the morning one day so that I can keep a close eye on it.
Attachment #8380696 -
Flags: review?(jhopkins)
Assignee | ||
Comment 1•10 years ago
|
||
Attachment #8380698 -
Flags: review?(jhopkins)
Updated•10 years ago
|
Attachment #8380696 -
Flags: review?(jhopkins) → review+
Updated•10 years ago
|
Attachment #8380698 -
Flags: review?(jhopkins) → review+
Assignee | ||
Updated•10 years ago
|
Attachment #8380696 -
Flags: checked-in+
Assignee | ||
Updated•10 years ago
|
Attachment #8380698 -
Flags: checked-in+
Assignee | ||
Comment 2•10 years ago
|
||
8 seems to be working fine without breaking buildapi, let's try even more :).
Assignee: nobody → bhearsum
Attachment #8382219 -
Flags: review?(jhopkins)
Comment 3•10 years ago
|
||
Comment on attachment 8382219 [details] [diff] [review] bump to 12 sure, what could go wrong ;)
Attachment #8382219 -
Flags: review?(jhopkins) → review+
Assignee | ||
Updated•10 years ago
|
Attachment #8382219 -
Flags: checked-in+
Assignee | ||
Comment 4•10 years ago
|
||
Even at 12 workers things are going fine. I think this is good enough for now - slaverebooter runs much more quickly than before.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•7 years ago
|
Component: Tools → General
You need to log in
before you can comment on or make changes to this bug.
Description
•