550876 - Backport eventHorizon, changeHorizon code to buildbot 0.7.10

Status: RESOLVED FIXED

(Release Engineering :: General, defect, P2)

Description

[Chris AtLee [:catlee]]

Attachment: Limit the number of events to 50 per builder

One major reason the current version of buildbot is so slow is that we accumlate thousands of events in the builders.  Every time a slave disconnects and reconnects, 2 events are logged in each builder that slave is attached to.  These events are stored in memory, and must be saved to and loaded from disk on a reconfig.  There's nothing in buildbot 0.7.10 to get rid of them, so they persist until the end of time.

The code to limit the # of events is pretty simple, and applies cleanly to our code base.  I've attached the patch against BUILDBOT_PRODUCTION.  This only needs deploying on the masters.

Comment 1

[Chris AtLee [:catlee]]

Attachment: Backport changeHorizon too

Comment 2

[Chris AtLee [:catlee]]

Attachment: Set changeHorizon to 100 on selected masters

Comment 3

[Chris AtLee [:catlee]]

Comment on attachment 431097 [details] [diff] [review]
Limit the number of events to 50 per builder

changeset:   63:e3f8db2dd3f9

Comment 4

[Chris AtLee [:catlee]]

Comment on attachment 431275 [details] [diff] [review]
Backport changeHorizon too

changeset:   64:002370ef5a45

Comment 5

[Chris AtLee [:catlee]]

Comment on attachment 431276 [details] [diff] [review]
Set changeHorizon to 100 on selected masters

changeset:   2146:fe9a5aa5837f