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

Remove deprecated flags in v1.26 #6574

Merged
merged 8 commits into from Dec 5, 2022

Conversation

dereknola
Copy link
Contributor

Proposed Changes

For 1.26 we are ready to take the next step with our deprecated flags. The following have been completely removed.

  • no-flannel
  • cluster-secret
  • no-deploy
  • disable-selinux

And wireguard backend for flannel has been converted to a fatal error. It will be completely removed in 1.27

Types of Changes

Flag deprecation

Verification

Attempt to start a server with:

  • no-flannel
  • cluster-secret
  • no-deploy
  • disable-selinux

Ensure that k3s complains about a nonexistent flag

Attempt to start the server with:

  • flannel-backend=wireguard

Ensure that k3s complains with a fatal error and details how to migrate to the native wireguard.

Testing

Linked Issues

#6333

User-Facing Change


Further Comments

Copy link
Contributor

@nikolaishields nikolaishields left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Derek Nola <derek.nola@suse.com>
Signed-off-by: Derek Nola <derek.nola@suse.com>
Signed-off-by: Derek Nola <derek.nola@suse.com>
Signed-off-by: Derek Nola <derek.nola@suse.com>
Signed-off-by: Derek Nola <derek.nola@suse.com>
Signed-off-by: Derek Nola <derek.nola@suse.com>
Signed-off-by: Derek Nola <derek.nola@suse.com>
Signed-off-by: Derek Nola <derek.nola@suse.com>
@dereknola dereknola changed the title [Wait For 1.26] Remove deprecated flags Remove deprecated flags in v1.26 Dec 5, 2022
@dereknola dereknola merged commit d723775 into k3s-io:master Dec 5, 2022
@dereknola dereknola deleted the remove_deprecated_126 branch December 7, 2022 19:17
@rbrtbnfgl
Copy link
Contributor

rbrtbnfgl commented Dec 14, 2022

This solves also #6440

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