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

Adding Telemetry #3815

Open
derekparker opened this issue Sep 23, 2024 · 0 comments
Open

Adding Telemetry #3815

derekparker opened this issue Sep 23, 2024 · 0 comments

Comments

@derekparker
Copy link
Member

The Delve team together with the Go team have proposed that Go telemetry metrics collection be added to Delve[0]. The gathering of this data would happen in accordance with the upstream guidelines for telemetry data collection[1]. Just as with telemetry in the Go toolchain, data collection will happen locally and exporting of this data can be controlled via go telemetry.

We are proposing to add metrics around the following data points:

  • Usage by GOOS/GOARCH, Go/Delve versions
  • Usage by Delve subcommands, DAP mode, RPC versions, flags
  • Performance metrics (latency)
  • Crash monitoring
  • Internal errors
  • Missing features in the expression language
  • Potential UI improvements for IDE’s

This information will be useful to the Delve maintainers to help guide development efforts, fix unreported bugs, and continue to ensure the overall user experience and functionality is in alignment with what our users expect.

This issue will track the ongoing work and serve as a place for community feedback.


[0]: golang/go#68384
[1]: https://go.dev/doc/telemetry#background

hyangah added a commit to hyangah/delve that referenced this issue Oct 25, 2024
One side effect of it is the go version requirement
change. Now it requires go1.22.0 (due to golang.org/x/ modules
requirements).

For go-delve#3815
hyangah added a commit to hyangah/delve that referenced this issue Oct 25, 2024
One side effect of it is the go version requirement
change. Now it requires go1.22.0 (due to golang.org/x/ modules
requirements).

For go-delve#3815
hyangah added a commit to hyangah/delve that referenced this issue Oct 25, 2024
Imported golang.org/x/telemetry@a0ccdf0a to avoid
go version requirement change.

For go-delve#3815
hyangah added a commit to hyangah/delve that referenced this issue Oct 28, 2024
Imported golang.org/x/telemetry@9c0d19e to avoid
go version requirement change.

For go-delve#3815
For golang/go#70056
derekparker pushed a commit that referenced this issue Oct 29, 2024
Imported golang.org/x/telemetry@9c0d19e to avoid
go version requirement change.

For #3815
For golang/go#70056
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

1 participant