Skip to content

500 internal error loading firefox profile data #851

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
jyn514 opened this issue Mar 18, 2021 · 5 comments · Fixed by #978
Closed

500 internal error loading firefox profile data #851

jyn514 opened this issue Mar 18, 2021 · 5 comments · Fixed by #978
Labels
A-api Issues related to the API exposed by the site C-bug A bug

Comments

@Mark-Simulacrum
Copy link
Member

Yeah, the self-profile summary views currently don't work; we need to migrate to the new profile format in the frontend. You can download the raw file though and that should work.

@jyn514
Copy link
Member Author

jyn514 commented Mar 18, 2021

the self-profile summary views currently don't work;

I'm confused - the summary view works fine:
image

It's just the 'processed-self-profile' page that doesn't work.

You can download the raw file though and that should work.

Thanks, that gave me an .mm_profdata file, but it doesn't track memory usage (rust-lang/rust#81348). Is there a way to see detailed memory usage other than with the 'max-rss' view? That only shows the total change, not what parts of the compiler changed.

@Mark-Simulacrum
Copy link
Member

right, yes, I meant the processed ones.

To my knowledge we don't track memory usage on any fine-grained level (I guess other than time-passes, but that has it's own problems) in the compiler today, including on a per-query level in self-profile.

@rylev rylev added C-bug A bug A-api Issues related to the API exposed by the site labels Jul 29, 2021
@tgnottingham
Copy link
Contributor

tgnottingham commented Aug 20, 2021

Fixed by #978. Technically doesn't support the old self-profile format, but that's probably not going to be an issue very often. If it becomes one, we can reopen.

@jyn514
Copy link
Member Author

jyn514 commented Aug 20, 2021

Works great, thank you!
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-api Issues related to the API exposed by the site C-bug A bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants