-
Notifications
You must be signed in to change notification settings - Fork 38.6k
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
Update to go 1.12.8 #81390
Update to go 1.12.8 #81390
Conversation
/priority critical-urgent |
Still to be done:
|
/assign |
/assign @javier-b-perez |
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
/approve
I built / pushed an image, the CL is pending |
/lgtm |
/retest |
/test pull-kubernetes-verify |
/test pull-kubernetes-bazel-test |
The port name previously didn't matter on these tests, but is now actively being checked in go1.12.8 and higher. golang/go@3226f2d
Pushed up a new commit to fix a test failure: Tests under go1.12.7:
Tests under go1.12.8:
Tests under go1.12.8 after fix:
|
/test pull-kubernetes-integration |
/retest |
@@ -159,7 +159,7 @@ func TestWithValidThumbprint(t *testing.T) { | |||
func TestWithInvalidCaCertPath(t *testing.T) { | |||
connection := &vclib.VSphereConnection{ | |||
Hostname: "should-not-matter", | |||
Port: "should-not-matter", |
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.
This could have been a fixture too may be. But its "doesn't matter" so may be we can live with this.
/lgtm |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: BenTheElder, cblecker, imkin 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 |
@@ -16,7 +16,7 @@ | |||
|
|||
import boilerplate | |||
import unittest | |||
import StringIO | |||
from io import StringIO |
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.
seems unrelated? how did the go bump switch this to python3?
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.
Bazel was failing this test locally on my side when I bumped things. May be my setup?
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.
ah. I thought we had it configured to force python2 /shrug
as long as it passes in CI, seems good to me
Update to go 1.12.8
What type of PR is this?
/kind bug
What this PR does / why we need it:
Update to go 1.12.8
Which issue(s) this PR fixes:
ref #79912
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: