talos-r3 should enable multiple masters, splittable by platform

RESOLVED FIXED

Status

Release Engineering
General
RESOLVED FIXED
8 years ago
4 years ago

People

(Reporter: bhearsum, Assigned: bhearsum)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [buildmasters][talos][unittest])

Attachments

(3 attachments, 1 obsolete attachment)

(Assignee)

Description

8 years ago
Just like we have the ability to have multiple masters in mozilla2/ and mozilla2-staging/, we need the ability to split up the test work between multiple test masters. For these masters, splitting by platform will be best for now to avoid affecting wait times.

When we move the test masters to 0.8.0 we should move back to everything on one master, or work split by branch.
(Assignee)

Comment 1

8 years ago
Created attachment 451992 [details] [diff] [review]
enable talos splitting by platform

This allows us to split up the talos work by platform (and branch, if we want to). I think it's pretty straightforward if you apply it and look at master*. I've added sendchanges for the new master, which will be ready to come up before this lands.

I gave this a run through in staging. I let a couple builds per platform run, all of which passed.
Attachment #451992 - Flags: review?(catlee)
(Assignee)

Comment 2

8 years ago
Created attachment 451993 [details] [diff] [review]
add sendchanges for 0.8.0 masters
Attachment #451993 - Flags: review?(catlee)
(Assignee)

Comment 3

8 years ago
Created attachment 451994 [details] [diff] [review]
talos split up, don't send staging to production

Sorry, just realized I stupidly had staging sendchanges go to production.
Attachment #451992 - Attachment is obsolete: true
Attachment #451994 - Flags: review?(catlee)
Attachment #451992 - Flags: review?(catlee)
(Assignee)

Updated

8 years ago
Whiteboard: [buildmasters][talos][unittest]

Updated

8 years ago
Attachment #451993 - Flags: review?(catlee) → review+
Comment on attachment 451994 [details] [diff] [review]
talos split up, don't send staging to production

The ACTIVE_PLATFORMS stuff is a bit awkward, but I'm at a loss for a cleaner alternative.
Attachment #451994 - Flags: review?(catlee) → review+
(Assignee)

Comment 5

7 years ago
Comment on attachment 451994 [details] [diff] [review]
talos split up, don't send staging to production

changeset:   2547:4423461117ca
Attachment #451994 - Flags: checked-in+
(Assignee)

Comment 6

7 years ago
Comment on attachment 451993 [details] [diff] [review]
add sendchanges for 0.8.0 masters

changeset:   2549:0795d54e9539
Attachment #451993 - Flags: checked-in+
(Assignee)

Comment 7

7 years ago
This landed, seems to be working fine.
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
(Assignee)

Comment 8

7 years ago
Created attachment 453386 [details] [diff] [review]
staging patch
Attachment #453386 - Flags: review?(catlee)

Updated

7 years ago
Attachment #453386 - Flags: review?(catlee) → review+
(Assignee)

Comment 9

7 years ago
Comment on attachment 453386 [details] [diff] [review]
staging patch

changeset:   2589:dd99dd06b222
Attachment #453386 - Flags: checked-in+
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.