Closed
Bug 1358818
Opened 7 years ago
Closed 4 years ago
We shouldn't hit the symbolication server for non-Windows platforms
Categories
(Testing :: Talos, enhancement, P3)
Testing
Talos
Tracking
(firefox56 fixed)
RESOLVED
INCOMPLETE
mozilla56
Tracking | Status | |
---|---|---|
firefox56 | --- | fixed |
People
(Reporter: kmag, Unassigned)
References
Details
(Whiteboard: [PI:July])
Attachments
(1 file)
Currently, when running with profiling enabled, we try to fetch symbols from the symbolication server after every test cycle. We do this for every platform, even though we only actually try to fetch symbols for Windows. This needlessly slows things down on other platforms under the best of circumstances. But after a few test runs, I usually seem to wind up getting throttled, and waiting for many minutes after each cycle. At this point, I've wound up manually disabling the symbolication server in symbolicationRequest.py, since there doesn't seem to be a way to configure it.
Reporter | ||
Comment 2•7 years ago
|
||
My current workaround is to just break the symbol server by adding a line to my hosts file: 0.0.0.0 symbolapi.mozilla.org
Updated•7 years ago
|
Whiteboard: [PI:June]
Updated•7 years ago
|
Flags: needinfo?(ionut.goldan)
Updated•7 years ago
|
Assignee: nobody → ionut.goldan
Comment 4•7 years ago
|
||
:kmag Do you want symbols for the windows platform and that only, in all scenarios, and ignore symbols for linux and macosx? Or are you looking for a platform match? By this second option, I mean that if you test a build on platform X, you should only fetch symbols from that same platform X, not from Y and Z also.
Flags: needinfo?(kmaglione+bmo)
Reporter | ||
Comment 5•7 years ago
|
||
(In reply to Ionuț Goldan [:igoldan], Performance Sheriffing from comment #4) > :kmag Do you want symbols for the windows platform and that only, in all > scenarios, and ignore symbols for linux and macosx? I'd prefer that we only try to fetch symbols on Windows. We can usually get symbols from nm on other platforms, anyway, and the symbol server won't have symbols for local builds, so I think we're better off avoiding the network overhead and throttling issues.
Flags: needinfo?(kmaglione+bmo)
Comment hidden (mozreview-request) |
Comment 7•7 years ago
|
||
mozreview-review |
Comment on attachment 8882431 [details] Bug 1358818 - avoid symbolication server for non-Windows platforms https://reviewboard.mozilla.org/r/153504/#review158722
Attachment #8882431 -
Flags: review?(jmaher) → review+
Comment 8•7 years ago
|
||
this doesn't pass on try, but it could be a --geckoProfiler issue: https://treeherder.mozilla.org/#/jobs?repo=try&revision=5df6d23c51b97b2e3e64d0d8026fd20cf38bc2c1 can you prove this works locally? Also we should make it check for both windows and microsoft: https://dxr.mozilla.org/mozilla-central/source/testing/talos/talos/utils.py#29
Comment hidden (mozreview-request) |
Comment 10•7 years ago
|
||
thanks for the update- I guess validating we can get a profile locally would be good prior to landing.
Comment 11•7 years ago
|
||
The best thing for this would be to add some basic unittests. I'll work on that.
Comment hidden (mozreview-request) |
Updated•7 years ago
|
Whiteboard: [PI:June] → [PI:July]
Comment 13•7 years ago
|
||
I validated this locally, by placing logs on the relevant GeckoProfile code. Non-windows platforms aren't hitting the symbols server anymore. Can you merge this?
Flags: needinfo?(jmaher)
Updated•7 years ago
|
Flags: needinfo?(jmaher)
Comment 14•7 years ago
|
||
Pushed by jmaher@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8c1975262e1a avoid symbolication server for non-Windows platforms r=jmaher
Comment 15•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/8c1975262e1a
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox56:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
Comment 16•6 years ago
|
||
This patch was backed out in bug 1493261. However, in the time since this bug was filed, the symbolication server was rewritten; it should now respond a lot faster and not throttle you, so the bug as filed should hopefully no longer exist. Kris, can you check whether this issue still happens for you?
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Updated•5 years ago
|
Priority: -- → P3
Updated•5 years ago
|
Assignee: igoldan → nobody
Comment 18•4 years ago
|
||
No response from Kris for 2 years. So I assume this is not a problem anymore. Feel free to reopen if that is the case.
Status: REOPENED → RESOLVED
Closed: 7 years ago → 4 years ago
Flags: needinfo?(kmaglione+bmo)
Resolution: --- → WORKSFORME
Updated•4 years ago
|
Resolution: WORKSFORME → INCOMPLETE
You need to log in
before you can comment on or make changes to this bug.
Description
•