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.

Sign up for GitHub

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

Jump to bottom

x/telemetry: improve readability of merged reports #63848

Open
findleyr opened this issue Oct 31, 2023 · 2 comments
Open

x/telemetry: improve readability of merged reports #63848

findleyr opened this issue Oct 31, 2023 · 2 comments
Labels
NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. telemetry x/telemetry issues
Milestone
Unreleased

Comments

@findleyr
Copy link
Contributor

On https://telemetry.go.dev, we're starting to get more data in the merged reports, which is great. However, as reports get larger (e.g. https://storage.googleapis.com/prod-telemetry-merged/2023-10-28.json), it's getting harder to identify individual stacks.

We should consider ways to improve the readability of these merged reports. Perhaps we can have an html handler that summarizes them and collapses stacks. For non-devel versions, perhaps we can map the stack to a cs.opensource.google link...
@gopherbot gopherbot added the telemetry x/telemetry issues label Oct 31, 2023
@gopherbot gopherbot added this to the Unreleased milestone Oct 31, 2023
@cherrymui cherrymui added the NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. label Oct 31, 2023
@hyangah
Copy link
Contributor

hyangah commented Nov 1, 2023

Can we consider client-side solutions?

curl https://storage.googleapis.com/prod-telemetry-merged/2023-10-28.json | jq

@findleyr
Copy link
Contributor Author

findleyr commented Nov 1, 2023

@hyangah nice. I'll use that for now :)

Yes, client-side solutions can work, though maybe it wouldn't be too hard to do the same in an html handler (e.g. parse, reformat the JSON).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. telemetry x/telemetry issues
Projects
None yet
Milestone
Unreleased
Development

No branches or pull requests
4 participants
@hyangah @gopherbot @cherrymui @findleyr