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

-tagroot and -tagleaf in UI aren't including the tag name? #811

Closed
NHDaly opened this issue Oct 20, 2023 · 4 comments
Closed

-tagroot and -tagleaf in UI aren't including the tag name? #811

NHDaly opened this issue Oct 20, 2023 · 4 comments

Comments

@NHDaly
Copy link
Contributor

NHDaly commented Oct 20, 2023

Please answer these questions before submitting your issue. Thanks!

What version of pprof are you using?

If you run pprof from GitHub, what's the Git revision?
Commit: 905365e

What operating system and processor architecture are you using?

MacOS M2

What did you do?

Running the following profile with -tagroot threadid is not showing the threadid: tag name in the UI:

pprof -tagroot threadid -http localhost:60004 profile.pb.gz

profile.pb.gz

What did you expect to see?

I expected to see frames with the names threadid:1 and threadid:2, similar to what I see in the screenshots from #649, which added this feature. For example, this one:

135353189-70b26230-b4e2-40a0-99df-403f8b5347bd

What did you see instead?

Instead, I just see frames with the values of the tags, i.e. 1 and 2:
Screenshot 2023-10-20 at 4 50 45 PM

And same for -tagleaf:
Screenshot 2023-10-20 at 4 46 09 PM

Was this changed on purpose at some point, or is this a bug? I think I very much preferred the view I expected.

@aalexand
Copy link
Collaborator

The tag name for these nodes is placed in the filename field when the profile with -tagroot or -tagleaf is generated. But I think the default granularity is -functions which discards the filename during the aggregation. Try passing -filefunctions flag.

@aalexand
Copy link
Collaborator

Closing for now, we can re-open if needed.

@aalexand aalexand closed this as not planned Won't fix, can't repro, duplicate, stale Oct 21, 2023
@NHDaly
Copy link
Contributor Author

NHDaly commented Oct 23, 2023

Thanks. Maybe we can figure it out with the right settings then, cool. 👍

pprof -filefunctions -tagroot threadid -http localhost:60004 profile.pb.gz

😅 That doesn't seem to be quite right either - this shows only the tag name, but not the tag value:
Screenshot 2023-10-22 at 8 53 56 PM
Screenshot 2023-10-22 at 8 55 08 PM

@NHDaly
Copy link
Contributor Author

NHDaly commented Apr 1, 2024

^ @aalexand: I just reread this thread, and I think this was not resolved. Could I ask you to reopen until we can figure out the right set of flags?

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

No branches or pull requests

2 participants