Closed
Bug 1289277
Opened 8 years ago
Closed 8 years ago
Reduce amount of buildbot-bridge hosts
Categories
(Release Engineering :: Release Automation: Other, defect)
Release Engineering
Release Automation: Other
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: rail, Assigned: rail)
References
Details
(Whiteboard: [bbb])
Attachments
(1 file)
58 bytes,
text/x-review-board-request
|
Callek
:
review+
rail
:
checked-in+
|
Details |
As a work around for race conditions we can try to reduce the amount of BBB hosts from 3 to 2 or even 1.
Assignee | ||
Comment 1•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/67200/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/67200/
Attachment #8774754 -
Flags: review?(bugspam.Callek)
Assignee | ||
Comment 2•8 years ago
|
||
The patch won't be deleting anything important, only the supervisor files (and maybe some other configs). This will stop the processes on those masters.
Comment 3•8 years ago
|
||
Comment on attachment 8774754 [details] Bug 1289277 - Reduce amount of buildbot-bridge hosts https://reviewboard.mozilla.org/r/67200/#review64026 I'm not really convinced this will help us that much. We have seen some relatively rare race conditions of things running twice, and this can help pinpoint that a bit, while absent other logging. Our underlying issue is lack of good [useful] logging and buildbot RETRY's reusing buildrequestid.
Attachment #8774754 -
Flags: review?(bugspam.Callek) → review+
Updated•8 years ago
|
Blocks: bbb-improvements
Updated•8 years ago
|
Whiteboard: [bbb]
Assignee | ||
Comment 4•8 years ago
|
||
I'm going to land this. We talked with catlee about dupes (bug 1288415) and the idea to add an extra check in handlePending before we add it to the DB, but it requires to be serial, so we have to run a single instance in this case.
Assignee | ||
Comment 5•8 years ago
|
||
Comment on attachment 8774754 [details] Bug 1289277 - Reduce amount of buildbot-bridge hosts remote: https://hg.mozilla.org/build/puppet/rev/f074712cc9bd remote: https://hg.mozilla.org/build/puppet/rev/8d7feedb3722
Attachment #8774754 -
Flags: checked-in+
Assignee | ||
Updated•8 years ago
|
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 6•8 years ago
|
||
It looks like 1 instance is not enough to process all incoming buildbot events. We had a lot of tasks cancelled by the reflector because of this.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 7•8 years ago
|
||
Let's wontfix this...
Status: REOPENED → RESOLVED
Closed: 8 years ago → 8 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•