Closed Bug 729426 Opened 12 years ago Closed 12 years ago

Do periodic PGO on services-central

Categories

(Release Engineering :: General, defect, P2)

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: philor, Assigned: philor)

Details

Attachments

(1 file)

This time, we got lucky: rather than it being the never-tested-by-PGO merge from services-central, the PGO bustage on mozilla-central was from something else pushed directly there.

Next time? Probably won't be so lucky.

Anything which merges to mozilla-central needs to do periodic PGO (along with every other visible thing on m-c).
Attached patch fixSplinter Review
Sure hope someone's counting builders on their fingers, so they'll know when we're about to hit the limit again.
Attachment #599512 - Flags: review?(catlee)
Comment on attachment 599512 [details] [diff] [review]
fix

Oh, no wonder this hadn't taken effect yet - I made a bad choice of reviewer, and haven't even landed it yet.
Attachment #599512 - Flags: review?(catlee) → review?(jhford)
Comment on attachment 599512 [details] [diff] [review]
fix

Review of attachment 599512 [details] [diff] [review]:
-----------------------------------------------------------------

The patch is fine, but we need to be careful not to have periodic PGO on too many branches, otherwise we'll end up with too many slaves doing PGO stuff full time.
Attachment #599512 - Flags: review?(jhford) → review+
(In reply to John Ford [:jhford] from comment #3)
> The patch is fine, but we need to be careful not to have periodic PGO on too
> many branches, otherwise we'll end up with too many slaves doing PGO stuff
> full time.

Yeah, arguably I should have made them per-checkin, and I'll watch how they go and maybe switch them.
This went live this morning in the scheduled reconfiguration.
Won't actually be visible until the second reconfig writes the builder pickle file, though, so I can see they are running by seeing the tests running and then disappearing, but we won't see anything finished until one more.
Can't remember which was the second, might have been Tuesday morning.
Status: ASSIGNED → RESOLVED
Closed: 12 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

Created:
Updated:
Size: