reduce utilization of windows builders by tracemonkey builds

RESOLVED FIXED

Status

Release Engineering
General
P2
normal
RESOLVED FIXED
10 years ago
5 years ago

People

(Reporter: dbaron, Assigned: nthomas)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

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.
(Assignee)

Comment 1

10 years ago
Created attachment 341387 [details] [diff] [review]
[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.
Assignee: nobody → nthomas
Attachment #341387 - Flags: review?(bhearsum)
(Assignee)

Updated

10 years ago
Priority: -- → P2
Comment on attachment 341387 [details] [diff] [review]
[checked in] Disable PGO and make periodic schduler configurable by branch

cool!
Attachment #341387 - Flags: review?(bhearsum) → review+
(Assignee)

Comment 3

10 years ago
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.
(Assignee)

Comment 4

10 years ago
Created attachment 341561 [details] [diff] [review]
[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.
Attachment #341561 - Flags: review+
(Assignee)

Comment 5

10 years ago
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.
Attachment #341387 - Attachment description: Disable PGO and make periodic schduler configurable by branch → [checked in] Disable PGO and make periodic schduler configurable by branch
(Assignee)

Updated

10 years ago
Status: NEW → RESOLVED
Last Resolved: 10 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.