Closed
Bug 1270045
Opened 9 years ago
Closed 9 years ago
Stop posting ctors data to graphs.m.o
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: nthomas, Assigned: nthomas)
References
Details
Attachments
(1 file)
3.21 KB,
patch
|
bhearsum
:
review+
nthomas
:
checked-in+
|
Details | Diff | Splinter Review |
We're doing this from these builds:
Linux mozilla-esr38 build NightlyBuildFactory
Linux mozilla-esr45 build NightlyBuildFactory
Linux x86-64 mozilla-esr38 build NightlyBuildFactory
Linux x86-64 mozilla-esr45 build NightlyBuildFactory
TB Linux comm-aurora build NightlyBuildFactory
TB Linux comm-beta build NightlyBuildFactory
TB Linux comm-central build NightlyBuildFactory
TB Linux comm-esr38 build NightlyBuildFactory
TB Linux comm-esr45 build NightlyBuildFactory
TB Linux cypress build NightlyBuildFactory
TB Linux x86-64 comm-aurora build NightlyBuildFactory
TB Linux x86-64 comm-beta build NightlyBuildFactory
TB Linux x86-64 comm-central build NightlyBuildFactory
TB Linux x86-64 comm-esr38 build NightlyBuildFactory
TB Linux x86-64 comm-esr45 build NightlyBuildFactory
TB Linux x86-64 cypress build NightlyBuildFactory
The code in question is http://hg.mozilla.org/build/buildbotcustom/file/default/process/factory.py#l1410, so just unsetting graph_server will mean we still collect the data in the build log, just not submit to graph.m.o.
This will also fix bug 1262760.
Assignee | ||
Comment 1•9 years ago
|
||
FWIW, in mozharness we collect ctors info and it makes it to perfherder, eg https://treeherder.mozilla.org/perf.html#/graphs?timerange=2592000&series=%5Bmozilla-central,45b561e2236d7631af037d960a78410227317660,1%5D
No point porting that to ESR38/45, and TB is stuck on buildbot. ctors are the least of their worries.
Attachment #8748552 -
Flags: review?(bhearsum)
Comment 2•9 years ago
|
||
:nthomas, I had thought this was resolved with the work I did here:
https://reviewboard.mozilla.org/r/36903/
I guess that doesn't apply to nightlies and esr. We don't have a need to collect ctors from esr, so this seems reasonable from the firefox side.
Assignee | ||
Comment 3•9 years ago
|
||
Thanks for the pointer, now I know how mozharness got fixed. This patch just affects the builds still using a Buildbot factory (NightlyBuildFactory) - boo code divergence. They're not actually nightlies, I just didn't trim the factory name off each of those lines in the initial comment.
Updated•9 years ago
|
Attachment #8748552 -
Flags: review?(bhearsum) → review+
Assignee | ||
Comment 4•9 years ago
|
||
Comment on attachment 8748552 [details] [diff] [review]
[buildbot-configs] Stop posting
https://hg.mozilla.org/build/buildbot-configs/rev/befb16732497
https://hg.mozilla.org/build/buildbot-configs/rev/2eefce8cc6de
Attachment #8748552 -
Flags: checked-in+
Assignee | ||
Comment 5•9 years ago
|
||
Did some spot checks on ESR and the graph post are gone.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
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
•