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

cmd/go: revendor x/telemetry prior to go1.23.0 #67617

Closed
findleyr opened this issue May 23, 2024 · 11 comments
Closed

cmd/go: revendor x/telemetry prior to go1.23.0 #67617

findleyr opened this issue May 23, 2024 · 11 comments
Assignees
Milestone

Comments

@findleyr
Copy link
Member

This is a reminder issue to re-vendor x/telemetry with bug-fixes prior to go1.23rc1.

We are effectively treating x/telemetry as frozen for the moment, and want to make sure any bugfixes are included in the Go release.

@findleyr findleyr added this to the Go1.23 milestone May 23, 2024
@findleyr findleyr self-assigned this May 23, 2024
@dmitshur
Copy link
Contributor

Note that #36905 tracks re-vendoring all dependencies in the main Go repo, which will happen soon now that we're in the freeze. This issue can be kept open after that happens if there are more x/telemetry-specific changes to bring in afterwards.

@dmitshur
Copy link
Contributor

dmitshur commented Jun 4, 2024

FYI CL 589935 updated x/telemetry to v0.0.0-20240603224550-f2b69109f79b. Feel free to close this if that's enough for the needs of RC 1, or keep it open if you'd like to check in again, closer to RC 1 target date, in case more bug-fixes land in x/telemetry by then.

@findleyr
Copy link
Member Author

findleyr commented Jun 4, 2024

Thanks. I plan to revendor again on Friday, and close this issue at that time.

@dmitshur dmitshur moved this to In Progress in Release Blockers Jun 12, 2024
@findleyr
Copy link
Member Author

We still have a couple fixes pending, notably https://go.dev/cl/592017, which we want to land before revendoring.

@dr2chase
Copy link
Contributor

This is marked as a release blocker, is open, is not marked okay-after-RC1, and the scheduled cut for 1.23RC1 is roughly 24 hours from now. Where do we stand on this?

@findleyr
Copy link
Member Author

I think we're good now. It looks like Michael re-vendored in https://go.dev/cl/592255.

I'm going to move this to ok-after-rc1. I suspect we may have additional bug fixes after the RC.

@findleyr findleyr added the okay-after-rc1 Used by release team to mark a release-blocker issue as okay to resolve either before or after rc1 label Jun 17, 2024
@gopherbot gopherbot removed the okay-after-rc1 Used by release team to mark a release-blocker issue as okay to resolve either before or after rc1 label Jun 21, 2024
@gopherbot
Copy link
Contributor

Change https://go.dev/cl/594021 mentions this issue: [release-branch.go1.23] cmd: vendor golang.org/x/telemetry@a740542

@gopherbot
Copy link
Contributor

Change https://go.dev/cl/594019 mentions this issue: [release-branch.go1.23] cmd/vendor: pull in golang.org/x/telemetry@b4de734

@dmitshur dmitshur changed the title cmd/go: revendor x/telemetry prior to go1.23rc1 cmd/go: revendor x/telemetry prior to go1.23.0 Jun 21, 2024
@joedian
Copy link

joedian commented Jun 26, 2024

@findleyr I see the last cl on this was abandoned. Is this issue resolved now?

@findleyr
Copy link
Member Author

Not resolved. The last CL was abandoned because it will instead be performed by the release team as part of the backporting process.

Comment in #67617 (comment) still applies: as a matter of process, I want to ensure we have the correct telemetry code prior to the final release.

@gopherbot
Copy link
Contributor

Change https://go.dev/cl/597896 mentions this issue: cmd: vendor golang.org/x/telemetry@268b4a8ec2d7

@github-project-automation github-project-automation bot moved this from In Progress to Done in Release Blockers Jul 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

No branches or pull requests

6 participants
@dmitshur @dr2chase @gopherbot @joedian @findleyr and others