-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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.mod to use go 1.13 #2957
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: saschagrunert 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 |
1 similar comment
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: saschagrunert 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 |
cdffcd5
to
27b22bc
Compare
768eb3f
to
71d72ad
Compare
/retest |
Change LGTM, but the tests don't. |
Failures are from an ansible bump causing the provisioning code to fail |
Is there anything I can do to solve this issue? Where did the ansible update happen? |
I'm not really sure, I think our best bet is to fix the ansible ATM |
Could this help fix the failures I am seeing in #2947? |
woah, gnarly. Yeah, it may |
71d72ad
to
9f7a984
Compare
@saschagrunert: The following tests failed, say
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. I understand the commands that are listed here. |
To avoid inconsistent behavior between go versions we now use go 1.13 per default. Signed-off-by: Sascha Grunert <sgrunert@suse.com>
9f7a984
to
509c3fd
Compare
@haircommander I think the focus for the |
uhhh I don't know why that's happening. Through, e2e_features* don't test anything e2e* doesn't so it's not a huge loss. definitely something to look into though. |
Test are green again, please take a look. |
/lgtm |
/lgtm |
thanks prow |
To avoid inconsistent behavior between go versions we now use go 1.13 per
default.