diff --git a/enhancements/machine-api/short-circuiting-backoff.md b/enhancements/machine-api/short-circuiting-backoff.md index 124d182ac5e..f5011cd8569 100644 --- a/enhancements/machine-api/short-circuiting-backoff.md +++ b/enhancements/machine-api/short-circuiting-backoff.md @@ -108,8 +108,8 @@ MachineHealthCheck: ``` ### Risks and Mitigations - -No known risks. +In case `FailedNodeStartupTimeout` is undefined default behaviour is preserved (i.e. remediation is not postponed). +The Pro is that naive users aren't being surprised with a new behavior however, the con is that naive users do benefit from the new behavior. ## Design Details @@ -125,25 +125,18 @@ The existing remediation tests will be reviewed / adapted / extended as needed. TBD -#### Examples - -TBD - -##### Dev Preview -> Tech Preview +#### Dev Preview -> Tech Preview TBD -##### Tech Preview -> GA +#### Tech Preview -> GA TBD -##### Removing a deprecated feature - +#### Removing a deprecated feature ### Upgrade / Downgrade Strategy -### Version Skew Strategy - ## Implementation History - [x] 03/01/2021: Opened enhancement PR @@ -154,12 +147,6 @@ no known drawbacks ## Alternatives -- Instead of delaying, canceling the remediation for failed machines. - -## Infrastructure Needed [optional] - -Use this section if you need things from the project. Examples include a new -subproject, repos requested, github details, and/or testing infrastructure. +In case a machine enters the `Failed` state and does not have a NodeRef or a ProviderID do not perform remediation on that machine. -Listing these here allows the community to get the process for these resources -started right away. +This alternative is simpler since it does not require `FailedNodeStartupTimeout` however it does not allow an option to retain the system previous behaviour and relies completely on manual fix of the failed machines. \ No newline at end of file