-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Bump k8s.io/client-go from 0.29.4 to 0.30.0 in /addon-resizer #6735
Conversation
Hi @dependabot[bot]. 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. |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: dependabot[bot] The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/ok-to-test |
/lgtm |
@@ -1,22 +1,23 @@ | |||
module k8s.io/autoscaler/addon-resizer | |||
|
|||
go 1.21 | |||
toolchain go1.22.2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this change is correct. It depends on what we build with, but right now this is being set to the go version dependabot is using.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, you are right @kwiesmueller, both should be same.
The toolchain is added now, previously it was not there.
https://github.com/kubernetes/autoscaler/blob/addon-resizer-release-1.8/addon-resizer/go.mod
It may work if we change go v1.21 to v1.21.0
https://go.dev/doc/toolchain
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As per above comment, removing lgtm
from the PR
/lgtm cancel
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, @kwiesmueller your thoughts on this.
e34a9e0
to
c20ee45
Compare
Bumps [k8s.io/client-go](https://github.com/kubernetes/client-go) from 0.29.4 to 0.30.0. - [Changelog](https://github.com/kubernetes/client-go/blob/master/CHANGELOG.md) - [Commits](kubernetes/client-go@v0.29.4...v0.30.0) --- updated-dependencies: - dependency-name: k8s.io/client-go dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
c20ee45
to
9464709
Compare
Superseded by #6839. |
Bumps k8s.io/client-go from 0.29.4 to 0.30.0.
Commits
3aa4577
Update dependencies to v0.30.0 tag2df4de1
Merge remote-tracking branch 'origin/master' into release-1.30ade2ae2
Update x/net for CVE-2023-45288b4632b7
Merge pull request #123932 from pohly/dra-api-resource-model-rename4467b1e
Merge pull request #123909 from AkihiroSuda/fix-123906650f392
dra api: NodeResourceModel -> ResourceModel00e4609
api: NodeStatus: rename RuntimeClasses to RuntimeHandlers7ebe0ea
Merge pull request #123180 from AkihiroSuda/rro3be09aa
api: KEP-3857: Recursive Read-only (RRO) mounts110b75b
Merge pull request #123344 from nilekhc/svm-controllerDependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase
.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebase
will rebase this PR@dependabot recreate
will recreate this PR, overwriting any edits that have been made to it@dependabot merge
will merge this PR after your CI passes on it@dependabot squash and merge
will squash and merge this PR after your CI passes on it@dependabot cancel merge
will cancel a previously requested merge and block automerging@dependabot reopen
will reopen this PR if it is closed@dependabot close
will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot show <dependency name> ignore conditions
will show all of the ignore conditions of the specified dependency@dependabot ignore this major version
will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor version
will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependency
will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)