Closed Bug 1358818 Opened 4 years ago Closed 9 months ago

We shouldn't hit the symbolication server for non-Windows platforms


(Testing :: Talos, enhancement, P3)



(firefox56 fixed)

Tracking Status
firefox56 --- fixed


(Reporter: kmag, Unassigned)



(Whiteboard: [PI:July])


(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, since there doesn't seem to be a way to configure it.
Duplicate of this bug: 1331807
My current workaround is to just break the symbol server by adding a line to my hosts file:
Whiteboard: [PI:June]
Flags: needinfo?(ionut.goldan)
I'm looking into it now.
Flags: needinfo?(ionut.goldan)
Assignee: nobody → ionut.goldan
: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)
(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 on attachment 8882431 [details]
Bug 1358818 - avoid symbolication server for non-Windows platforms
Attachment #8882431 - Flags: review?(jmaher) → review+
this doesn't pass on try, but it could be a --geckoProfiler issue:

can you prove this works locally?

Also we should make it check for both windows and microsoft:
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.
Whiteboard: [PI:June] → [PI:July]
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)
Flags: needinfo?(jmaher)
Pushed by
avoid symbolication server for non-Windows platforms r=jmaher
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
Depends on: 1493261
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?
Resolution: FIXED → ---
ni?ing :kmag, based on comment 16.
Flags: needinfo?(kmaglione+bmo)
Priority: -- → P3
Assignee: igoldan → nobody

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.

Closed: 4 years ago9 months ago
Flags: needinfo?(kmaglione+bmo)
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.