Closed
Bug 1317758
Opened 9 years ago
Closed 9 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•9 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•9 years ago
|
||
Attachment #8811013 -
Flags: review?(bugspam.Callek)
| Assignee | ||
Comment 3•9 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•9 years ago
|
||
There's a couple in-tree mozharness references too that I'll be sure to remove.
| Assignee | ||
Comment 5•9 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•9 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•9 years ago
|
Attachment #8811013 -
Flags: review?(bugspam.Callek) → review+
Updated•9 years ago
|
Attachment #8811016 -
Flags: review?(bugspam.Callek) → review+
Comment 7•9 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•9 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•9 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•9 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•9 years ago
|
||
Comment 12•9 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•9 years ago
|
||
| bugherder | ||
Updated•7 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•