machine-controller-manager
machine-controller-manager copied to clipboard
Improve overshooting logic
Issue
Machine safety controller only checks for machine objects while overshooting VMs
Solution
Machine safety controller to check infra VMs instead of machine objects while freezing for overshooting.
Grooming Discussion Results:
- Nobody knows why current overshooting freeze logic checks for machine object count instead of VM (aka instance) count. History on this was lost and not captured in issue.
- The use-case for user-driven overshooting freeze is understood but why should automatic freeze be necessary is still not clear. Is it just a nice to have safety net or needed from past experience.
- Currently only the MachineSet controller ignores replica change when overshooting freeze is enabled. The Machine Deployment controller only logs the overshooting freeze condition.
- Machine Set Overshooting freeze is separate and independent from Machine Controller lifecycle freeze.
- Some limited background for original motivation of this feature covered in #3