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

x/telemetry/config: start collecting counters from delve #72129

Open
aarzilli opened this issue Mar 6, 2025 · 3 comments
Open

x/telemetry/config: start collecting counters from delve #72129

aarzilli opened this issue Mar 6, 2025 · 3 comments
Labels
telemetry x/telemetry issues Telemetry-Proposal Issues proposing new telemetry counters.
Milestone

Comments

@aarzilli
Copy link
Contributor

aarzilli commented Mar 6, 2025

Summary

Adds delve's counter to the telemetry configuration. There are two counters, one for crashes and one for panics we catch to avoid crashing.

This is a follow up to the approved proposal #68384

Proposed Config Change

https://go-review.googlesource.com/c/telemetry/+/655415

@aarzilli aarzilli added the Telemetry-Proposal Issues proposing new telemetry counters. label Mar 6, 2025
@gopherbot gopherbot added the telemetry x/telemetry issues label Mar 6, 2025
@gopherbot gopherbot added this to the Unreleased milestone Mar 6, 2025
@gopherbot
Copy link
Contributor

Change https://go.dev/cl/655415 mentions this issue: internal/chartconfig: collect counters for Delve

@aarzilli
Copy link
Contributor Author

aarzilli commented Mar 6, 2025

cc @golang/tools-team
cc @derekparker

@gabyhelp
Copy link

gabyhelp commented Mar 6, 2025

Related Issues

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
telemetry x/telemetry issues Telemetry-Proposal Issues proposing new telemetry counters.
Projects
None yet
Development

No branches or pull requests

3 participants