Skip to content
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 k8s 1.19 #2098

Merged
merged 10 commits into from Aug 29, 2020
Merged

Update to k8s 1.19 #2098

merged 10 commits into from Aug 29, 2020

Conversation

erikwilson
Copy link
Contributor

@erikwilson erikwilson commented Aug 5, 2020

Proposed changes

k8s update to v1.19.0

Types of changes

Component Upgrade

Verification

Needs certified conformance test for cncf

Linked Issues

For #2020

Further comments

@brandond
Copy link
Contributor

brandond commented Aug 5, 2020

Are we going to follow k/k up to golang 1.15?

@erikwilson
Copy link
Contributor Author

The minimum requirement for k8s 1.19 is golang 1.14.4, and since 1.15 is not officially out yet I think we should stick with 1.14.

go.mod Outdated Show resolved Hide resolved
@erikwilson erikwilson force-pushed the k8s-1.19 branch 18 times, most recently from af4c29a to 274e74d Compare August 12, 2020 14:58
@erikwilson erikwilson force-pushed the k8s-1.19 branch 2 times, most recently from 88c1c20 to 9d9f03f Compare August 26, 2020 22:14
@brandond
Copy link
Contributor

If you're waiting for a LGTM to push tags to rancher/kubernetes, then: LGTM

@erikwilson erikwilson force-pushed the k8s-1.19 branch 2 times, most recently from 944476f to 78a5dd9 Compare August 28, 2020 20:47
@erikwilson erikwilson changed the title WIP- Update to k8s 1.19 Update to k8s 1.19 Aug 28, 2020
@erikwilson erikwilson marked this pull request as ready for review August 28, 2020 22:31
@erikwilson erikwilson requested a review from a team as a code owner August 28, 2020 22:31
Dockerfile.manifest Outdated Show resolved Hide resolved
@erikwilson erikwilson force-pushed the k8s-1.19 branch 2 times, most recently from aa586a4 to fbb7343 Compare August 28, 2020 23:24
@erikwilson erikwilson requested a review from a team August 29, 2020 00:03
Copy link
Contributor

@brandond brandond left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🍾🍾🍾

@erikwilson erikwilson merged commit 447097a into k3s-io:master Aug 29, 2020
@davidnuzik
Copy link
Contributor

davidnuzik commented Aug 31, 2020

@warmchang go was bumped to 1.15 you can see the commit here debf800 or look above

@warmchang
Copy link
Contributor

@davidnuzik Actually I want to say whether we need to change the go.mod from "go 1.13" to "go 1.15":
https://github.com/rancher/k3s/blob/447097a597532ae710d22ff471c97d845720e1cf/go.mod#L3

@brandond
Copy link
Contributor

brandond commented Sep 1, 2020

We probably could bump that as well, but it shouldn't be required as per golang/go#30791 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants