Stop sending talos sendchanges to talos masters for try

RESOLVED FIXED

Status

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

People

(Reporter: lsblakk, Assigned: armenzg)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [tryserver])

Attachments

(2 attachments, 1 obsolete attachment)

Created attachment 458009 [details] [diff] [review]
turn off talos sendchanges to test-masters

Instead of running talos by default on try we can save resources by turning it off and on the occasion that someone needs a talos run they can request the buildduty sheriff from RelEng do the sendchange to the actual masters. Will keep a dummy localhost in the talos-masters list so that all the info needed for a sendchange will be in the build's log/steps.
Attachment #458009 - Flags: review?(catlee)
We should have a checkbox on the tryserver webpage that allows users to do this without having to figure out who the buildduty sheriff is and running them down. Even if that just results in a bug being filed, it's a better user experience.
(Reporter)

Comment 2

8 years ago
(In reply to comment #1)
> We should have a checkbox on the tryserver webpage 

There is not currently a tryserver webpage so while this would be a better user experience it's not possible with the reality of push to try. It's unfortunate that someone who wants Talos on their try run needs to come find one of us but we truly believe that very few of the daily pushes to try will be resulting in this type of request.
Comment on attachment 458009 [details] [diff] [review]
turn off talos sendchanges to test-masters

I think I'd rather have this not warn if the sendchange fails. (i.e. have it set to [('localhost:9011', False)].
Attachment #458009 - Flags: review?(catlee) → review-
(Reporter)

Comment 4

8 years ago
Created attachment 458448 [details] [diff] [review]
turn off talos sendchanges to test-masters

good catch.  now with 'False'.
Attachment #458009 - Attachment is obsolete: true
Attachment #458448 - Flags: review?(catlee)

Updated

8 years ago
Attachment #458448 - Flags: review?(catlee) → review+
(Reporter)

Comment 5

8 years ago
Comment on attachment 458448 [details] [diff] [review]
turn off talos sendchanges to test-masters

http://hg.mozilla.org/build/buildbot-configs/rev/577e8b8950e0

Reconfig'd try master at 14:52 PDT. Closing.
Attachment #458448 - Flags: checked-in+
(Reporter)

Updated

8 years ago
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
This didn't do the trick, apparently. Try jobs are still sendchanging to all talos masters.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 7

8 years ago
I believe it should be something like this (per platform instead of per branch):
BRANCHES['tryserver']['platforms']['macosx64-debug']['talos_masters'] = [('localhost:9011', False)]
(Assignee)

Updated

8 years ago
Assignee: lsblakk → armenzg
(Assignee)

Comment 8

8 years ago
Created attachment 460513 [details] [diff] [review]
stop sending try talos jobs

This worked on staging.
Let me know if you want mozilla/staging_config.py to match mozilla/production_config.py.
Attachment #460513 - Flags: review?(catlee)

Updated

8 years ago
Attachment #460513 - Flags: review?(catlee) → review+
(Assignee)

Comment 9

8 years ago
Comment on attachment 460513 [details] [diff] [review]
stop sending try talos jobs

http://hg.mozilla.org/build/buildbot-configs/rev/905b9b693254

pm02:8011 has been reconfigured.
Attachment #460513 - Flags: checked-in+
(Assignee)

Comment 10

8 years ago
We are now down from 11 to 5 sendchanges and all 5 are only for unit tests. We are done in here.
Status: REOPENED → RESOLVED
Last Resolved: 8 years ago8 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.