Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Revert "Remove default-config and validate sub-commands" #5479

Merged
merged 1 commit into from
Jan 27, 2025

Conversation

twz123
Copy link
Member

@twz123 twz123 commented Jan 27, 2025

Reverts #5376

There's a bug in k0sproject/version that causes k0sctl's version-based detection to make false assumptions about the supported sub-commands in k0s. That needs to be addressed first before we can remove these deprecated commands for realz.

@twz123 twz123 force-pushed the revert-5376-remove-deprecated-subcommands branch 2 times, most recently from 11555f3 to a056eb1 Compare January 27, 2025 08:05
@@ -0,0 +1,43 @@
/*
Copyright 2021 k0s authors
Copy link
Contributor

Choose a reason for hiding this comment

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

The copyright linter complains about this https://github.com/k0sproject/k0s/blob/main/hack/copyright.sh#L23-L25

We could either remove the -1 flag from git log in the copyright script and then it works, or we can update the copyright to 2025.

Copy link
Member Author

Choose a reason for hiding this comment

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

Damn. Did a force push and forgot to add the local change first ... that's what you get when you're in a hurry ... 😞

Signed-off-by: Tom Wieczorek <twieczorek@mirantis.com>
@twz123 twz123 force-pushed the revert-5376-remove-deprecated-subcommands branch from a056eb1 to 5310ef9 Compare January 27, 2025 11:02
@twz123 twz123 marked this pull request as ready for review January 27, 2025 11:48
@twz123 twz123 requested review from a team as code owners January 27, 2025 11:48
@twz123 twz123 requested review from kke and juanluisvaladas January 27, 2025 11:48
@twz123 twz123 enabled auto-merge January 27, 2025 11:48
@twz123 twz123 merged commit 4606c46 into main Jan 27, 2025
90 checks passed
@twz123 twz123 deleted the revert-5376-remove-deprecated-subcommands branch January 27, 2025 11:48
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants