-
Notifications
You must be signed in to change notification settings - Fork 16
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
Install RKE2 by default #437
Conversation
51db170
to
c1f2b6b
Compare
We have to make sure the uninstall process is not broken with this change, which IMO would probably be the case, since we are currently deleting deployment/webhooks specific to Kubeadm only in post-delete hook |
c1f2b6b
to
3977722
Compare
3977722
to
f963f8e
Compare
f963f8e
to
9504e5c
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.
Thanks, this looks good to me.
d730e2a
9504e5c
to
d730e2a
Compare
ea47d2d
to
920e21a
Compare
Signed-off-by: Alexandr Demicev <alexandr.demicev@suse.com>
920e21a
to
bc7de99
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.
The CI test is flaky but we can fix that later. I think this is good to go.
What this PR does / why we need it:
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 #
Special notes for your reviewer:
Checklist: