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
Updates deps in order to remove rsc.io/sampler #97337
Conversation
/remove-sig instrumentation |
/triage accepted |
/lgtm /assign @cheftako for approval. |
/lgtm |
No code changes in vendor? |
Nope! For reasons I still don't completely understand, go modules can get resolved and end up as part of the go.mod file even if they are never compiled into the final binaries. This issue is the closest to an explanation I've found: golang/go#27900 So, this change should not have any actual effect. I think it's still a good practice to keep these type of things cleaned up and out of the go.mod files where possible to make future updates easier, though. |
/test all |
I noticed rsc.io/sampler was in here, which didn't make any sense. It appears to have been mistakenly included in a transitive dependency at some point. This PR attempts to fix that by updating the intermediate deps far enough to stop requiring the rsc.io packages.
/lgtm |
/assign @liggitt |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: BenTheElder, cheftako, dlorenc, liggitt The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This re-removes rsc.io/quote and rsc.io/sampler from the go.mod. They never made it into the vendor/ tree, but still contribute to dependency resolution complexity. These were originally removed in kubernetes#97337 but slipped back in. Signed-off-by: Dan Lorenc <dlorenc@google.com>
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
I noticed rsc.io/sampler was in here, which didn't make any sense.
It appears to have been mistakenly included in a transitive dependency
at some point.
This PR attempts to fix that by updating the intermediate deps far enough
to stop requiring the rsc.io packages.
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: