Closed Bug 1317758 Opened 3 years ago Closed 3 years ago

Remove linux64-cc and linux64-tsan from buildbot-configs

Categories

(Release Engineering :: General, defect)

defect
Not set

Tracking

(firefox53 fixed)

RESOLVED FIXED
Tracking Status
firefox53 --- fixed

People

(Reporter: RyanVM, Assigned: RyanVM)

References

Details

Attachments

(4 files)

They're only available on Try and nobody's actually using them. If/when someone wants jobs like these, they should use Taskcluster anyway. I pinged Chris on IRC and he agrees that they can go.
Per ahal and jmaher on IRC, linux64-cc can go too. The UCOSP folks are doing code coverage work via different Taskcluster jobs nowadays.
Summary: Remove linux64-tsan from buildbot-configs → Remove linux64-cc and linux64-tsan from buildbot-configs
Unfortunately, list_builder_differences.sh is giving me conniptions playing nicely with the tools repo changes (I appear to have stumped Armen too :P). Getting these up for review for now, but I'll hold off on pushing until they've had a chance to go through proper testing.
Attachment #8811016 - Flags: review?(bugspam.Callek)
There's a couple in-tree mozharness references too that I'll be sure to remove.
Attached file builder diff
After some unspeakable hacks, I finally managed to get a functional builder diff. As a bonus, it even looks sane!
I'm purposely leaving testing/mozharness/configs/builds/releng_sub_linux_configs/64_tsan.py in the tree because it looks like it could end up being useful whenever someone decides to stand up Taskcluster-based Linux TSAN jobs.
Attachment #8811101 - Flags: review?(bugspam.Callek)
Blocks: 1276377
Attachment #8811013 - Flags: review?(bugspam.Callek) → review+
Attachment #8811016 - Flags: review?(bugspam.Callek) → review+
(In reply to Ryan VanderMeulen [:RyanVM] from comment #3)
> Created attachment 8811016 [details] [diff] [review]
> (tools) remove linux64-cc and linux64-tsan
> 
> Unfortunately, list_builder_differences.sh is giving me conniptions playing
> nicely with the tools repo changes (I appear to have stumped Armen too :P).
> Getting these up for review for now, but I'll hold off on pushing until
> they've had a chance to go through proper testing.

AIUI you do need to push this at the same time (or just before) buildbot-configs otherwise things break...

See https://dxr.mozilla.org/build-central/search?q=MASTERS_JSON_URL&redirect=false for more info on why this failed :-)
Comment on attachment 8811101 [details] [diff] [review]
remove in-tree references to linux64-cc and linux64-tsan

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

Theres also https://dxr.mozilla.org/mozilla-central/source/testing/mozharness/configs/builds/releng_sub_linux_configs/64_tsan.py too :-)
Attachment #8811101 - Flags: review?(bugspam.Callek) → review+
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c68af995d66d
Remove in-tree references to the linux64-cc and linux64-tsan jobs. r=Callek
https://hg.mozilla.org/mozilla-central/rev/c68af995d66d
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.