Closed Bug 1358818 Opened 4 years ago Closed 9 months ago
We shouldn't hit the symbolication server for non-Windows platforms
59 bytes, text/x-review-board-request
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.
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
I'm looking into it now.
: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.
(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.
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+
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
thanks for the update- I guess validating we can get a profile locally would be good prior to landing.
The best thing for this would be to add some basic unittests. I'll work on that.
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?
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/8c1975262e1a avoid symbolication server for non-Windows platforms r=jmaher
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 → ---
ni?ing :kmag, based on comment 16.
Status: REOPENED → RESOLVED
Closed: 4 years ago → 9 months ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.