-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
Update vendor package github.com/hashicorp/golang-lru #74260
Update vendor package github.com/hashicorp/golang-lru #74260
Conversation
@stafot: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Hi @stafot. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@stafot we are now asking for a justification for updating the dependency, can you please see the list and see what will fit here? I've also updated #69776 with the same guidance |
@dims I am not sure that it applies in any of the justifications reasons, though before this update kubernetes uses a |
/ok-to-test @stafot thanks! let's see what others say first. |
This is still tagged as a pre-release (0.5.0). Do we have evidence of changes that matter to us? I'm inclined to allow this, since it's better than a random hash, but we should be pushing back on deps that are 0.x when we can. |
/test pull-kubernetes-godeps |
Test failure is legitimate. Please run |
6546ba7
to
df3fbf9
Compare
/lgtm Yeah, I'm inclined to let this one in too, even if it's prerelease. I don't see any changes of concern. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cblecker, stafot 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 |
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Part of #69776 involves updating github.com/hashicorp/golang-lru
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):N/A
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE