-
Notifications
You must be signed in to change notification settings - Fork 39.3k
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
Move to golang 1.12.1 official image #75422
Move to golang 1.12.1 official image #75422
Conversation
We used 1.12.0 + hack to download 1.12.1 binaries as we were in a rush on friday since the images were not published at that time. Let's remove the hack now and republish the kube-cross image Change-Id: I3ffff3283b6ca755320adfca3c8f4a36dc1c2b9e
@dims: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it. 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. |
/assign @BenTheElder @ixdy |
/release-note-none |
related to #75372 |
/assign @BenTheElder @ixdy @tpepper @spiffxp |
/test pull-kubernetes-kubemark-e2e-gce-big |
/kind cleanup |
/sig release |
Need to bump the version file as well to 1.12.1-2 |
Ack @liggitt @ixdy if i remember right you had mentioned that we don't bump version numbers, is that the same for the Jordan, Thanks! |
cc @jpbetz basically we had a custom golang 1.12.1 image for a couple of days, but now we need to use the official one, thus having a revision:
seems appropriate? |
The first part of the kube-cross version (before the dash) refers to the golang version. The piece after the dash is the image revision. So |
added a commit to bump to |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dims, ixdy 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 |
(need to build/push the new kube-cross image before CI will go green on this PR) |
I'm building the images now... should take ~2 hours. |
/test pull-kubernetes-cross |
the image k8s.gcr.io/kube-cross:v1.12.1-2 is available now |
/retest |
and thanks @javier-b-perez ! |
We used 1.12.0 + hack to download 1.12.1 binaries as we were in a rush
on friday since the images were not published at that time. Let's remove
the hack now and republish the kube-cross image
Change-Id: I3ffff3283b6ca755320adfca3c8f4a36dc1c2b9e
What type of PR is this?
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?: