458148 - reduce utilization of windows builders by tracemonkey builds

Status: RESOLVED FIXED

(Release Engineering :: General, defect, P2)

Description

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

I just had a quick discussion with Andreas about the 5 windows builders (slaves 1-3, 5-6) currently shared between mozilla-central (build and leak build) and tracemonkey build.  We should:

(1) change the tracemonkey tinderbox's config to remove the automatic triggering of builds every 2 hours.  There's no need for that, since there's no talos, and thus no need to generate extra builds to work around talos's inability to test the same build multiple times (and our inability to graph that, etc.)

(2) change the windows builds on the tracemonkey tinderbox from being PGO builds to being non-PGO depend builds.  This will make them much faster, which Andreas says is better for what that tinderbox is used for.

Comment 1

[Nick Thomas [:nthomas] (UTC+12)]

Attachment: [checked in] Disable PGO and make periodic schduler configurable by branch

This should let us set a configurable value for the idle period - I went with 10 hours for tracemonkey so that we keep the builder on the waterfall. We can set it to 0 if we really don't care about them dropping off some of the time. Also disables PGO for win32 tracemonkey.

Comment 2

[[e-mail] (:bhearsum)]

Comment on attachment 341387 [details] [diff] [review]
[checked in] Disable PGO and make periodic schduler configurable by branch

cool!

Comment 3

[Nick Thomas [:nthomas] (UTC+12)]

I just pushed the disabling of PGO on win32, rev c14fa0b0ab88, and will go round and clobber the build dirs to be safe.

By the by, we seem to set optimization as a default build option, so it doesn't matter that we do --enable-optimize on in the Linux and Mac mozconfigs and not Windows. Would be nice to iron out that inconsistency though.

Comment 4

[Nick Thomas [:nthomas] (UTC+12)]

Attachment: [checked in] Matching changes for staging

Almost identical patch, carrying review. rev:78d58e499011 for the push.

Going to get this going on staging to catch any bugs before rolling out to the production master.

Comment 5

[Nick Thomas [:nthomas] (UTC+12)]

Comment on attachment 341387 [details] [diff] [review]
[checked in] Disable PGO and make periodic schduler configurable by branch

Landed the scheduler part of this, rev: fba0ede20c61, and reconfig'd the master.