-
Notifications
You must be signed in to change notification settings - Fork 119
Update to Go 1.20 and latest KRTE for e2e tests #257
Conversation
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: adrianludwin 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 |
|
/assign @rjbez17 |
|
Filed kubernetes/test-infra#28722 to fix presubmit |
|
@adrianludwin |
|
/lgtm |
|
/retest Now that kubernetes/test-infra#28722 is fixed |
|
/retest |
|
cc @rjbez17 can you please re-approve this? I forgot to push my latest changes :P cc @mochizuki875 sorry about that, please try again, it should be fixed now |
E2Es broke a few days ago, likely when test-infra upgraded to 1.20. I also couldn't run staticcheck on my Google workstation, which has 1.20, with the same failure message. Upgrading staticcheck fixed the problem; I also took the opportunity to match the test-infra krte image as well as the Golang version (which caused some minor reformatting), and also made the integ tests hermetic by including the setup-envtest tool (bug kubernetes-retired#252, based on an approach suggested in kubernetes-retired#253). Tested: couldn't build on my workstation on Go 1.20 before this change; can after it. Can also run integ tests after deleting local output of setup-envtest.
|
/lgtm |
|
@adrianludwin |
|
Great, thanks!
…On Wed, Feb 15, 2023 at 8:52 AM Keita Mochizuki ***@***.***> wrote:
@adrianludwin <https://github.com/adrianludwin>
I tried again and confirmed it was fixed.
Thank you for your work!
—
Reply to this email directly, view it on GitHub
<#257 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AE43PZCLGCC5J6M5PSGSH2LWXTNRXANCNFSM6AAAAAAUYEF2NA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
E2Es broke a few days ago, likely when test-infra upgraded to 1.20. I
also couldn't run staticcheck on my Google workstation, which has 1.20,
with the same failure message. Upgrading staticcheck fixed the problem;
I also took the opportunity to match the test-infra krte image as well
as the Golang version (which caused some minor reformatting), and also
made the integ tests hermetic by including the setup-envtest tool (bug
#252, based on an approach suggested in #253).
Tested: couldn't build on my workstation on Go 1.20 before this change;
can after it. Can also run integ tests after deleting local output of
setup-envtest.
Fixes #252