-
Notifications
You must be signed in to change notification settings - Fork 238
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 Go to 1.18 #785
Bump Go to 1.18 #785
Conversation
Hold on after -> kubernetes/test-infra#25651 |
/hold |
golangci-lint@1.45 now supports go1.18 we are good to review this |
Signed-off-by: Carlos Eduardo Arango Gutierrez <carangog@redhat.com>
96c2857
to
87b29f6
Compare
Test-infra pair -> kubernetes/test-infra#25651 |
@marquiz can this make it to v0.11? or defer? |
I suggest to defer to minimize risks and hassle on that front |
v0.11 is cut, we can circle back to this |
Co-authored-by: Markus Lehtonen <markus.lehtonen@intel.com>
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.
Thanks @ArangoGutierrez, looks good 👍
Let's wait for kubernetes/test-infra#25651 to land first.
/hold
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ArangoGutierrez, marquiz 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 |
/test all |
/retest |
/lgtm |
You need to make sure that 1.18.1+ is used. 1.18.0 had regression in PEM certificates handling which was fixed very recently. |
Yeah. Looks like the docker images were also published a few hours ago |
/test all |
Signed-off-by: Carlos Eduardo Arango Gutierrez carangog@redhat.com