Closed
Bug 579573
Opened 14 years ago
Closed 14 years ago
Stop sending talos sendchanges to talos masters for try
Categories
(Release Engineering :: General, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: lsblakk, Assigned: armenzg)
Details
(Whiteboard: [tryserver])
Attachments
(2 files, 1 obsolete file)
966 bytes,
patch
|
catlee
:
review+
lsblakk
:
checked-in+
|
Details | Diff | Splinter Review |
3.97 KB,
patch
|
catlee
:
review+
armenzg
:
checked-in+
|
Details | Diff | Splinter Review |
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)
Comment 1•14 years ago
|
||
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•14 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 3•14 years ago
|
||
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•14 years ago
|
||
good catch. now with 'False'.
Attachment #458009 -
Attachment is obsolete: true
Attachment #458448 -
Flags: review?(catlee)
Updated•14 years ago
|
Attachment #458448 -
Flags: review?(catlee) → review+
Reporter | ||
Comment 5•14 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•14 years ago
|
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Comment 6•14 years ago
|
||
This didn't do the trick, apparently. Try jobs are still sendchanging to all talos masters.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 7•14 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•14 years ago
|
Assignee: lsblakk → armenzg
Assignee | ||
Comment 8•14 years ago
|
||
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•14 years ago
|
Attachment #460513 -
Flags: review?(catlee) → review+
Assignee | ||
Comment 9•14 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•14 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
Closed: 14 years ago → 14 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•