Closed
Bug 1317758
Opened 8 years ago
Closed 8 years ago
Remove linux64-cc and linux64-tsan from buildbot-configs
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(firefox53 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox53 | --- | fixed |
People
(Reporter: RyanVM, Assigned: RyanVM)
References
Details
Attachments
(4 files)
36.12 KB,
patch
|
Callek
:
review+
RyanVM
:
checked-in+
|
Details | Diff | Splinter Review |
20.22 KB,
patch
|
Callek
:
review+
RyanVM
:
checked-in+
|
Details | Diff | Splinter Review |
9.12 KB,
text/plain
|
Details | |
5.07 KB,
patch
|
Callek
:
review+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 1•8 years ago
|
||
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
Assignee | ||
Comment 2•8 years ago
|
||
Attachment #8811013 -
Flags: review?(bugspam.Callek)
Assignee | ||
Comment 3•8 years ago
|
||
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)
Assignee | ||
Comment 4•8 years ago
|
||
There's a couple in-tree mozharness references too that I'll be sure to remove.
Assignee | ||
Comment 5•8 years ago
|
||
After some unspeakable hacks, I finally managed to get a functional builder diff. As a bonus, it even looks sane!
Assignee | ||
Comment 6•8 years ago
|
||
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)
Updated•8 years ago
|
Attachment #8811013 -
Flags: review?(bugspam.Callek) → review+
Updated•8 years ago
|
Attachment #8811016 -
Flags: review?(bugspam.Callek) → review+
Comment 7•8 years ago
|
||
(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 8•8 years ago
|
||
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+
Assignee | ||
Comment 9•8 years ago
|
||
Comment on attachment 8811016 [details] [diff] [review] (tools) remove linux64-cc and linux64-tsan https://hg.mozilla.org/build/tools/rev/7b508a245906b1b205bfbccdd7019ea0ff3569b9
Attachment #8811016 -
Flags: checked-in+
Assignee | ||
Comment 10•8 years ago
|
||
Comment on attachment 8811013 [details] [diff] [review] (buildbot-configs) remove linux64-cc and linux64-tsan https://hg.mozilla.org/build/buildbot-configs/rev/a28fcb0b762511311a748d47df571c70435bd9e1
Attachment #8811013 -
Flags: checked-in+
Comment 11•8 years ago
|
||
In production: https://hg.mozilla.org/build/buildbot-configs/rev/a28fcb0b7625
Comment 12•8 years ago
|
||
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
Comment 13•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/c68af995d66d
Updated•6 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•