-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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 release-1.24 to go1.19 #113956
update release-1.24 to go1.19 #113956
Changes from 5 commits
0181a46
64b323a
8ef165a
ad796a0
f978e33
a8f6d00
be208e4
0ed6d6c
15d9d19
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1 @@ | ||
v1.24.0-go1.18.9-bullseye.0 | ||
v1.24.0-go1.19.4-bullseye.0 |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -480,6 +480,7 @@ EOF | |
local go_version | ||
IFS=" " read -ra go_version <<< "$(GOFLAGS='' go version)" | ||
local minimum_go_version | ||
# TODO(liggitt): Need to switch this to 1.19 once we update images to newer go version | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. needs to switch? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. yeah, I'll update this once we switch presubmit images to run with go1.19, it would have turned all CI jobs red before that There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Actually, I think we should avoid bumping this until we're happy with post-submits and maybe until we cut the next 1.24 patch release. As soon as we do this, anything currently happily building off of release-1.24 with go1.18 will start failing, and I don't want to ripple out build/config changes past the immediate k8s.io/test-infra and k8s.io/release updates we're making quite yet There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. opened #114616 with this follow-up |
||
minimum_go_version=go1.18.1 | ||
if [[ "${minimum_go_version}" != $(echo -e "${minimum_go_version}\n${go_version[2]}" | sort -s -t. -k 1,1 -k 2,2n -k 3,3n | head -n1) && "${go_version[2]}" != "devel" ]]; then | ||
kube::log::usage_from_stdin <<EOF | ||
|
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 should be
1.<major>
, i.e.1.19
. See on master for example:kubernetes/build/dependencies.yaml
Line 105 in 7814d10
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.
As discussed on Slack, it will be fixed in a follow-up PR along with the comment from @aojea.
Ref: https://kubernetes.slack.com/archives/C2C40FMNF/p1671026815368429
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.
opened #114616 with this follow-up