Skip to content
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

Flamegraph returned by SelectMergeStacktraces is not stable #2061

Closed
kolesnikovae opened this issue May 12, 2023 · 3 comments · Fixed by #2196
Closed

Flamegraph returned by SelectMergeStacktraces is not stable #2061

kolesnikovae opened this issue May 12, 2023 · 3 comments · Fixed by #2196
Assignees

Comments

@kolesnikovae
Copy link
Collaborator

kolesnikovae commented May 12, 2023

There are noticeable discrepancies in the flamegraphs built for the same query, even if it only covers immutable blocks. The difference may be in the stack trace structure, proportions, and numbers. Sometimes an entire call path may be missing, which is misleading. We need to verify the correctness of the query path.

Screen.Recording.2023-05-12.at.12.00.53.mov
@kolesnikovae kolesnikovae self-assigned this May 12, 2023
@kolesnikovae
Copy link
Collaborator Author

Might be related: #2067

@cyriltovena
Copy link
Contributor

cyriltovena commented Jun 8, 2023

@aocenas I think this is actually the javascript rendering doing this, it seems that some rounding might cause this. The total time at the top never changes.

EDIT: There's potential multiple bug, but I'd like to make sure this is fine on the frontend.

@aocenas
Copy link
Member

aocenas commented Jun 8, 2023

@cyriltovena hmm I remember something like this when we used the pprof format. At that point the response was unstable and I added a sort but still some things switched places. The reason was that some labels are the same but they had different binary/address so they weren't logically the same thing and were represented by 2 separate node next to each other but with the same label. If we sorted by label that would still be unstable. This seems like the same case but we don't have the sort in pyroscope data source anymore so I assume this could be an issue in the Pyroscope API or somewhere where this is sorted?

@simonswine simonswine transferred this issue from grafana/phlare Jul 19, 2023
@kolesnikovae kolesnikovae linked a pull request Aug 10, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants