-
Notifications
You must be signed in to change notification settings - Fork 114
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 dependencies: K8S, controller-runtime and Go #495
Conversation
Thanks for your PR,
To skip the vendors CIs use one of:
|
Pull Request Test Coverage Report for Build 6011025912
💛 - Coveralls |
Thanks for your PR,
To skip the vendors CIs use one of:
|
This commit bumps Go to v1.20, K8S to v0.27.4 and the controller-runtime to v0.15.2 Updated go.mod, go.sum and the vendor dir. Also bumps golangci-lint version to latest to support Go v1.20 Signed-off-by: Lior Noy <[email protected]>
This commit fixes the breaking changes from the version bump. Signed-off-by: Lior Noy <[email protected]>
This commit updates go to v1.20 in the workflows and Dockerfiles. Signed-off-by: Lior Noy <[email protected]>
Thanks for your PR,
To skip the vendors CIs use one of:
|
Some errors were raised by the golangci-lint. I will handle them soon. Meanwhile reviews are more than welcomed. |
Bump the golangci-lint version to support Go v1.20 and remove deprecated linters. Signed-off-by: Lior Noy <[email protected]>
Thanks for your PR,
To skip the vendors CIs use one of:
|
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.
LGTM
thanks!
In this PR we bump different core dependencies to be more updated with current releases
and avoid incompatibilities. The dependencies we bump here are:
K8S to 1.27
controller-runtime to v0.15
Go to v1.20