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

UX: warn if using talosctl less than server version #9474

Open
steverfrancis opened this issue Oct 8, 2024 · 2 comments
Open

UX: warn if using talosctl less than server version #9474

steverfrancis opened this issue Oct 8, 2024 · 2 comments

Comments

@steverfrancis
Copy link
Collaborator

This seems worth having talosctl issue a warning about:

Maybe it's worth breaking the talosctl invocation when server [major.minor] version > talosctl [major.minor] version ? Such invocation is most times incorrect or even dangerous.

Originally posted by @vladimirfx in #9469 (comment)

@vladimirfx
Copy link

The original idea was to stop some operations if the server (machined) is a higher version than the client (talosctl). That is because such operations can damage installation in the worst case.
Proposed operations:

  • apply-config
  • etcd
  • inject
  • machineconfig
  • meta
  • reset
  • upgrade
  • upgrade-k8s

And other modifying machine config and k8s config.

@smira
Copy link
Member

smira commented Oct 15, 2024

If you have examples of damaging the Talos install, it'd be interesting to see. So far it's more of a UX issue that the command fails until it is in sync with Talos version.

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

No branches or pull requests

3 participants