-
Notifications
You must be signed in to change notification settings - Fork 155
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
Default upgrade #6899
Default upgrade #6899
Conversation
@emmanuel-kubermatic Overall LGTM. In order to fix the failing tests you should run:
and:
|
Thanks a ton !
…On Fri, 16 Apr 2021 at 14:57, Iacopo Rozzo ***@***.***> wrote:
@emmanuel-kubermatic <https://github.com/emmanuel-kubermatic> Overall
LGTM. In order to fix the failing tests you should run:
hack/update-docs.sh
hack/update-kubermatic-chart.sh
and:
gofmt -s -w pkg/controller/operator/common/defaults.go
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#6899 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ATIC5CXFR467GZOHL5IIZNDTJA645ANCNFSM43BK5J6A>
.
|
501533d
to
4af2693
Compare
@@ -190,6 +190,8 @@ var ( | |||
DefaultKubernetesVersioning = operatorv1alpha1.KubermaticVersioningConfiguration{ | |||
Default: semver.MustParse("v1.19.3"), | |||
Versions: []*semver.Version{ | |||
// Kubernetes 1.16 | |||
semver.MustParse("v1.16.13"), |
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.
Mmm, I was thinking that putting back a v1.16
version just to put the auto-upgrade from 1.15
to 1.16
does not seem to be a good option. This version will be visible to users. I think that for the long term we should rework the auto-upgrade mechanism to cover the case where the destination minor version is not supported, but for the moment I would just remove this and the auto-upgrade 1.15
-> 1.16
, customers that still have 1.15 clusters will need to override the default settings to be able to upgrade. WDYT?
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.
That's fine
If that cluster version is Out Of Life, no point for the customer using it
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.
Ok, you can remove this version then
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.
I've removed those and updating the docs for the next push
- type: kubernetes | ||
version: 1.16.13 |
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.
Remember to remove this too, and to regenerate the charts ;-)
6c82a2f
to
b460aaf
Compare
Done
…On Fri, 30 Apr 2021 at 13:54, Iacopo Rozzo ***@***.***> wrote:
***@***.***A* commented on this pull request.
------------------------------
In pkg/controller/operator/common/defaults.go
<#6899 (comment)>
:
> @@ -190,6 +190,8 @@ var (
DefaultKubernetesVersioning = operatorv1alpha1.KubermaticVersioningConfiguration{
Default: semver.MustParse("v1.19.3"),
Versions: []*semver.Version{
+ // Kubernetes 1.16
+ semver.MustParse("v1.16.13"),
Ok, you can remove this version then
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#6899 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ATIC5CSGHAEWRWXSBX5NUVDTLKSALANCNFSM43BK5J6A>
.
|
/lgtm |
LGTM label has been added. Git tree hash: 327755ec1d476fb75684318cbd5d25c538cbe1fa
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: emmanuel-kubermatic, irozzo-1A 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 |
/retest |
What this PR does / why we need it:
The PR fixes #6683 and allows default upgrades on startup
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #6683
Special notes for your reviewer:
Documentation:
https://github.com/kubermatic/kubermatic/blob/master/docs/dev-getting-started.md
Does this PR introduce a user-facing change?: