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: add collection for gopls/gotoolchain #68771

Closed
findleyr opened this issue Aug 7, 2024 · 4 comments
Closed

x/telemetry/config: add collection for gopls/gotoolchain #68771

findleyr opened this issue Aug 7, 2024 · 4 comments
Labels
NeedsFix The path to resolution is known, but the work has not been done. telemetry x/telemetry issues Telemetry-Accepted Telemetry-Proposal
Milestone

Comments

@findleyr
Copy link
Contributor

findleyr commented Aug 7, 2024

Summary

In #65917, we plan to use forward compatibility to ensure that gopls is installed with the latest Go toolchain.
Before doing so, we want to understand what fraction of users work in an environment where toolchain upgrades are supported. By collecting the gopls/gotoolchain counter, we can help inform this understanding.

Proposed Config Change

https://go.dev/cl/603857

@gopherbot gopherbot added the telemetry x/telemetry issues label Aug 7, 2024
@gopherbot gopherbot added this to the Unreleased milestone Aug 7, 2024
@gabyhelp
Copy link

gabyhelp commented Aug 7, 2024

@cagedmantis cagedmantis added NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. NeedsFix The path to resolution is known, but the work has not been done. and removed NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. labels Aug 12, 2024
@cagedmantis
Copy link
Contributor

cc @golang/telemetry

@findleyr
Copy link
Contributor Author

This was discussed in the telemetry meeting last week, with no objections. Since there have been no external objections, I'll accept and implement.

@gopherbot
Copy link
Contributor

Change https://go.dev/cl/603857 mentions this issue: internal/chartconfig: add collection for gopls/gotoolchain

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NeedsFix The path to resolution is known, but the work has not been done. telemetry x/telemetry issues Telemetry-Accepted Telemetry-Proposal
Projects
Status: No status
Development

No branches or pull requests

4 participants