Closed Bug 554341 Opened 16 years ago Closed 16 years ago

Backport slave buffering to buildbot 0.7.10

Categories

(Release Engineering :: General, defect, P3)

Product:
Release Engineering
Component:
General
Platform:
x86
Linux
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: catlee, Assigned: catlee)

Details

Attachments

(3 files, 2 obsolete files)

Backport of buffering code from upstream revision e0d019555a1fd1fe7779299e3a55b9a60d5a10bb
18.60 KB, patch
bhearsum
: review+
catlee
: checked-in+
Details | Diff | Splinter Review
Puppet manifests to update buildbot on linux/mac to new revision
2.44 KB, patch
bhearsum
: review+
catlee
: checked-in+
Details | Diff | Splinter Review
Changes to opsi buildbot package to make sure the old version gets removed
1.50 KB, patch
bhearsum
: review+
catlee
: checked-in+
Details | Diff | Splinter Review
We suspect that handling log output is a significant load on the buildbot masters. Adding slave-side buffering mitigates this somewhat. It might be useful to backport the 0.8.0 changes to 0.7.10
Attachment #434250 - Flags: review?(bhearsum)
Comment on attachment 434250 [details] [diff] [review] Backport of buffering code from upstream revision e0d019555a1fd1fe7779299e3a55b9a60d5a10bb rubberstamp+
Attachment #434250 - Flags: review?(bhearsum) → review+
Turns out that twisted 2.4/2.5 don't have reactor.seconds, which breaks the other patch on linux.
Attachment #434250 - Attachment is obsolete: true
Attachment #434557 - Flags: review?(bhearsum)
Comment on attachment 434557 [details] [diff] [review] Backport of buffering code from upstream revision e0d019555a1fd1fe7779299e3a55b9a60d5a10bb stamp
Attachment #434557 - Flags: review?(bhearsum) → review+
Attachment #434867 - Flags: review?(bhearsum)
Comment on attachment 434557 [details] [diff] [review] Backport of buffering code from upstream revision e0d019555a1fd1fe7779299e3a55b9a60d5a10bb changeset: 65:38f8fbae09bb
Attachment #434557 - Flags: checked-in+
Attachment #434867 - Attachment is obsolete: true
Attachment #434880 - Flags: review?(bhearsum)
Attachment #434867 - Flags: review?(bhearsum)
Attachment #434880 - Flags: review?(bhearsum) → review+
Attachment #434922 - Flags: review?(bhearsum) → review+
Comment on attachment 434922 [details] [diff] [review] Changes to opsi buildbot package to make sure the old version gets removed changeset: 42:c1bf48d9ec6d
Attachment #434922 - Flags: checked-in+
Comment on attachment 434880 [details] [diff] [review] Puppet manifests to update buildbot on linux/mac to new revision changeset: 117:cbce0ea22072
Attachment #434880 - Flags: checked-in+
Assigning to catlee based on patches-in-hand.
Assignee: nobody → catlee
Priority: -- → P3
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: