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
kubeadm: update the version constants for 1.15 #80834
kubeadm: update the version constants for 1.15 #80834
Conversation
/release-note-none |
- common_test.go: use constants.CurrentKubernetesVersion - diff_test.go: write temporary files instead of using testdata. this allows us to not have to bump kubernetesVersions in the testdata files (now removed) - policy_test.go: apply fixes to tests that were previously passing, but a bump in constants.go breaks them. these tests now work for any version.
69bc3cf
to
fc9e04f
Compare
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.
@neolit123 thanks for this fix!
@ patch team, sorry for the noise. we are changing how to track before release tasks in order to avoid this to happen again
/approve
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: fabriziopandini, neolit123 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 |
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.
Thanks @neolit123 !
Sure, this does more than it's expected, but it's mostly test changes, so I am OK with that.
/lgtm
/cc @kubernetes/patch-release-team |
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
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.
Thanks!
LGTM
this is not a cherry pick, but rather we forgot to update some constants in for kubeadm in 1.15.
similar PR for 1.16:
#80833
while this is a cleanup WRT to 1.16, it is a bug in kubeadm 1.15 and can result in problems, if say the user attempt to deploy a version of the cluster that is way too old.
we have plans to automate the update of these constants except there are complications and they are tracked in: kubernetes/kubeadm#1260
sorry for the noise, patch team.
/assign @fabriziopandini @rosti
/cc @yagonobre
/kind bug
/priority critical-urgent