-
Notifications
You must be signed in to change notification settings - Fork 457
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix: make
apply-config
work reliably in any Talos state
Fixes #4587 The gist is removing `ApplyConfiguration` sequence and refactoring whole flow. We can break down any `ApplyConfiguration` mode into following steps: * validate incoming config * apply dynamic config patches (we should get rid of these eventually) * write down the config to `/state` If we run in `--immediate` mode, we should also apply configuration in-memory immediately. If we run in default mode (apply with reboot), we should actually reboot the machine (equivalent of `talosctl reboot`), no matter if the sequencer is stuck in the boot sequence right now. This fixes mostly apply with reboot mode with following changes: * machine config is no longer applied in memory (it should be only applied after a reboot) * sequence reboot runs with take over, so it overrides sequencer locks if the machine is stuck in boot sequence and config change is required to fix that Signed-off-by: Andrey Smirnov <andrey.smirnov@talos-systems.com>
- Loading branch information
Showing
4 changed files
with
34 additions
and
95 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters